:: Buchempfehlung ::

Silencer · #1 16.09.2002, 17:32:15

Kann mir jemand helfen?
was ist am folgenden code falsch?
es ist ein upload script das nur für bilddateien mit der endung bmp jpg oder gif ist und zusätzlich einen link ausgibt wie man das bild zb in einem newssystem anzeigt

PHP-Code:


			
<?php

 $file_dir="uploads/";



 $url_prefix="URL zur Domain";

 $url_prefix2="2te url als relativer link";

 if($userfile) {

  $userfile_name=basename($userfile_name);

if(!stristr($userfile_name, ".gif")||!stristr($userfile_name, ".jpg")||!stristr($userfile_name, ".bmp") ) { 

    die("Sorry es sind nur BMP, JPG oder GIF erlaubt.");

  }

  if(copy($userfile, $file_dir.$userfile_name)) {

    unlink($userfile);

    echo "Image vollständig hochgeladen.<br>";

    echo "Nimm als Url <h3>$url_prefix$userfile_name</h3>";

    echo "oder <h3>$url_prefix2$userfile_name</h3> damit das Bild in den News erscheint";

  } else {

    echo "Upload fehlgeschlagen...";

  }

 }



 ?>

MiH · #2 16.09.2002, 17:42:06

ich persönlich würde die prüfung nicht auf der dateiendung aufbauen da auch eine *.zip als *.bmp durchgeht wenn man einfach nur ein *.zip.bmp anhängt.

besser ist es da, wenn du den upload-type prüfst.
das machst du per $_FILES["userfile"]["type"] nun liefert dir das array element beispielsweise bei einem *.jpg den wert "image/jpeg" mit einem preg_match kannst du nun prüfen ob es eine erlaubte datei ist oder nicht.

statt copy + unlink kannst du einfach rename() benutzen das hat den gleichen effekt.

Silencer · #3 16.09.2002, 17:49:38

und wie baue ich das in das script ein?

sorry aber ich habe fast 2 wochen gebraucht um dieses uploadscript zu basteln. ich bin darin noch ein totaler n00b

kann man mir das korrigierte per pm oder sowas schicken?

MiH · #4 17.09.2002, 08:23:58

Code:

<html>
<head>
<title>Untitled Document</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
<body>
<?php
$file_dir="uploads/";
if(isset($_POST["submit"])){
 $a_patter = array("^image/gif$","^image/jpeg$","^image/png$","^image/x-MS-bmp$");
 if(preg_match($a_patter,$_FILE["usrfile"]["type"])){
  if($_FILE["usrfile"]["size"] > 0){
   rename($_FILE["usrfile"]["tmp_name"], $file_dir.$_FILE["usrfile"]["name"]);   
   echo "Image vollständig hochgeladen.<br />";
  }else{
   echo "Datei hat keinen inhalt...";
  }  
 }else{
  echo "ne es geht nur GIF, JPG, PNG und MS-BMP";
 }
?>
<form action="upload.php" method="post" enctype="multipart/form-data">
 <input type="file" name="usrfile" /><br />
 <input type="submit" name="submit" value="senden" /><br />
</form>
</body>
</html>

Silencer · #5 18.09.2002, 02:34:10

wenn ich das so mache bekomme ich zuerst einen parse error weil eine klammer fehlt und ist der fehler behoben bekomme ich sowas

Warning: Delimiter must not be alphanumeric or backslash in /home/web60/html/test.php on line 11
ne es geht nur GIF, JPG, PNG und MS-BMP

und zeile 11 ist das hier

Code:

 if(preg_match($a_patter,$_FILE["usrfile"]["type"])){

MiH · #6 18.09.2002, 08:36:41

das
$a_patter = array("^image/gif$","^image/jpeg$","^image/png$","^image/x-MS-bmp$");
durch das ersetzen
$a_patter = array("|^image/gif$|i","|^image/jpeg$|i","|^image/png$|i","|^image/x-MS-bmp$|i");

Silencer · #7 18.09.2002, 10:56:03

dann bekomme ich wieder die selbe fehlermeldung

Warning: Delimiter must not be alphanumeric or backslash in /home/web60/html/test.php on line 11
ne es geht nur GIF, JPG, PNG und MS-BMP

MiH · #8 18.09.2002, 23:36:02

sorry aber das passiert eben wenn man es nicht testen kann.
hier mal ein fnktionierender code.

Code:

<html>
<head>
<title>Untitled Document</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
<body>
<?php
$file_dir="uploads/";

if(isset($_POST["submit"])){ 
 $a_patter = array("|^image/gif$|","|^image/jpeg$|","|^image/png$|","|^image/x-MS-bmp$|");
 for($a=0;$a<count($a_patter);$a++){
  if(preg_match($a_patter[$a],$_FILES["usrfile"]["type"])){   
   if($_FILES["usrfile"]["size"] > 0){
    rename($_FILES["usrfile"]["tmp_name"], $file_dir.$_FILES["usrfile"]["name"]);   
    echo "Image vollständig hochgeladen.<br />";
    $ok = 1;
   }else{
    echo "Datei hat keinen inhalt...";
   }  
  }
 }
 
 if(!isset($ok)){
  echo "falscher dateityp";
 }
}
?>
<form action="_test.php" method="post" enctype="multipart/form-data">
 <input type="file" name="usrfile" /><br />
 <input type="submit" name="submit" value="senden" /><br />
</form>
</body>
</html>

arrays als patter geht nur bei preg_replace... *g*

Silencer · #9 19.09.2002, 11:47:55

jetzt versteh ich nix mehr???

wenn ich das jetzt so mache dann klappt der upload bei gif
bei jpg jpeg bmp png sagt der dann falscher dateityp

wieso das?

irgendwo ein kleiner fehler drin?

ich bin mit meinem latein am ende

MiH · #10 19.09.2002, 12:50:23

dann stimmen die angaben in den patter nicht.
es kann sein das der mime type bei PNG anders lautet als image/png und bei BMP genauso.

mein tip. lade einfach mal ein PNG + BMP hoch und lass dir den mime type über echo $_FILES["usrfile"]["type"]; ausgeben. dann siehst du wie der typ richtig heisst und du kannst ihn dann im array ersetzen.

:: Buchempfehlung ::

:: Anbieterverzeichnis ::

Globale Branchen

Informieren Sie sich über ausgewählte Unternehmen im Anbieterverzeichnis von SELFPHP

:: Newsletter ::

Abonnieren Sie hier den kostenlosen SELFPHP Newsletter!