fehler in einer funktion

imported_master · 07.04.2006, 22:07

Hab ein fehler in dieser funktion, weiß aber nicht wo der ist.
kann mir vllt einer helfen?

ps.: alle bilder die ich hochlade werden schwarz

PHP-Code:

  <?php

function resize($rfile, $rext, $rheight, $rwidth) {

 
// Nur ändern wenn die automatische Erkennung fehlschlägt

$tmppath = getcwd() . "/tmp";

 
$image_path = $rfile;

$img = null;

if ($rext == 'jpg' || $rext == 'jpeg') {

    $img = @imagecreatefromjpeg($image_path);

} else if ($rext == 'png') {

    $img = @imagecreatefrompng($image_path);

} else if ($rext == 'gif') {

    $img = @imagecreatefrompng($image_path);

}

 
if ($img) {

 
    $width = imagesx($img);

    $height = imagesy($img);

    $scale = min($rwidth/$width, $rheight/$height);

 
    if ($scale < 1) {

        $new_width = floor($scale*$width);

        $new_height = floor($scale*$height);

 
        $tmp_img = imagecreatetruecolor($new_width, $new_height);

 
        imagecopyresized($tmp_img, $img, 0, 0, 0, 0,

                         $new_width, $new_height, $width, $height);

        imagedestroy($img);

        $img = $tmp_img;

    }

}

if (!$img) {

    $img = imagecreate($rwidth, $rheight);

    imagecolorallocate($img,0,0,0);

    $c = imagecolorallocate($img,70,70,70);

    imageline($img,0,0,$rwidth,$rheight,$c2);

    imageline($img,$rwidth,0,0,$rheight,$c2);

}

$tempfile = tempnam ($tmppath, "resize");

imagejpeg($img, $tempfile);

if(copy($tempfile, $rfile)) {return 1;} else {return 0;}

}

?>

Ryson · 07.04.2006, 23:12

Statt if()...if(!) könntest Du if()..else() schreiben.
Außerdem handelst Du nicht alle Fälle ab.

Das Gelingen von imagecreatefromjpeg() & Co. prüfst Du noch,
imagecreate() aber nicht mehr.
Wenn letzteres fehlschlägt, was macht dann imagejpeg()?

imagecolorallocate() läuft einmal ins Leere,
einmal auf eine Variable die nicht verwendet wird: $c.
Dafür übergibst Du $c2, die sicher keinen gültigen Wert hat.

Ein paar Ausgaben an den richtigen Stellen könnten die Fehlersuche erleichtern.
Wenn Du error_reporting(E_ALL); einstellst,
siehst Du außerdem alle Meldungen, sofern sie nicht mit @ unterdrückt werden.

Der_Gerhard · 08.04.2006, 20:11

Zitat:

Zitat von Ryson

Statt if()...if(!) könntest Du if()..else() schreiben.
Außerdem handelst Du nicht alle Fälle ab.

Nein, geht nicht.

Wenn Du mal in den if-Zweig guckst, wirst Du bemerken, dass $img dort verändert wird.

Die zweite Abfrage wertet somit den Wert erneut aus. Ein else würde das nicht tun.

Damit könnte man nie den Code in beiden Abfragen ausführen lassen.

Basti · 08.04.2006, 21:10

Zitat:

Zitat von master

PHP-Code:

  <?php

} else if ($rext == '[b]gif[/b]') {

    $img = @imagecreatefrompng($image_path);

} //                                     ^^

?>

Basti

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
JS: Einführung - Javascript im Schichtenmodell	nikosch	Tutorials	4	11.04.2009 17:06
Rekursive Funktion bricht ab...	duras666	PHP Tipps 2008	9	28.04.2008 11:36
Name der aufrufenden Funktion	mAy^daY	PHP Tipps 2007	5	20.01.2007 22:55
Fehler in rekursiver Funktion	tinchen	PHP Tipps 2006	1	27.08.2006 20:24
Komplexe Funktion: +Übersichtlichkeit, -Performance	Jacks Rache	PHP Tipps 2006	3	07.06.2006 14:22
PHP-GTK Tutorial		Beitragsarchiv	9	02.11.2005 21:07
Rückgabewert einer rekrusiven Funktion		PHP-Fortgeschrittene	7	06.10.2005 18:44
(schnellere) Funktion zum Zusammenfassen von CSS		PHP-Fortgeschrittene	21	08.08.2005 16:47
In einer Funktion auf eine Funktion der Klasse zugreifen	phpbeginner	PHP Tipps 2005-2	2	28.07.2005 00:30
Funktion in einer Funktion aufrufen?		PHP Tipps 2005-2	11	14.06.2005 15:14
Problem mit einer rekursiven Funktion in einer Klasse	ajo_silent	PHP-Fortgeschrittene	6	13.06.2005 16:40
[Erledigt] Wie kann ich beliebig viele Werte an eine Funktion übergeben		PHP Tipps 2005	11	25.01.2005 10:44
Rekursive Funktion ... finde fehler nicht ...		PHP-Fortgeschrittene	1	27.09.2004 03:44
mail funktion funzt net es kommt ein fehler	phpfortgeschrittener	PHP Tipps 2004	4	29.08.2004 01:07
LIMIT(?) Fehler	Schaelle	Datenbanken	0	28.08.2004 14:11

07.04.2006, 22:07
imported_master Neuer Benutzer Registriert seit: 17.07.2005 Beiträge: 7	fehler in einer funktion Hab ein fehler in dieser funktion, weiß aber nicht wo der ist. kann mir vllt einer helfen? ps.: alle bilder die ich hochlade werden schwarz PHP-Code: <?php function resize($rfile, $rext, $rheight, $rwidth) { // Nur ändern wenn die automatische Erkennung fehlschlägt $tmppath = getcwd() . "/tmp"; $image_path = $rfile; $img = null; if ($rext == 'jpg' \|\| $rext == 'jpeg') { $img = @imagecreatefromjpeg($image_path); } else if ($rext == 'png') { $img = @imagecreatefrompng($image_path); } else if ($rext == 'gif') { $img = @imagecreatefrompng($image_path); } if ($img) { $width = imagesx($img); $height = imagesy($img); $scale = min($rwidth/$width, $rheight/$height); if ($scale < 1) { $new_width = floor($scale$width); $new_height = floor($scale$height); $tmp_img = imagecreatetruecolor($new_width, $new_height); imagecopyresized($tmp_img, $img, 0, 0, 0, 0, $new_width, $new_height, $width, $height); imagedestroy($img); $img = $tmp_img; } } if (!$img) { $img = imagecreate($rwidth, $rheight); imagecolorallocate($img,0,0,0); $c = imagecolorallocate($img,70,70,70); imageline($img,0,0,$rwidth,$rheight,$c2); imageline($img,$rwidth,0,0,$rheight,$c2); } $tempfile = tempnam ($tmppath, "resize"); imagejpeg($img, $tempfile); if(copy($tempfile, $rfile)) {return 1;} else {return 0;} } ?>


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

07.04.2006, 23:12
Ryson Benutzer Registriert seit: 28.03.2005 Beiträge: 46	Statt if()...if(!) könntest Du if()..else() schreiben. Außerdem handelst Du nicht alle Fälle ab. Das Gelingen von imagecreatefromjpeg() & Co. prüfst Du noch, imagecreate() aber nicht mehr. Wenn letzteres fehlschlägt, was macht dann imagejpeg()? imagecolorallocate() läuft einmal ins Leere, einmal auf eine Variable die nicht verwendet wird: $c. Dafür übergibst Du $c2, die sicher keinen gültigen Wert hat. Ein paar Ausgaben an den richtigen Stellen könnten die Fehlersuche erleichtern. Wenn Du error_reporting(E_ALL); einstellst, siehst Du außerdem alle Meldungen, sofern sie nicht mit @ unterdrückt werden.