edycja + zmiana plików na dysku, co tu zmienić Opcje

[krzyszbi]

witam
mam taki problem z którymmęcze się już od rana
problem to edycja danych z jednoczesną zanianą lub usunięciem zdjęcia jeżeli są ze soba powiązane w bazie Mysql

a oto kod mojego skryptu za pomoca którego staram się tego dokonać
jeżeli w formulażu się wypełni wszystkie pola (dotyczy tylko zmian ze zdjęciami, edycja wpisów w bazie mi działa) to wszystko gra ale jak jakiegoą pola nie uzupełnie to się sypie

[PHP] pobierz, plaintext 
<?php
include("../files/konfig.inc");
 
					  $zamiana = $_POST['zamiana'];
 
 
 
for ($i = 0; $i < 5; $i++ )
		   {
			if ($zamiana[($i)]==1)
			   {
				 if (file_exists('../picture_library/'.$id.'traktor'.($i+1).'.jpg'))
					{
						unlink('../picture_library/'.$id.'xxx'.($i+1).'.jpg');
					}
				 if( is_uploaded_file( $_FILES['userfile']['tmp_name'][$i] ) )
					 {
					  $strUploadDir ='../picture_library/'.$_FILES['userfile']['name'][$i];
					   if( move_uploaded_file( $_FILES['userfile']['tmp_name'][$i], $strUploadDir ))
						   {
							   $nowa_nazwa = $id.'xxx'.($i+1).'.jpg';
								 @rename('../picture_library/'.$_FILES['userfile']['name'][$i],'../picture_library/'.$nowa_nazwa);
							   $query1 = "INSERT INTO zdjecia SET id='$id',zdjecie='$nowa_nazwa'";
										  mysql_query ($query1);
						   }
						else
							{
							   echo 'Upload plików nie powiódł się!';
							}
				 }
 
			   }
			   if ($zamiana[($i)]==2)
			   {
				 if (file_exists('../picture_library/'.$id.'xxx'.($i+1).'.jpg'))
					{
						unlink('../picture_library/'.$id.'xxx'.($i+1).'.jpg');
					}
 
 
			   }
			}
echp'zamieniono';
?>
[PHP] pobierz, plaintext 

może ktoś zna jakiś lepszy/skuteczniejszy sposób na zanianę plików na dysku poprzez formularz zpoziomu http
z góry dzięki za podpowiedzi

[nospor]

Twój podstawowy i największy błąd to:
nie sprawdzasz, czy wszystkie pola są wypełnione. W przypadku, gdy pola wymagane nie są wypełnione powienieś wrócić do formularza, a nie wykonywać akcję dla pół wypełnionych.

[krzyszbi]

ale niekonieczne jest wypełnianie wszystkich pól
ponieważ moim zdaniem chyba bezsensowne jest abym musiał do formularza
edycji wyisywaś co mam zrobić tzn
zamienić plik
skasować plik
pozostawić bez zmian

czyba że do każdego pola w formularzu zrobić po trzy przyciski typu
radio o w/w nazwach i wtedy polecieć pętlą z akcję odnośnie każdej możliwości

tylko że juz tak próbowałem i mi nie wyszło ponieważ pole do zmiany/skasowanie/pozostawienia bez zmian są tworzone za pomoca następujacej pętli

[PHP] pobierz, plaintext 
<?php
 
for ($i=0; $i<5; $i++)
        {
          if (file_exists('../picture_library/'.$id.'xxx'.($i+1).'.jpg'))
              echo'<tr><td class="opis_s_bg">Zdjęcie nr. '.($i+1).'</td><td align="center"><img src="../picture_library/'.$id.'xxx'.($i+1).'.jpg" width="200" height="175" border="0" /><br />
               <input type="checkbox" name="zamiana[]" value="1" />Zamiana zdjęcia??<br />
               <input type="checkbox" name="zamiana[]" value="2" />Skasowanie zdjęcia??<br />
               
 
 
              <input class="form_i_admin_dod" name="userfile[]" type="file"></td></tr>';
          else
              echo'<tr><td class="opis_s_bg">Zdjęcie nr. '.($i+1).'</td><td align="center">Brak zdjęcia<br /><input type="checkbox" name="zamiana[]" value="1" />Dodanie zdjęcia??<br><input class="form_i_admin_dod" name="userfile[]" type="file"></td></tr>';
        }
 
?>
[PHP] pobierz, plaintext 

a może zmienić coś tutaj??

[nospor]

I tu się kłania znajomośc tego co sie wysyła a co nie. pola typu Checkbox wysyłają sie tylko wtedy, gdy są zaznaczone. Tak więc jesli robisz dla nazwy checkboxa tablicę, to jesli zaznaczysz drugiego a pierwszego nie, to drugi wskoczy ci jako pierwszy z indexem. Musisz jakos jednoznacznie je nazywać. Poazatym coś może byc skasowane albo zmienione. Twoje chcekboxy tego nie wykluczają , powinno byc raczej radio.

ps: formularz piszę sie przez rz a nie ż

[krzyszbi]

próbowałem właśnie z przyciskami typu radio ale coś mi nie wychodzi

jak za pomoca pętli utwożyć pola w formularzu z przyciskami radio

poniższym kodem mi nie działa

[PHP] pobierz, plaintext 
<?php
 
for ($i=0; $i<5; $i++)
		{
		  if (file_exists('../picture_library/'.$id.'traktor'.($i+1).'.jpg'))
			  echo'<tr><td class="opis_s_bg">Zdjęcie nr. '.($i+1).'</td><td align="center"><img src="../picture_library/'.$id.'traktor'.($i+1).'.jpg" width="200" height="175" border="0" /><br />
			   <input type="radio" name="zamiana'.$i.'" value="1" />Zamiana zdjęcia??<br />
			   <input type="radio" name="zamiana'.$i.'" value="2" />Skasowanie zdjęcia??<br />
			   <input type="radio" name="zamiana'.$i.'" value="3" CHECKED />Bez zmin..<br />
  <input class="form_i_admin_dod" name="userfile[]" type="file"></td></tr>';
		  else
			  echo'<tr><td class="opis_s_bg">Zdjęcie nr. '.($i+1).'</td><td align="center">Brak zdjęcia<br />
			  <input type="radio" name="zamiana'.$i.'" value="4" />Dodanie zdjęcia??<br />
			  <br><input class="form_i_admin_dod" name="userfile[]" type="file"></td></tr>';
		}
 
?>
[PHP] pobierz, plaintext 

a jeśli bedzie w następujacu sposób to wszystkie pola radio mają tę samą nazwe nie da się zaznaczyć tylko jedno z nich

[PHP] pobierz, plaintext 
<?php
 
for ($i=0; $i<5; $i++)
		{
		  if (file_exists('../picture_library/'.$id.'traktor'.($i+1).'.jpg'))
			  echo'<tr><td class="opis_s_bg">Zdjęcie nr. '.($i+1).'</td><td align="center"><img src="../picture_library/'.$id.'traktor'.($i+1).'.jpg" width="200" height="175" border="0" /><br />
			   <input type="radio" name="zamiana[]" value="1" />Zamiana zdjęcia??<br />
			   <input type="radio" name="zamiana[]" value="2" />Skasowanie zdjęcia??<br />
			   <input type="radio" name="zamiana[]" value="3" CHECKED />Bez zmin..<br />
<input class="form_i_admin_dod" name="userfile[]" type="file"></td></tr>';
		  else
			  echo'<tr><td class="opis_s_bg">Zdjęcie nr. '.($i+1).'</td><td align="center">Brak zdjęcia<br />
			  <input type="radio" name="zamiana[]" value="4" />Dodanie zdjęcia??<br />
			  <br><input class="form_i_admin_dod" name="userfile[]" type="file"></td></tr>';
		}
 
?>
[PHP] pobierz, plaintext 

co robie źle
możecie mi powiedzieć

[nospor]

jak na moj gust to radio tworzone tak: zamiana'.$i jest wlasnie ok. Co ci nie działa? Może źle to odbierasz?

[krzyszbi]

męcze się z tym i męcze ale jakos mi nie wychodzi

ja to próbóje odbierać tak ale po moich próbach już wiem że to zapewne jest kiepski sposób
(to tylko aby sprawdzić jak zostanie wykonana akcja dana akcja mi działa- z tym nie mam problemu)

[PHP] pobierz, plaintext 
<?php
 
 echo'<table>';
 for ($i = 0; $i < 5; $i++ )
		   {
			echo'<tr>';
			if ($zamiana.$i==1)
			   {
				 echo'<td>zaniana<br /></td><td>'.($i+1).'</td><td>';
 
			   }
			   elseif ($zamiana.$i==2)
			   {
 
					echo'<td>kasacja<br /></td><td>'.($i+1).'</td>';
 
 
			   }
			   elseif ($zamiana.$i==3)
					  {
					   echo'<td>dodanie<br /></td><td>'.($i+1).'</td>';
					   }
			   elseif($zamiana.$i==3)
					  {
						  echo'<td>bez zmian<br /></td><td>'.($i+1).'</td>';
					  }
					  else echo'<td>koniec<br /></td><td>'.($i+1).'</td>';
				  echo'</tr>';
			   }
			   echo'</table>';
 
?>
[PHP] pobierz, plaintext 

jak to odebrać
pola typu radio i owszem twożą się z innymi indexami i nie zaznacza sie tylko jedno
mała pomoc by nie zaszkodziła
tylko co ty mi nie gra??

[nospor]

źle odbierasz. zamiast $zamiana.$i daj $_POST['zamiana'.$i]
zakladam ze formularz wysylasz postem

[krzyszbi]

ok
dzięki
w końcu zaczeło działać tak jak ja chcę

dzięki za podpowiedzi