[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 

[skowron-line]

1. NIE KRZYCZ
2. poczytaj sobie o MVC i rozdzielaniu kodu html od php.
3. jeżeli kod się wykonuje poprawnie to jest poprawny (IMG:style_emoticons/default/smile.gif)

4. mysql_connect i mysql_select_db zamknij sobie w jakiejś funkcji bo jak Ci się zmieni hasło do bazy danych to 2 mln miejsc będziesz musiał zmienić.