Thread Fileupload mit Statusanzeige (17 answers)
Opened by Paulaner at 2010-08-08 16:21

pq
 2010-08-08 16:34
#140471 #140471
User since
2003-08-04
12208 Artikel
Admin1
[Homepage]
user image
mir sind sofort 3 sachen aufgefallen:
  • du setzt POST_MAX erst nach dem new CGI. an der stelle duerfte das zu spaet sein, da das CGI-objekt dann schon erstellt wurde.
  • mkdir() erstellt ein verzeichnis, wenn es noch nicht vorhanden ist, ansonsten tut es nichts und beschwert sich auch nicht. deine logik ist also genau verkehrt herum. du fragst bei dem einen verzeichnis vorher ab und machst dann ein mkdir(). dort koenntest du die abfrage auch sein lassen. beim zweiten schreibst du als kommentar direkt vor das mkdir, dass es das verzeichnis noch nicht geben darf, aber mkdir() stoert das wie gesagt nicht. es wuerde also weiterlaufen, auch wenn das verzeichnis schon existiert. machev hier zumindest eine abfrage auf den rueckgabewert, der dir sagt, ob mkdir ein verzeichnis erstellt hat oder nicht.
  • du benutzt einen filehandle namens UPLOADFILE und machst dann aber binmode uploadfile. das sollte zumindest eine warnung ausgeben. das ist nicht derselbe filehandle.
Always code as if the guy who ends up maintaining your code will be a violent psychopath who knows where you live. -- Damian Conway in "Perl Best Practices"
lesen: Wiki:Wie frage ich & perlintro Wiki:brian's Leitfaden für jedes Perl-Problem

View full thread Fileupload mit Statusanzeige