Miałem pewien kod:

[PHP] pobierz, plaintext 
  1. <?php
  2. $title='Sale laboratoryjne';
  3. include ('dodatki/bonus.php');
  4. include ('meta.php');
  5. include ('gora_logo.php');
  6. lewe_menu ();
  7. open_title();
  8. polacz_baza();
  9.  
  10. $i = 0;
  11. $ile = 4;
  12.  
  13. $sale = mysql_query("SELECT * FROM sale ORDER BY sala desc;") or die(mysql_error());
  14. //zmienna $rekord jest przypisana wierszowi (rekordowi) zmiennej $news a następnie pobieranie danych z konkretnych kolumn (komórka po komórce)
  15. WHILE ($rekord = mysql_fetch_assoc ($sale)) 
  16. 	{
  17. 		$id_sali = $rekord['id_sali'];
  18. 		$sala = $rekord['sala'];
  19. 		$opiekun = $rekord['opiekun'];
  20. 		$info = $rekord['info'];
  21. 		//funkcja nl2br odpowiada za odpowiednie wyświetlanie po przez użycie np. znaków następnej linii
  22. 		if (!isset($_GET['sala']) || $_GET['sala'] == '') 
  23. 			{
  24. 				echo '<font color="#FF000"-><h3><a href="sale.php?sala='.$id_sali.'">'.stripslashes($sala).'</a></h3></font><br />';
  25. 			}
  26.  
  27. 		if (isset($_GET['sala']) && $_GET['sala'] == $id_sali)
  28. 			{
  29.  
  30. 						echo '<font color="#FF0000"-><h3>'.stripslashes($sala).'</h3></font><br />
  31. 						'.stripslashes($opiekun).'<br />
  32. 						'.stripslashes($info).'<br />';
  33.  
  34. 						$zdjecia = mysql_query("SELECT * FROM zdjecia WHERE id_kat = '".$sala."' ORDER by id_zdjecia DESC LIMIT 4;");
  35. 						$licz_zdjecia = mysql_num_rows($zdjecia);
  36. 						if ($licz_zdjecia!=0)
  37. 							{
  38. 								echo '<u>Zdjęcia z galerii dla sali: '.$sala.'</u><br /><p align="center">';
  39. 							}
  40. 						else
  41. 							{
  42. 								echo '<u>Nie ma zdjęć dla danej galerii</u><br /><p align="center">';
  43. 							}
  44.  
  45.  
  46.  
  47. 						WHILE ($rekord = mysql_fetch_assoc ($zdjecia)) 
  48. 							{
  49. 								$zdjecie = $rekord['zdjecie'];
  50. 								$opis = $rekord['opis'];
  51. 								echo '<a href="./upload/galeria/'.$zdjecie.'" border="0" alt="'.$opis.'" title="'.$opis.'">
  52. 								<img src="./upload/galeria/mini/'.$zdjecie.'" border="0"></a>';
  53. 								$i++;
  54. 								if($i==$ile) 
  55. 									{
  56. 										$i = 0;
  57. 										echo '<br />';
  58. 									}
  59. 							}
  60. 						echo '<a href="./sale.php">Powróć do SAL</a>';	
  61.  
  62. 				echo '</p>';
  63. 			}
  64.  
  65. 	}
  66.  
  67.  
  68.  
  69. echo '<br /></p>';
  70. close_title();
  71. include ('./stopka.php');
  72. ?>
[PHP] pobierz, plaintext

Aby zaoszczędzić czasu na zapytanie i poprawić kod napisałem:

[PHP] pobierz, plaintext 
  1. <?php
  2. $title='Sale laboratoryjne';
  3. include ('dodatki/bonus.php');
  4. include ('meta.php');
  5. include ('gora_logo.php');
  6. lewe_menu ();
  7. open_title();
  8. polacz_baza();
  9.  
  10. $i = 0;
  11. $ile = 4;
  12.  
  13. if (!isset($_GET['sala']) || $_GET['sala'] == '') 
  14. 			{
  15. 				$sale = mysql_query("SELECT id_sali, sala FROM sale ORDER BY sala desc;") or die(mysql_error());
  16. 				WHILE ($rekord = mysql_fetch_assoc ($sale)) 
  17. 				{
  18. 					$id_sali = $rekord['id_sali'];
  19. 					$sala = $rekord['sala'];
  20. 					echo '<font color="#FF000"-><h3><a href="sale.php?sala='.$id_sali.'">'.stripslashes($sala).'</a></h3></font><br />';
  21. 				}
  22. 			}
  23.  
  24. elseif (isset($_GET['sala']))
  25. 	{
  26.  
  27. 		$sale = mysql_query("SELECT * FROM sale ORDER BY sala desc;") or die(mysql_error());
  28. 		WHILE ($rekord = mysql_fetch_assoc ($sale)) 
  29. 			{
  30. 				$id_sali = $rekord['id_sali'];
  31. 				$sala = $rekord['sala'];
  32. 				$opiekun = $rekord['opiekun'];
  33. 				$info = $rekord['info'];
  34.  
  35. 				if (isset($_GET['sala']) && $_GET['sala']==$id_sali)
  36. 					{
  37. 						echo '<font color="#FF0000"-><h3>'.stripslashes($sala).'</h3></font><br />
  38. 						'.stripslashes($opiekun).'<br />
  39. 						'.stripslashes($info).'<br />';
  40.  
  41. 						$zdjecia = mysql_query("SELECT * FROM zdjecia WHERE id_kat = '".$sala."' ORDER by id_zdjecia DESC LIMIT 4;");
  42. 						$licz_zdjecia = mysql_num_rows($zdjecia);
  43. 						if ($licz_zdjecia!=0)
  44. 							{
  45. 								echo '<u>Zdjęcia z galerii dla sali: '.$sala.'</u><br /><p align="center">';
  46. 								WHILE ($rekord = mysql_fetch_assoc ($zdjecia)) 
  47. 								{
  48. 									$zdjecie = $rekord['zdjecie'];
  49. 									$opis = $rekord['opis'];
  50. 									echo '<a href="./upload/galeria/'.$zdjecie.'" border="0" alt="'.$opis.'" title="'.$opis.'">
  51. 									<img src="./upload/galeria/mini/'.$zdjecie.'" border="0"></a>';
  52.  
  53. 								}	
  54.  
  55. 							}
  56.  
  57. 						else
  58. 							{
  59. 								echo '<u>Nie ma zdjęć dla danej galerii</u><br /><p align="center">';
  60. 							}
  61. 					}					
  62. 			}	
  63.  
  64. 						echo '<a href="./sale.php">Powróć do SAL</a>';	
  65.  
  66. 				echo '</p>';
  67. 			}
  68. else
  69. 		{
  70. 			echo 'Błąd. Nie ma takiej strony.';
  71. 		}
  72.  
  73.  
  74.  
  75.  
  76.  
  77. echo '<br /></p>';
  78. close_title();
  79. include ('./stopka.php');
  80. ?>
[PHP] pobierz, plaintext

I teraz pytanie czy ten kod jest poprawny? A właściwie czy lepiej użyć ten drugi kod i czy jest bezpieczniejszy?