[PHP] PYTANIE O POPRAWNOŚĆ SKŁADNI Opcje

[adi456]

Proszę o sprawdzenie poprawności kodu

[PHP] pobierz, plaintext 
 
mysql_connect("***",$username,$password);
mysql_select_db($database) or die("<font color='red'>Baza zajeta kliknij cofnij lub odswiez</font>");
$query = mysql_query("SELECT * FROM przeniesienie WHERE id=".$kto." ORDER BY numeracja DESC ") or die(mysql_error());
 
mysql_close();
 
while ($row = mysql_fetch_array($query))
{
$numeracja= $row['numeracja'];
$id= $row['id'];
$nr= $row['nr'];
$kod = $row['kod'];
 
$przeniesione_tel = $przeniesione_tel.'<BR><BR> Telefon: '.$tel.' nr imei '.$imei.' kod: <font color="lime">'.$kod.'</font>';
 
mysql_connect("***",$username,$password);
@mysql_select_db($database) or die("Nie odnaleziono bazy danych");
$spr_id=mysql_fetch_array(mysql_query("SELECT * FROM telefon WHERE id=".$nr." "));
$id_tel= $spr_id['przenies'];
$przen_co = $id_tel.$przenies;
 
$poprawka = mysql_query("UPDATE towar SET gdzie='$gdzie', przenies='$przen_co' WHERE id= '$nr' AND gdzie='$gdzie_byl' AND stan='1'");
 
mysql_close();
 
if ($poprawka) 
{
echo '<font color="green">Telefon: '.$tel.' nr imei '.$imei.' kod: '.$kod.' - przeniesiony</font><br>'; 
}
else
{
echo '<font color="red">Telefon: '.$tel.' nr imei '.$imei.' kod: '.$kod.' - Nie przeniesiony!</font><br>'; 
};
 
 
};
[PHP] pobierz, plaintext 

a czy zamiast tego co jest wyżej można zrobić tak:

[PHP] pobierz, plaintext 
 
$id_tel= $spr_id['przenies'];
$przen_co = $id_tel.$przenies;
 
mysql_connect("***",$username,$password);
mysql_select_db($database) or die("<font color='red'>Baza zajeta kliknij cofnij lub odswiez</font>");
$query = mysql_query("SELECT * FROM przeniesienie WHERE id=".$kto." ORDER BY numeracja DESC ") or die(mysql_error());
$spr_id=mysql_fetch_array(mysql_query("SELECT * FROM telefon WHERE id=".$nr." "));
$poprawka = mysql_query("UPDATE towar SET gdzie='$gdzie', przenies='$przen_co' WHERE id= '$nr' AND gdzie='$gdzie_byl' AND stan='1'");
 
mysql_close();
 
while ($row = mysql_fetch_array($query))
{
$numeracja= $row['numeracja'];
$id= $row['id'];
$nr= $row['nr'];
$kod = $row['kod'];
 
$przeniesione_tel = $przeniesione_tel.'<BR><BR> Telefon: '.$tel.' nr imei '.$imei.' kod: <font color="lime">'.$kod.'</font>';
 
if ($poprawka) 
{
echo '<font color="green">Telefon: '.$tel.' nr imei '.$imei.' kod: '.$kod.' - przeniesiony</font><br>'; 
}
else
{
echo '<font color="red">Telefon: '.$tel.' nr imei '.$imei.' kod: '.$kod.' - Nie przeniesiony!</font><br>'; 
};
 
 
};
[PHP] pobierz, plaintext 

[adi456]

[PHP] pobierz, plaintext 
$kto = $_SESSION["kto"];
[PHP] pobierz, plaintext 

$kto jest to nr zależny od użytkownika który się zaloguje

a mam jeszcze pytanie na temat mysql_close, do tej pory robiąc skrypt gdy łączyłem się z bazą i odczytałem to co mnie interesowało zamykałem połączenie ale były problemy i błędy, a gdy teraz z wszystkich skryptów wykasowałem mysql_close wszystko świetnie działa i nie ma problemów z połączeniami więc czy może zostać bez mysql_close?

Ten post edytował adi456 10.12.2010, 15:36:21