Uploader problem - Seite 2

Gumbo · #11 (**permalink**) 27.11.2008, 13:04

Zitat:

Zitat von Skullsplitter

Najo finde das relativ schlecht, weil was interessiert einen User eine Fehlermeldung wie "CHMOD konnte nicht gesetzt werden".

Dass eine Funktion im Fehlfall den Grund des Fehlers zurückgibt, muss nicht heißen, dass du diesen direkt an den Benutzer zurückgibst.
Für dich als Entwickler ist es jedoch hilfreich, neben der Tatsache, dass ein Fehler auftrat, auch zu erfahren, was der mögliche Grund dafür ist.

Und noch ein Hinweis zum Programmierstil: Da bei deinem Konstrukt fast jeder „else“-Zweig eine Abbruchbedingung ist, kann deine Schreibweise schnell unübersichtlich werden (was du ja schon durch Kommentare zu verhindern suchst):

PHP-Code:


			
if (foo) {
    if (bar) {
        if (baz) {
            // alle Bedingungen sind erfüllt
        } else {
            return errorBaz;
        }
    } else {
        return errorBar;
    }
} else {
    return errorFoo;
}

Übersichtlicher wird es, wenn du die Abbruchbedingungen zu Anfang aufzählst, so bleibt die Hierarchie flach:

PHP-Code:


			
if (!foo) {
    return errorFoo;
}
if (!bar) {
    return errorBar;
}
if (!baz) {
    return errorBaz;
}
// alle Bedingungen sind erfüllt

Skullsplitter · #12 (**permalink**) 27.11.2008, 14:38

Hmh.

So habe ich das ganze noch garnicht betrachtet.

Werde mir das ganze mal ansehen und schauen, dass ich das ein bisschen besser Handeln kann mit den Errors.

Fehler so zu behandeln, wie du es beschrieben hast, ist aber nur in Funktionen möglich?
Weil sonst habe ich ja kein return und die Funktion wird ja nicht unterbrochen.

Angenommen ich prüfe, ob ein Formular abgeschickt wurde, indem ich den Button überprüfe.

PHP-Code:


			
if(isset($_POSt['button']))
{
   //speichern
}
else
{
  //Fehler
}

Würde es auch funktionieren, wenn ich das dann so mache:

PHP-Code:


			
if(!isset($_POSt['button']))
{
   //Fehler
   die();
  //oder besser exit();?
}

//speichern

So könnte man das handeln, wenn es keine Funktion ist oder?

@protonenbeschleuniger:
error_reporting(E_ALL | E_STRICT) läuft bei mir schon mit, keine angst.

Der gezeigte Code, sind jeweils nur Fragmente aus einer Datei.

protonenbeschleuniger · #13 (**permalink**) 27.11.2008, 14:54

PHP-Code:


			
$abmessungen = getimagesize($path.$name);

Sowas z.b. woher bist du so sicher dass $path.$name eine existierende Datei ist? wenn nicht erfährst du es nicht. Ist es wirklich ein Bild, d.h. ist die Funktion getimagesize() erfolgreich gewesen? Wenn nicht hantierst du später mit undefinierten Werten rum.

PHP-Code:


			
 $image_neu = imagecreate($width_neu, $height_neu);

imagecreate() gibt laut Doku false zurück, wenn kein Bild erzeugt werden kann - wird nicht abgefragt.

usw. bei den folgenden image... Funktionen

Und da es offensichtlich Probleme gibt, tritt der Fehler irgendwo in einem von diesen Aufrufen auf. Nur hat niemand die Chance diesen zu finden.

Skullsplitter · #14 (**permalink**) 27.11.2008, 14:56

Hmh gute Einwende.

Werde sie gleich mal abprüfen lassen

Gumbo · #15 (**permalink**) 27.11.2008, 15:13

Zitat:

Zitat von Skullsplitter

So könnte man das handeln, wenn es keine Funktion ist oder?

Das ist eine Möglichkeit. Weitere wären Ausnahmen oder eher unschöne Konstrukte wie etwa folgendes:

PHP-Code:


			
$error = false;
do {
    if (!foo) {
        $error = errorFoo;
        break;
    }
    // …
} while (false);
if ($error) {
    // Fehlerbehandlung
}

Skullsplitter · #16 (**permalink**) 27.11.2008, 16:11

Hier mal meine überarbeitete Version:

user.php

PHP-Code:


			
            case 'saveavatar':
                //checken ob das Formular abgeschickt wurde
                if(!isset($_POST['speichern']))
                {
                     //Aufrufen der Seite ohne Formular abzuschicken
                    $reg_tmp_file = "design/templates/error.html";
                    $reg_template = new template($reg_tmp_file);                
                    $reg_template->ersetzen('ERROR', '<b>Sie haben einen ungültigen Verweis aufgerufen!</b><br /><br />
                                                  <a href="javascript:history.back();">zurück</a>');
                    $inhalt = $reg_template->returnTemplate();    
                
                    //Script beenden
                    //die();
                }
                
                //Avatar des Users
                if(!$_FILES['Foto']['tmp_name'] == "")
                {
                    //Fehler Kein Foto ausgewählt zum Hochladen
                    $reg_tmp_file = "design/templates/error.html";
                    $reg_template = new template($reg_tmp_file);                
                    $reg_template->ersetzen('ERROR', 
                    'Sie haben kein Foto ausgewählt!<br /><br />
                                                          <a href="javascript:history.back();">zurück</a>');
                    $inhalt = $reg_template->returnTemplate();
                
                    //Script beenden
                    //die();
                }
                
                //Nur Jpg und Gif erlaubt
                if($_FILES['Foto']['type'] != "image/jpeg" OR $_FILES['Foto']['type'] != "image/jpg" OR
                   $_FILES['Foto']['type'] != "image/pjpeg" OR $_FILES['Foto']['type'] != "image/gif")
                {
                    //Fehler Falsches Format
                    $reg_tmp_file = "design/templates/error.html";
                    $reg_template = new template($reg_tmp_file);                
                    $reg_template->ersetzen('ERROR', 
                    'Das Foto <b>'.$_FILES['Foto']['name'].'</b> hat kein gültiges Format!<br /><br />
                                          <a href="javascript:history.back();">zurück</a>');
                    $inhalt = $reg_template->returnTemplate();    
                
                    //Script beenden
                    //die();
                }
                
                //wenn größer wie 2 MB fehler
                //2097152 == 2 MB
                if($_FILES['Foto']['size'] > 1024)
                {
                    //Fehler - größer als 2 MB
                    $reg_tmp_file = "design/templates/error.html";
                    $reg_template = new template($reg_tmp_file);                
                    $reg_template->ersetzen('ERROR',
                    'Das Foto <b>'.$_FILES['Foto']['name'].'</b> ist größer als 2 MB!<br /><br />
                                                      <a href="javascript:history.back();">zurück</a>');
                    $inhalt = $reg_template->returnTemplate();    
                    
                    //Script beenden
                    //die();)
                }
                
        //Upload
                //Fehlerspeicher für Upload
                $error = '';
                
                //Status | true = erfolgreich; false = fehler
                $stauts = true;
                
                //Pfad
                $path = "design/images/user_pics/";
                
                //hochladen des Fotos mit der upload_foto funktion
                $foto = upload_foto($_FILES['Foto']['tmp_name'], $_FILES['Foto']['type'], $path, $status, $error);
                
                if($status)
                {
                    //Name des Bildes
                        
                    if($_FILES['Foto']['type'] == "image/jpeg" OR $_FILES['Foto']['type'] == "image/jpg" OR
                       $_FILES['Foto']['type'] == "image/pjpeg")
                    {
                        $neuer_name = "design/images/user_pics/".$_SESSION['u_id'].".jpg";
                    }
                    else
                    {
                        $neuer_name = "design/images/user_pics/".$_SESSION['u_id'].".gif";
                    }
                            
                        
                    //ermitteln des alten Bildes
                    $altesbild_sql = mysql_query("SELECT `Foto` FROM `benutzer_daten`
                                                WHERE `users_BenutzerID` = '".$_SESSION['u_id']."'");
                            
                    //Wenn vorhanden, wird es gelöscht
                    if(mysql_num_rows($altesbild_sql) == 1)
                    {
                        $altesbild_res = mysql_fetch_array($altesbild_sql);
                            
                        //löschen des alten Bildes, wenn gefunden
                        if(file_exists($altesbild_res[0]))
                        {
                            //löschen
                            unlink($altesbild_res[0]);
                        }
                    }
                                    
                    //Updaten der Datenbank
                    $update_foto = mysql_query("UPDATE `benutzer_daten` SET `Foto` = '".$neuer_name."' 
                                                                WHERE `users_BenutzerID` = '".$_SESSION['u_id']."'");
                    //Umbennen des Fotos
                    if(!rename($foto, $neuer_name))
                    {
                        //Fehler - CHMOD konnte nicht gesetzt werden
                        $reg_tmp_file = "design/templates/error.html";
                        $reg_template = new template($reg_tmp_file);                
                        $reg_template->ersetzen('ERROR','Das neue Foto konnte nicht umbenannt werden!<br /><br />
                                                      <a href="javascript:history.back();">zurück</a>');
                        $inhalt = $reg_template->returnTemplate();
                        
                        //Script abbrechen
                        //die();
                    }
                        
                    //Laden der Templatefile - gibt die Erfolgsmeldung zurück
                    $save_tmp_file = "design/templates/user_speichern.html";
                    $save_template = new template($save_tmp_file);
                    
                    //Ausgabe der Erfolgsmeldung (TEMPLATE)
                    $inhalt = $save_template->returnTemplate();    
                }
                else
                {
                    //Fehler - CHMOD konnte nicht gesetzt werden
                    $reg_tmp_file = "design/templates/error.html";
                    $reg_template = new template($reg_tmp_file);                
                    $reg_template->ersetzen('ERROR',$error.'<br /><br />
                                                  <a href="javascript:history.back();">zurück</a>');
                    $inhalt = $reg_template->returnTemplate();    
                }
            break;

Hier mit weniger else und mehr Abfrage auf false

$error = '' -> error meldung
$status => true bei erfolg, false bei misserfolg
werden mitgeschickt

hier die funktion:

PHP-Code:


			
function upload_foto($filename, $type, $path, &$status, &$error)
{
    //$path
    //liefert das Verzeichnis - ob das Bild von einem User oder Unternehmen hochgeladen wird!
    //$status liefert true oder false bei Erfolg/Misserfolg
    //$error liefert die Fehlermeldung
    
    //eindeutiger zufallsname für das Bild!
    $zufallsname = md5(uniqid(rand(), TRUE));
    
    $endung = 'JPG';
    
    if($type == "image/jpeg" OR $type == "image/jpg"    OR $type == "image/pjpeg")
    {
        $name = $zufallsname.".jpg";
    }
    else
    {
        $name = $zufallsname.".gif";
        $endung = 'GIF';
    }
    
    //Verschieben
    if(!move_uploaded_file($filename, $path.$name))
    {
        $error = "upload";
    }
        
    //schreib- und leserechte für das bild 
    //sonst fehler auf servern mit hohen sicherheitseinstellungen
    if(!chmod($path.$name, 0644))
    {
        $error = "chmod";
    }
    
    //Falls die Datei nicht existiert
    if(!file_exists($path.$name))
    {
        $error = "datei";
    }
    
    //Bildmaße ermitteln
    $abmessungen = getimagesize($path.$name);
        
    //Die Datei
    $file = $path.$name;
    
    $width_orig = $abmessungen[0];
    $height_orig = $abmessungen[1];
    
    //Maximale Höhe des neuen Bildes!
    $width_neu = 200;
    $height_neu = 200;
    
    //name des Bildes
    $thumb_name = $path.$name;
    
    //Erzeugen eines neuen Bildes mit den Abmessungen
    $image_neu = imagecreatetruecolor($width_neu, $height_neu);
    if(!$image_neu)
    {
        $error = "true color";
    }
    
    if($endung == "JPG")
    {
        $image_alt = imagecreatefromjpeg($file);
        
        if(!$image_alt)
        {
            $error = "create";
        }
        
        if(!imagecopyresized($image_neu, $image_alt, 0, 0, 0, 0, $width_neu, $height_neu, $width_orig, $height_orig))
        {
            $error = "copy";
        }
            
        if(!imagejpeg($image_neu, $thumb_name, 100))
        {
            $error = "image";
        }
    }
    else
    {
        $image = imagecreatefromgif($file);
        
        if(!$image_alt)
        {
            return "create";
        }
        
        if(!imagecopyresized($image_neu, $image_alt, 0, 0, 0, 0, $width_neu, $height_neu, $width_orig, $height_orig))
        {
            return "copy";
        }

        if(!imagegif($image_neu, $thumb_name, 100))
        {
            return "image";
        }
    }
    
    if(!empty($error))
    {
        $status = false;
        return $status + $error;
        die();
    }
    
    $status = true;
    return $status + $thumb_name;
}

Die Fehlermeldungen sind nur zum Testen. Da käme schon was vernünftiges hin.

Habe ich soweit das Prinzip richtig verstanden?

------------

Neues Problem, was wahrscheinlich auch auf dem Webserver die Ursache war...:

jetzt geht es aufm Localhost auch nicht mehr, wenn ich die "die();" bzw "exit();" setze. ==> Leere Seite

In einer Testdatei habe ich versucht eine If-Schleife mit "die()" bzw "exit()" zu verlassen und es hat geklappt...
Dachte mir, vielleicht hat ja mein apache ne macke ^^

Bewirkt "die()" bzw. "exit()" ein Totalabbruch des kompletten Scripts?
Wenn ja, wäre das das Problem:

ich include die user.php in die index.
Dann würde ja rein theoretisch das Script an der Stelle des Includes abgebrochen werden, oder?

kampfgnom · #17 (**permalink**) 27.11.2008, 16:14

Zitat:

exit — Gibt eine Meldung aus und beendet das aktuelle Skript

Zitat:

die — Entspricht exit()

siehe Manual

Skullsplitter · #18 (**permalink**) 27.11.2008, 16:38

Hmh najo.

Wenn die die() wegfallen, was habe ich dann noch für Optionen?

Mit fällt jetzt nix mehr ein.

Gumbo · #19 (**permalink**) 27.11.2008, 18:41

Alternativen habe ich dir bereits genannt. Ich rate dir aber zu Ausnahmen.

protonenbeschleuniger · #20 (**permalink**) 28.11.2008, 09:48

ich kann, wie gesagt, kein PHP, aber gibt die() nicht eine Meldung aus?
Dann müßtest du so wenigstens was sehen

PHP-Code:


			
die $status + $error;

die() ist halt mit Vorsicht zu genießen, besser du verabschiedest die freundlich und baust einen Debug Modus ein (um später den Besucher nicht deatilierte Meldungen zu zeigen).

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Font der Überschrift ändern (Problem)	artist	CSS	2	07.12.2007 10:01
Design zu CSS/HTML verarbeiten - Problem häufen sich.	Grafikamateur	CSS	2	06.08.2007 09:57
Problem mit WordPress und dem Bilder Upload	Maxefix	Serveradministration und serverseitige Scripte	0	19.12.2006 14:58
Problem bei Div's - zwei mal das gleiche und doch nicht ...	Niriel	CSS	10	09.06.2005 17:39
Problem mit einem CSS Layout	nARC	CSS	20	21.05.2005 06:28