:: Buchempfehlung ::

alffrom · #1 02.07.2006, 13:39:49

Hi, ich hab hier mal einen BilderUpload programmiert aber es haben sich Fehler eingeschlichen die ich leider nicht finde, könnt ihr mir da weiterhelfen?

Zitat:

Hier der code:

<html><head>
<title>HabboShout BilderUpload</title>
</head><body>
<?php
//bildupload
$uverz="dateien";
if($_POST['sent'])
{
//bilder
if(!file_exists($uverz))
{mkdir($uverz,0755);}
//Dateigröße kontrollieren
if($_FILES['neuedatei']['size']>100000)
{$fehler="<br>Die Datei ist zu groß";}
//Zulässige Dateitypen kontrollieren
if(!($_FILES['neuedatei']['type'] == 'image/jpeg' OR
$_FILES['neuedatei']['type'] == 'image/gif'))
{$fehler.="<br>Der Dateityp ist nicht zulässig.<br>";}
//Datei speicher, wenn kein Fehler vorliegt
if(!$fehler)
{
//Eindeutige Namen generieren
$name=uniqid("bild");
//Dateiendung je nach Typ an den Namen anhängen
switch($_FILES['neuedatei']['type'])
{
case 'image/jpeg':
$name.=".jpg";
break;
case 'image/gif':
$name.='.gif';
break;
}
//Den Dateinamen um die Pfadangabe erweitern
$name=$uverz."/".$name;
//Die Datei in das Unterverzeichnis kopieren
copy/$_FILES['neuedatei']['tmp_name'], $name);
}
else
{echo "<h2><font color=red>".$fehler."</font></h2>;}
}
//Ausgeben der bereits vorhandenen Datein im Unterverzeichnis
echo "<h2>Bisher hochgeladene Dateien</h2>";
$i=1;
$verzeichnis=opendir($uverz);
while ($file = readdir($verzeichnis))
{
if(is_file($uverz."/".$file))
{
echo "<a href=\"".$uverz."/".$file."\"target=\"_blank\">Datei_".$i."</a><br>
}
}
closedir($verzeichnis);
?>
<h2>Eine Datei hochladen</h2>
<form action="<? echo $_SERVER['PHP_SELF']; method="post"
enctype="multipart/form-data">
<input type="hidden" name="send" value="1">
<input type="submit"><input type="Reset" value="Zurücksetzen">
</form>
</body></html>

MatMel · #2 02.07.2006, 14:12:45

Was passiert denn ? Kommt ein Fehler, bricht es ab oder passiert einfach garnichts ?

Jürg · #3 02.07.2006, 14:18:49

Ich hab da mal was programmiert: Bildupload
Wahrscheinlich hilfts.

Andes · #4 02.07.2006, 14:41:11

Ist das denn wirklich so schwer ordentlich formatierten Quelltext hier ein zu stellen. Ich finde es langsam etwas frech von den Usern die Hilfe suchen und dann den Leuten, die helfen wollen, solche unstrukturierten Sachen vor die Nase zu setzten.
Also stell mal deinen Quelltext ordentlich strukturiert ein und nutze die php-Option über dem Textfeld. Oder setzt dem ein \[PHP\] vor und ein \[/PHP\] als Abschluss. Allerdings ohne die Maskierungen.

alffrom · #5 02.07.2006, 16:16:18

also sorry aber so lautet mein Quelltext :)

Andes · #6 02.07.2006, 16:45:21

Dann halt nicht, ich hab das Problem ja nicht. Wenn du nicht mal bereit bist das obige ordentlich strukturiert hier ein zu stellen. Warum soll man sich dann die Mühe machen dir zu helfen.

alffrom · #7 02.07.2006, 16:50:52

PHP-Code:


			
 <html><head>

<title>HabboShout BilderUpload</title>

</head><body>

<?php

//bildupload

$uverz="dateien";

if($_POST['sent'])

{

//bilder

if(!file_exists($uverz))

{mkdir($uverz,0755);}

//Dateigröße kontrollieren

if($_FILES['neuedatei']['size']>100000)

{$fehler="<br>Die Datei ist zu groß";}

//Zulässige Dateitypen kontrollieren

if(!($_FILES['neuedatei']['type'] == 'image/jpeg' OR

$_FILES['neuedatei']['type'] == 'image/gif'))

{$fehler.="<br>Der Dateityp ist nicht zulässig.<br>";}

//Datei speicher, wenn kein Fehler vorliegt

if(!$fehler)

{

//Eindeutige Namen generieren

$name=uniqid("bild");

//Dateiendung je nach Typ an den Namen anhängen

switch($_FILES['neuedatei']['type'])

{

case 'image/jpeg':

$name.=".jpg";

break;

case 'image/gif':

$name.='.gif';

break;

}

//Den Dateinamen um die Pfadangabe erweitern

$name=$uverz."/".$name;

//Die Datei in das Unterverzeichnis kopieren

copy/$_FILES['neuedatei']['tmp_name'], $name);

}

else

{echo "<h2><font color=red>".$fehler."</font></h2>;}

}

//Ausgeben der bereits vorhandenen Datein im Unterverzeichnis

echo "<h2>Bisher hochgeladene Dateien</h2>";

$i=1;

$verzeichnis=opendir($uverz);

while ($file = readdir($verzeichnis))

{

if(is_file($uverz."/".$file))

{

echo "<a href="".$uverz."/".$file."\"target=\"_blank\">Datei_".$i."</a><br>

}

}

closedir($verzeichnis);

?>

<h2>Eine Datei hochladen</h2>

<form action="<? echo $_SERVER['PHP_SELF']; method="post"

enctype="multipart/form-data">

<input type="hidden" name="send" value="1">

<input type="submit"><input type="Reset" value="Zurücksetzen">

</form>

</body></html>

xabbuh · #8 02.07.2006, 17:28:10

Zu ordentlich formatiertem Quelltext gehört nicht nur Syntaxhighlighting sondern auch ordentlich eingerückter Code: http://tut.php-q.net/einruecken.html

Andes · #9 02.07.2006, 17:45:54

Nun dürfte dir der erste Fehler schon selbst auffallen. Ohne das man sich groß dein Quelltext anschauen muss. Fällt einem schon auf, dass im letzten Drittel was nicht stimmen kann.

Andes · #10 13.07.2006, 21:11:58

Hab dir an entsprechenden Stellen mit Fehlern ein Kommentar angefügt.
Von der Syntax dürfte es soweit stimmen.

PHP-Code:


			
<html>



<head> 

<title>HabboShout BilderUpload</title> 

</head>



<body> 

<?php 

//bildupload 

$uverz="dateien"; 

if($_POST['sent']) 

{ 

    //bilder 

    if(!file_exists($uverz)) 

        {mkdir($uverz,0755);

    } 

    

    //Dateigröße kontrollieren 

    if($_FILES['neuedatei']['size']>100000) {

        $fehler="<br>Die Datei ist zu groß";

    } 

    

    //Zulässige Dateitypen kontrollieren 

    if(!($_FILES['neuedatei']['type'] == 'image/jpeg' OR $_FILES['neuedatei']['type'] == 'image/gif')) {

        $fehler.="<br>Der Dateityp ist nicht zulässig.<br>"; //Hier fehlte das abschließende Anführungszeichen

    } 



    //Datei speicher, wenn kein Fehler vorliegt 

    if(!$fehler) { 

        //Eindeutige Namen generieren 

        $name=uniqid("bild"); 

        

        //Dateiendung je nach Typ an den Namen anhängen 

        switch($_FILES['neuedatei']['type']) { 

        case 'image/jpeg': $name.=".jpg"; 

            break; 

        case 'image/gif': $name.='.gif'; 

            break; 

        } 



        //Den Dateinamen um die Pfadangabe erweitern 

        $name=$uverz."/".$name; 



        //Die Datei in das Unterverzeichnis kopieren 

        copy($_FILES['neuedatei']['tmp_name'], $name); //Hier gehört nach 'copy' ein '(' statt '/' hin

    } 

    else {

        echo "<h2><font color=red>".$fehler."</font></h2>";

    } 

} 



//Ausgeben der bereits vorhandenen Datein im Unterverzeichnis 

echo "<h2>Bisher hochgeladene Dateien</h2>"; 

$i=1; 

$verzeichnis=opendir($uverz); 

while ($file = readdir($verzeichnis)) 

{ 

    if(is_file($uverz."/".$file)) { 

        echo "<a href=\"".$uverz."/".$file."\"target=\"_blank\">Datei_".$i."</a><br>"; //Hier fehlte bei 'href=" der Slash '\'

    } 

} 

closedir($verzeichnis); 

?> 

<h2>Eine Datei hochladen</h2> 

<form action="<? echo $_SERVER['PHP_SELF']; ?>" method="post" enctype="multipart/form-data"> <!-- Hier fehlte das abschließende ?> -->

    <input type="hidden" name="send" value="1"> 

    <input type="submit"><input type="Reset" value="Zurücksetzen"> 

</form> 

</body>



</html>

:: Buchempfehlung ::

:: Anbieterverzeichnis ::

Globale Branchen

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

:: Newsletter ::

Abonnieren Sie hier den kostenlosen SELFPHP Newsletter!