2 svar · 372 visningar · startad av lillebror
Hej,
Jag har ett bilduppladdningsscript som jag försökt lägga till lite felhantering i för att minimera eventuella fel i databasen.
Tidigare kunde jag ladda upp vilken fil som helst.
Detta försökte jag åtgärda med följande kod:
$img_x_y_size = getimagesize($uploaded_img_location);
if($img_x_y_size[2] <> 1 OR $img_x_y_size[2] <> 2) {
exit("<script Language='JavaScript'>location.href=('uploadscript.php?error=1');</script>");
}
I $img_x_y_size[2] får jag ut vilken filtyp det är i form av ett heltal. Detta heltal stämmer jag av mot if-satsen som kommer under. Om variabeln är skild från 1 (jpg) eller 2 (gif) så skall if-satsen exekveras. Som det är nu exekveras den trots att jag laddar upp en giltig jpg-fil.
Vad kan vara fel?
$img_x_y_size[2] <> 1 OR $img_x_y_size[2] <> 2
Om $img_x_y_size[2] = 2 så kommer den första halvan av if-satsen vara sann. Är den 1 så kommer den andra halvan att vara sann. Med andra ord. Byt ut OR mot AND!
Jag hade nog även bytt <> mot != för jag tycker det är tydligare. :)
Får det vara så enkelt!? :e Tur att det finns ett antal falkögon här på wF!