Forum PHP.pl

Witam,

Mam napisany taki skrypt jak można go uprościć by był bardziej optymalny, bezpieczny.

[PHP] pobierz, plaintext 
<?php
if (isset($_POST['dodaj'])) {
	$ilosc_foto = trim($_POST['ilosc_foto']);
	$kat		= trim($_POST['kat']);
 
print '
<p><b>Dodawanie Fotografii</b></p>
<p>';
 
$max_no_img=$ilosc_foto;  // Maximum number of images value to be set here
print "<form method=post enctype='multipart/form-data'>";
print "<table border='0' width='400' cellspacing='0' cellpadding='0' align=center>";
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> ';
 
 
}
elseif(isset($_POST['addimg'])) {
 
	$kat = trim($_POST['kat']);
 
while(list($key,$value) = each($_FILES['images']['name']))
		{
 
			if(!empty($value))
			{
				 $filename = $value;
					$filename=str_replace(" ","_",$filename);// Add _ inplace of blank space in file name, you can remove this line
                    $ext = pathinfo($filename, PATHINFO_EXTENSION);
 
	$zapytanie = "INSERT foto VALUES ('', '$ext', '$kat')";
	$wstaw = mysql_query($zapytanie);
	$id = mysql_insert_id();
 
	                $kodowanie = $id;
                    $lacz = $kodowanie.".".$ext;
					$add = "../uploads/foto/$lacz";
	//echo $_FILES['images']['type'][$key];
			     // echo "<br>";
						copy($_FILES['images']['tmp_name'][$key], $add);
 
 
$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);}
 
					unlink("../uploads/foto/".$lacz."");
 
			}
			else print'Dodawanie nieudane.';
		}
	print'<div id="dymkaP">Brawo udało Ci się dodać fotografie.</div>';	
 
	}
[PHP] pobierz, plaintext 

Proszę o pomoc, chodzi o uproszczenie by skrypt był mniejszy oraz bezpieczniejszy.