[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

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ć.

[Mephistofeles]

[PHP] pobierz, plaintext 
mysql_query("SELECT * FROM przeniesienie WHERE id=".$kto." ORDER BY numeracja DESC ")
[PHP] pobierz, plaintext 

Skąd masz $kto? Filtrujesz to?

[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

[phpion]

Używanie mysql_close() nie jest zwykle konieczne, ponieważ nie stałe linki są automatycznie zamykane na końcu wykonywania skryptu.

Źródło: mysql_close

[Crozin]

2. poczytaj sobie o MVC i rozdzielaniu kodu html od php.

MVC? To jest zbyt skomplikowany wzorzec wymagający jakiejś tam wiedzy - w dodatku jego użycie jest sensowne tylko w nieco bardziej rozbudowanych aplikacjach. Zresztą jego implementacja jest poza zasięgiem OPa. Co do rozdzielenia HTMLa i PHP - jak najbardziej tak.

Co do tematu: mysql_*()? , <font />? Mamy prawie 2011 rok, a Ty używasz reliktów zapomnianej już przeszłości. Co do pierwszego: PDO, co do drugiego: CSS

[Mephistofeles]

Szczerze mówiąc nie czytałem dokładnie przyklejonych tematów, ale może (jeśli tego nie ma ) czas odnowić informacje, napisać o wspomnianych przez Crozina reliktach, przecież męczenie się z z mysql_ w porównaniu do ładnego, a do tego względnie bezpiecznego obiektowego API PDO to jak wbijanie gwoździ kamieniem, czasami jeszcze umocowanym sznurkiem do patyka.