Optymalizacja kodu do zapisywania zdjęć

Question

Bardzo proszę o pomoc w optymalizacji tego kodu. Z góry dziękuję.

<?php

$folder="img/ImgGaleria";
$znaks="5";
$chm = 0644;


	if (isset($_FILES['plik1']))
{

	$plik_tmp = $_FILES['plik1']['tmp_name'];
	$f = $_FILES['plik1'];
	IF($f['type'] == 'image/png' or $f['type'] == 'image/bmp' or $f['type'] == 'image/jpeg' or $f['type'] == 'image/gif')
	{

		$length="$znaks";
		$list=array_merge(range(0,9));
		shuffle($list);
		$string=substr(join($list),0,$length);
		
		$plik_nazwa="1.png";
		
		if(is_uploaded_file($plik_tmp)) 
		{

			 move_uploaded_file($plik_tmp, "./$folder/$plik_nazwa");
			 chmod("./$folder/$plik_nazwa",$chm);
		}

	}
else
{
	echo 'Dozwolone formaty: <b>.jpg .jpeg .png .gif .bmp</b>';
}
}


	if (isset($_FILES['plik2']))
{
	$plik_tmp = $_FILES['plik2']['tmp_name'];
	$f = $_FILES['plik2'];
	IF($f['type'] == 'image/png' or $f['type'] == 'image/bmp' or $f['type'] == 'image/jpeg' or $f['type'] == 'image/gif')
	{

		$length="$znaks";
		$list=array_merge(range(0,9));
		shuffle($list);
		$string=substr(join($list),0,$length);

		$plik_nazwa="2.png";

		if(is_uploaded_file($plik_tmp)) {

			 move_uploaded_file($plik_tmp, "./$folder/$plik_nazwa");
			 chmod("./$folder/$plik_nazwa",$chm);
	}

	}
	else
{
	echo 'Dozwolone formaty: <b>.jpg .jpeg .png .gif .bmp</b>';
}
}

	if (isset($_FILES['plik3']))
{
	$plik_tmp = $_FILES['plik3']['tmp_name'];
	$f = $_FILES['plik3'];
	IF($f['type'] == 'image/png' or $f['type'] == 'image/bmp' or $f['type'] == 'image/jpeg' or $f['type'] == 'image/gif')
	{

		$length="$znaks";
		$list=array_merge(range(0,9));
		shuffle($list);
		$string=substr(join($list),0,$length);

		$plik_nazwa="3.png";

		if(is_uploaded_file($plik_tmp)) {

			 move_uploaded_file($plik_tmp, "./$folder/$plik_nazwa");
			 chmod("./$folder/$plik_nazwa",$chm);
	}

	}
	else
{
	echo 'Dozwolone formaty: <b>.jpg .jpeg .png .gif .bmp</b>';
}
}

	if (isset($_FILES['plik4']))
{
	$plik_tmp = $_FILES['plik4']['tmp_name'];
	$f = $_FILES['plik4'];
	IF($f['type'] == 'image/png' or $f['type'] == 'image/bmp' or $f['type'] == 'image/jpeg' or $f['type'] == 'image/gif')
	{

		$length="$znaks";
		$list=array_merge(range(0,9));
		shuffle($list);
		$string=substr(join($list),0,$length);

		$plik_nazwa="4.png";

		if(is_uploaded_file($plik_tmp)) {

			 move_uploaded_file($plik_tmp, "./$folder/$plik_nazwa");
			 chmod("./$folder/$plik_nazwa",$chm);
	}

	}
	else
{
	echo 'Dozwolone formaty: <b>.jpg .jpeg .png .gif .bmp</b>';
}
}

	if (isset($_FILES['plik5']))
{
	$plik_tmp = $_FILES['plik5']['tmp_name'];
	$f = $_FILES['plik5'];
	IF($f['type'] == 'image/png' or $f['type'] == 'image/bmp' or $f['type'] == 'image/jpeg' or $f['type'] == 'image/gif')
	{

		$length="$znaks";
		$list=array_merge(range(0,9));
		shuffle($list);
		$string=substr(join($list),0,$length);

		$plik_nazwa="5.png";

		if(is_uploaded_file($plik_tmp)) {

			 move_uploaded_file($plik_tmp, "./$folder/$plik_nazwa");
			 chmod("./$folder/$plik_nazwa",$chm);
	}

	}
	else
{
	echo 'Dozwolone formaty: <b>.jpg .jpeg .png .gif .bmp</b>';
}
}

	if (isset($_FILES['plik6']))
{
	$plik_tmp = $_FILES['plik6']['tmp_name'];
	$f = $_FILES['plik6'];
	IF($f['type'] == 'image/png' or $f['type'] == 'image/bmp' or $f['type'] == 'image/jpeg' or $f['type'] == 'image/gif')
	{

		$length="$znaks";
		$list=array_merge(range(0,9));
		shuffle($list);
		$string=substr(join($list),0,$length);

		$plik_nazwa="6.png";

		if(is_uploaded_file($plik_tmp)) {

			 move_uploaded_file($plik_tmp, "./$folder/$plik_nazwa");
			 chmod("./$folder/$plik_nazwa",$chm);
	}

	}
	else
{
	echo 'Dozwolone formaty: <b>.jpg .jpeg .png .gif .bmp</b>';
}
}

header('Location: '.$_SERVER['HTTP_REFERER']);

exit();

?>

 

Answer 1

Przesyłaj pliki w tablicy (<input type="file" name="file[]" ...>) i wszystkie operacje na nich wykonuj w jednej pętli zamiast powielać kod.

Comments

Zrobiłem coś takiego i nie działa (z tablicą też nie działało)

<?php

$folder="img/ImgGaleria";
$znaks="5";
$odst="_";
$chm = 0644;

for($i = 0; $i <= 10; $i++)
{

$nazwa = '"plik'.$i.'"';

	if (isset($_FILES[$nazwa]))
{
	echo $nazwa;

	$plik_tmp = $_FILES[$nazwa]['tmp_name'];
	$f = $_FILES[$nazwa];
	IF($f['type'] == 'image/png' or $f['type'] == 'image/bmp' or $f['type'] == 'image/jpeg' or $f['type'] == 'image/gif')
	{

		$length="$znaks";
		$list=array_merge(range(0,9));
		shuffle($list);
		$string=substr(join($list),0,$length);
		
		$plik_nazwa=$nazwa.".png";
		
		if(is_uploaded_file($plik_tmp)) 
		{

			 move_uploaded_file($plik_tmp, "./$folder/$plik_nazwa");
			 chmod("./$folder/$plik_nazwa",$chm);
		}

	}
else
{
	echo 'Dozwolone formaty: <b>.jpg .jpeg .png .gif .bmp</b>';
}
}
}

header('Location: '.$_SERVER['HTTP_REFERER']);

exit();

?>

 

---

Co to znaczy "nie działa"? Jakiś błąd? Co się wykonuje a co w ogóle nie?

Z tablicą na pewno działa bo robiłem tak nie raz, musiałeś zrobić coś źle.

---

Już wszystko działa,  wystarczyło zrobić

tak: $nazwa = 'plik'.$i;

A nie tak $nazwa = '"plik'.$i.'"';

Dziękuję za pomoc, pozdrawiam.