Miałem pewien kod:
<?php
$title='Sale laboratoryjne';
include ('dodatki/bonus.php');
include ('meta.php');
include ('gora_logo.php');
lewe_menu ();
open_title();
polacz_baza();
$i = 0;
$ile = 4;
//zmienna $rekord jest przypisana wierszowi (rekordowi) zmiennej $news a następnie pobieranie danych z konkretnych kolumn (komórka po komórce)
{
$id_sali = $rekord['id_sali'];
$sala = $rekord['sala'];
$opiekun = $rekord['opiekun'];
$info = $rekord['info'];
//funkcja nl2br odpowiada za odpowiednie wyświetlanie po przez użycie np. znaków następnej linii
if (!isset($_GET['sala']) || $_GET['sala'] == '') {
echo '<font color="#FF000"-><h3><a href="sale.php?sala='.$id_sali.'">'.stripslashes($sala).'</a></h3></font><br />'; }
if (isset($_GET['sala']) && $_GET['sala'] == $id_sali) {
$zdjecia = mysql_query("SELECT * FROM zdjecia WHERE id_kat = '".$sala."' ORDER by id_zdjecia DESC LIMIT 4;"); if ($licz_zdjecia!=0)
{
echo '<u>Zdjęcia z galerii dla sali: '.$sala.'</u><br /><p align="center">'; }
else
{
echo '<u>Nie ma zdjęć dla danej galerii</u><br /><p align="center">'; }
{
$zdjecie = $rekord['zdjecie'];
$opis = $rekord['opis'];
echo '<a href="./upload/galeria/'.$zdjecie.'" border="0" alt="'.$opis.'" title="'.$opis.'"> <img src="./upload/galeria/mini/'.$zdjecie.'" border="0"></a>';
$i++;
if($i==$ile)
{
$i = 0;
}
}
echo '<a href="./sale.php">Powróć do SAL</a>';
}
}
close_title();
include ('./stopka.php');
?>
Aby zaoszczędzić czasu na zapytanie i poprawić kod napisałem:
<?php
$title='Sale laboratoryjne';
include ('dodatki/bonus.php');
include ('meta.php');
include ('gora_logo.php');
lewe_menu ();
open_title();
polacz_baza();
$i = 0;
$ile = 4;
if (!isset($_GET['sala']) || $_GET['sala'] == '') {
{
$id_sali = $rekord['id_sali'];
$sala = $rekord['sala'];
echo '<font color="#FF000"-><h3><a href="sale.php?sala='.$id_sali.'">'.stripslashes($sala).'</a></h3></font><br />'; }
}
elseif (isset($_GET['sala'])) {
{
$id_sali = $rekord['id_sali'];
$sala = $rekord['sala'];
$opiekun = $rekord['opiekun'];
$info = $rekord['info'];
if (isset($_GET['sala']) && $_GET['sala']==$id_sali) {
$zdjecia = mysql_query("SELECT * FROM zdjecia WHERE id_kat = '".$sala."' ORDER by id_zdjecia DESC LIMIT 4;"); if ($licz_zdjecia!=0)
{
echo '<u>Zdjęcia z galerii dla sali: '.$sala.'</u><br /><p align="center">'; {
$zdjecie = $rekord['zdjecie'];
$opis = $rekord['opis'];
echo '<a href="./upload/galeria/'.$zdjecie.'" border="0" alt="'.$opis.'" title="'.$opis.'"> <img src="./upload/galeria/mini/'.$zdjecie.'" border="0"></a>';
}
}
else
{
echo '<u>Nie ma zdjęć dla danej galerii</u><br /><p align="center">'; }
}
}
echo '<a href="./sale.php">Powróć do SAL</a>';
}
else
{
echo 'Błąd. Nie ma takiej strony.'; }
close_title();
include ('./stopka.php');
?>
I teraz pytanie czy ten kod jest poprawny? A właściwie czy lepiej użyć ten drugi kod i czy jest bezpieczniejszy?