uproszczenie skryptu edycji bloków Opcje

[colinQ]

[PHP] pobierz, plaintext 
<table cellspacing=0 align=center class=bg_color width=250>
  <tr><td><center><font color="white" face="arial" style=font-size:12px>Zmiana Bloków</td></tr>
  <tr><td><center><font color="white" face="arial" style=font-size:11px>
<?
$sql = mysql_num_rows( $sql2 = mysql_query("select * from bloki") );
 
If(isset($_POST['potwierdzenie'])) {
  $a=1;
  while($a <= $sql) {
	$fetch2 = mysql_fetch_assoc($sql2);
	$f = $fetch2['nazwa'];
	$p = $_POST[$f];
	if( isset($p) ) {
	  if($p !== $fetch2['opcja'])
		mysql_query("update bloki set opcja='$p' where nazwa='$f'");
	echo $f;
	}
  }
 
} else {
 echo '<form method=post><center>';
  for($i=1;$i<=$sql;$i++) {
   $fetch = mysql_fetch_assoc($sql2);
   echo '<input type=checkbox '; if($fetch['opcja'] == 1) echo 'checked=1'; echo ' name='.$fetch['nazwa'].' value=1 />'.$fetch['nazwa'].'<br>';
  }
  echo '
	<input type="hidden" value="wyslano" name="potwierdzenie">
	<input type=submit>
	</form>
  ';
}
?>
 
</td></tr></table>
[PHP] pobierz, plaintext 

Pomóżcie mi jakoś uprościć ten skrypt.

Ma on wyświetlać nazwy bloków pobranych z bazy, gdy pole [opcja] == 1 to ma zaznaczyć w checkboxie a jeśli == 0 to ni zaznaczać.
Następnie ma wysyłać do bazy wartość tego checkboxa czyli:
gdy wybrany - 1
gdy nie wybrany - 0


Problem w tym że napisałem w jakiś nie logiczny sposób :/

a przy okazji wywala mi:

Fatal error: Maximum execution time of 30 seconds exceeded in [tutaj adresik] on line 11

Ten post edytował colinQ 24.10.2007, 18:58:46

[phpion]

Wywala Ci fatal error bo nigdzie nie dajesz $a++ więc warunek w while'u jest zawsze spełniony. Rób wcięcia bo taki kod jak Twój jest bardzo nieczytelny :/

[colinQ]

A no tak zapomniałem, a kod ogólnie może być??

Czy jakoś go uprościć??

[domis86]

wcięcia, wcięcia i po trzykroć wcięcia

przepisz ten kod z wcienciami i pokaz go tu