Mam napisany taki skrypt jak można go uprościć by był bardziej optymalny, bezpieczny.
<?php print ' <p><b>Dodawanie Fotografii</b></p> <p>'; $max_no_img=$ilosc_foto; // Maximum number of images value to be set here for($i=1; $i<=$max_no_img; $i++){ print "<tr><td>Zdjecie $i</td><td> <input type='file' name='images[]' class='bginput'></td></tr>"; } print '</table></p> <div class="pagesPrzyciski" style="margin-left:120px; margin-top:10px;"> <input type="hidden" name="kat" value="'.$kat.'" /> <input type="submit" name="addimg" class="pages" value="Wyślij" /> <input type="reset" class="pages" value="Resetuj" /> <input type="button" class="pages" onclick="java script:history.back();" name="wstecz" value="Wstecz" /> </div> </form> '; } { { $filename = $value; $filename=str_replace(" ","_",$filename);// Add _ inplace of blank space in file name, you can remove this line $zapytanie = "INSERT foto VALUES ('', '$ext', '$kat')"; $kodowanie = $id; $lacz = $kodowanie.".".$ext; $add = "../uploads/foto/$lacz"; //echo $_FILES['images']['type'][$key]; // echo "<br>"; $max_width = 800; $max_height = 600; $proporcja = $max_width / $max_height; $format = $ext; if($format== "png"){$zdjecie = imagecreatefrompng("../uploads/foto/".$lacz);} if($format== "jpg"){$zdjecie = imagecreatefromjpeg("../uploads/foto/".$lacz);} if($format== "gif"){$zdjecie = imagecreatefromgif("../uploads/foto/".$lacz);} $proporcja_zdjecia = imagesx($zdjecie) / imagesy($zdjecie); $nowe_zdjecie = imagecreatetruecolor($max_width, $max_height); if($proporcja_zdjecia > $proporcja) { //$srodek = $proporcja/imagesy($zdjecie); $srodek = (imagesx($zdjecie) - imagesx($zdjecie)/$proporcja ) / 4; imagecopyresampled($nowe_zdjecie, $zdjecie, 0, 0, $srodek, 0, $max_width, $max_height, imagesy($zdjecie)*$proporcja, imagesy($zdjecie) ); } else { $srodek = (imagesy($zdjecie) - imagesy($zdjecie)/$proporcja ) / 4; //$srodek = imagesx($zdjecie)*$proporcja; imagecopyresampled($nowe_zdjecie, $zdjecie, 0, 0, 0, $srodek, $max_width, $max_height, imagesx($zdjecie), imagesx($zdjecie)/$proporcja ); } if($format== "png"){imagepng($nowe_zdjecie, "../uploads/foto/photo/".$lacz);} if($format== "jpg"){imagejpeg($nowe_zdjecie, "../uploads/foto/photo/".$lacz);} if($format== "gif"){imagegif($nowe_zdjecie, "../uploads/foto/photo/".$lacz);} $max_width = 145; $max_height = 120; $proporcja = $max_width / $max_height; $format = $ext; if($format== "png"){$zdjecie = imagecreatefrompng("../uploads/foto/".$lacz);} if($format== "jpg"){$zdjecie = imagecreatefromjpeg("../uploads/foto/".$lacz);} if($format== "gif"){$zdjecie = imagecreatefromgif("../uploads/foto/".$lacz);} $proporcja_zdjecia = imagesx($zdjecie) / imagesy($zdjecie); $nowe_zdjecie = imagecreatetruecolor($max_width, $max_height); if($proporcja_zdjecia > $proporcja) { //$srodek = $proporcja/imagesy($zdjecie); $srodek = (imagesx($zdjecie) - imagesx($zdjecie)/$proporcja ) / 4; imagecopyresampled($nowe_zdjecie, $zdjecie, 0, 0, $srodek, 0, $max_width, $max_height, imagesy($zdjecie)*$proporcja, imagesy($zdjecie) ); } else { $srodek = (imagesy($zdjecie) - imagesy($zdjecie)/$proporcja ) / 4; //$srodek = imagesx($zdjecie)*$proporcja; imagecopyresampled($nowe_zdjecie, $zdjecie, 0, 0, 0, $srodek, $max_width, $max_height, imagesx($zdjecie), imagesx($zdjecie)/$proporcja ); } if($format== "png"){imagepng($nowe_zdjecie, "../uploads/foto/mini/".$lacz);} if($format== "jpg"){imagejpeg($nowe_zdjecie, "../uploads/foto/mini/".$lacz);} if($format== "gif"){imagegif($nowe_zdjecie, "../uploads/foto/mini/".$lacz);} } } }
Proszę o pomoc, chodzi o uproszczenie by skrypt był mniejszy oraz bezpieczniejszy.