[HTML][PHP] Multiple upload, mały problem Opcje

[OneCode]

Witam,

Chciałbym poprosić Was o pomoc w rozwiązaniu jednego problemu.
Napisałem sobie kawałek kodu do uploadu kilku plików na serwer.
Generalnie wszystko działa, jeżeli zdjęcia przejdą walidację rozmiaru i formatu to się wysyłają na serwer, jeżeli nie to nie.
Ale teraz nie mogę sobie poradzić z takim problemem, jeżeli użytkownik wysyła 4 obrazki, i któryś nie przejdzie walidacji, to żaden obrazek nie powinien się wysłać na serwer.

Będę bardzo wdzięczny za pomoc.

Oto mój kod

[PHP] pobierz, plaintext 
<?php 
if(!empty ($_POST['wyslij']))
{
while(list($key,$value) = each($_FILES[images][name]))
{
	if(!empty($value))
	{   
		$filename = $value;
		$error = false;
		$plik_ext = pathinfo($filename);
		$ext = $plik_ext['extension'];
		if ($ext !="JPG" && $ext !="PJPG" && $ext !="JPEG" && $ext !="jpg" && $ext !="pjpeg" && $ext !="jpeg" && $ext !="gif" && $ext !="GIF" && $ext !="png" && $ext !="PNG")
		{
			$error = true;
			$errorMsg .= '<p class="info">Niedozwolone rozszerzenie pliku '.$filename.'<br />Dozwolone pliki to: JPG lub GIF.</p>';
			echo $errorMsg;
			$file_upload = false;
		}
		elseif ($_FILES[images][size][$key] > 102400)
		{
			$error = true;
			$errorMsg .= '<p class="info">Plik '.$filename.' jest większy niż 100KB.</p>';
			echo $errorMsg
			$file_upload = false;
		}
		elseif ($error == false)
		{					
			$useruploads = "img/$filename";
			copy($_FILES[images][tmp_name][$key], $useruploads);
			$errorMsg .= '<p class="info">Plik '.$filename.' wyslany na serwer.</p>';
		}
	}
	else
	{
		$errorMsg .= '<p class="info">Pliki sa puste.</p>';
	}
}
}
 
$form .= '<form enctype="multipart/form-data" action="up.php" method="post" id="FormAd">';
$form .= '<input type="hidden" name="MAX_FILE_SIZE" value="1048576" />';
$form .= '          <fieldset>';
$form .= '            <h2>dodaj ogłoszenie</h2>';
$form .= '            <label for="FormAdPhoto1">zdjęcie 1 (to zdjęcie będzie widoczne jako miniaturka)</label>';
$form .= '            <input type="file" name="images[]" id="FormAdPhoto1" class="iphoto" />';
$form .= '            <label for="FormAdPhoto2">zdjęcie 2</label>';
$form .= '            <input type="file" name="images[]" id="FormAdPhoto2" class="iphoto" />';
$form .= '            <label for="FormAdPhoto3">zdjęcie 3</label>';
$form .= '            <input type="file" name="images[]" id="FormAdPhoto3" class="iphoto" />';
$form .= '           <label for="FormAdPhoto4">zdjęcie 4</label>';
$form .= '            <input type="file" name="images[]" id="FormAdPhoto4" class="iphoto" />';
$form .= '            <input type="submit" value="dodaj ogłoszenie" name="wyslij" id="FormAdSubmit" />';
$form .= '          </fieldset>';
$form .= '</form>';
 
echo $form;
?>
[PHP] pobierz, plaintext 

Pozdrawiam,
Bartek.

[piotrooo89]

ja bym to robił tak:

sprawdzam czy zostało wysłane -> jak jest to sprawdzam każdy obrazek -> jeśli któryś jest błędny to unset($_FILES) -> jeśli przesło poprawnie walidacje to przekazujesz tablice $_FILES do wysłania.

[OneCode]

ja bym to robił tak:

sprawdzam czy zostało wysłane -> jak jest to sprawdzam każdy obrazek -> jeśli któryś jest błędny to unset($_FILES) -> jeśli przesło poprawnie walidacje to przekazujesz tablice $_FILES do wysłania.

Mógłbym Cię poprosić o wytłumaczenie tego na przykładzie.
Rozumiem, to co napisałeś, ale nie bardzo wiem jak to zapisać.

Z góry dziękuję,
Bartek.

[piotrooo89]

no jak to, gotowca Ci nie napisze. taki pseudo kod:

[PHP] pobierz, plaintext 
if ($_POST['wyslij'])
{
if ($ext !="JPG" && $ext !="PJPG" && $ext !="JPEG" && $ext !="jpg" && $ext !="pjpeg" && $ext !="jpeg" && $ext !="gif" && $ext !="GIF" && $ext !="png" && $ext !="PNG")
{
			$errorMsg .= '<p class="info">Niedozwolone rozszerzenie pliku '.$filename.'<br />Dozwolone pliki to: JPG lub GIF.</p>';
unset(FILES);
}
elseif ($_FILES[images][size][$key] > 102400)
{
			$errorMsg .= '<p class="info">Plik '.$filename.' jest większy niż 100KB.</p>';
unset(FILES);
}
itd...
}
[PHP] pobierz, plaintext 

po tym jak już wszystko sprawdziłeś i nie masz żadnego errora, wpisujesz:

[PHP] pobierz, plaintext 
if($FILES)
{
$useruploads = "img/$filename";
			copy($_FILES[images][tmp_name][$key], $useruploads);
			$errorMsg .= '<p class="info">Plik '.$filename.' wyslany na serwer.</p>';
}
[PHP] pobierz, plaintext 

teraz posklejaj to w logiczną całość.

[OneCode]

no jak to, gotowca Ci nie napisze. taki pseudo kod:

[PHP] pobierz, plaintext 
if ($_POST['wyslij'])
{
if ($ext !="JPG" && $ext !="PJPG" && $ext !="JPEG" && $ext !="jpg" && $ext !="pjpeg" && $ext !="jpeg" && $ext !="gif" && $ext !="GIF" && $ext !="png" && $ext !="PNG")
{
			$errorMsg .= '<p class="info">Niedozwolone rozszerzenie pliku '.$filename.'<br />Dozwolone pliki to: JPG lub GIF.</p>';
unset(FILES);
}
elseif ($_FILES[images][size][$key] > 102400)
{
			$errorMsg .= '<p class="info">Plik '.$filename.' jest większy niż 100KB.</p>';
unset(FILES);
}
itd...
}
[PHP] pobierz, plaintext 

po tym jak już wszystko sprawdziłeś i nie masz żadnego errora, wpisujesz:

[PHP] pobierz, plaintext 
if($FILES)
{
$useruploads = "img/$filename";
			copy($_FILES[images][tmp_name][$key], $useruploads);
			$errorMsg .= '<p class="info">Plik '.$filename.' wyslany na serwer.</p>';
}
[PHP] pobierz, plaintext 

teraz posklejaj to w logiczną całość.

No dobrze, ale jeżeli zrobię tak jak napisałeś, czyli unset wstawię po $errorMsg, to unset zadziała tylko dla danego (niepoprawnego) pliku, a nie usunie całej tablicy z wszystkimi plikami.
Czy się mylę?

Bartek.

[piotrooo89]

nie jak dasz cała tablice FILES to usunie Ci wszystko.