Obsługa błędów Opcje

[-Code46-]

[PHP] pobierz, plaintext 
<?php
 
		  //------------------->ROZPOCZCIE TRANSAKCJI
		  $query = &#092;"BEGIN WORK\";
		  $result = mysql_query($query)
				or die (&#092;"Zapytanie nieudane! BŁĄD: \".mysql_error());
 
		  //Dodanie loginu i hasła
		  $query = &#092;"INSERT INTO konto(login, haslo, kto) VALUES\" .
				 &#092;"('$_POST[login]','\". MD5($_POST[haslo]) . \"','$_POST[Kto]');\";
		  $result = mysql_query($query)
				or die (&#092;"Zapytanie nieudane! BŁĄD: \".mysql_error());
		  //Sprawdzenie wartości klucza ID
		  $query = &#092;"SELECT LAST_INSERT_ID()\";
		  $result = mysql_query($query)
				or die (&#092;"Zapytanie nieudane! BŁĄD: \".mysql_error());
		  $query_data = mysql_fetch_row($result);
		  $ID = $query_data[0];
 
		  //Dodanie reszty danych
		  $query = &#092;"INSERT INTO klient (konto_id, imie, nazwisko, pesel, email, firma, ulica, kod, miejscowosc, tel
e
on, fax, regon, nip)\".
				 \" VALUES ($ID,'$_POST[imie]','$_POST[nazwisko]','$_POST[pesel]','$_POST[email]','$_POST[firma]','$_POST[ulica]','$_POST[kod]','$_POST[miejscowosc]','$_POST[telefon]','$_POST[fax]','$_POST[regon]','$_POST[nip]');\";
		  $result = mysql_query($query)
				or die (\"Zapytanie nieudane! BŁĄD: \".mysql_error());
 
		  //-------------------> ZATWIERDZENIE TRANSAKCJI
		  $query = \"COMMIT\";
		  $result = mysql_query($query)
				or die (\"Zapytanie nieudane! BŁĄD: \".mysql_error());
 
		  //Zamknięcie połączenia z bazą danych
		  mysql_close($conn);
 
?>
[PHP] pobierz, plaintext 

Parser php jeśli napotka błąd w zapytaniu mysqla, wyświelti na stronie napis "Zapytanie nieudane i opis błędu" a skrypt przestanie się wykonywać czyli żadna zmiana nie zostanie wprowadzona do bazy.

1. Czy powyższy kod jest poprawny? On działa ale czy jest "profesionalny"? Może robię coś nie dokońca poprawnie?
2. Jak zrobić, żeby zamiast opisu błędu podawał kod błędu (liczbowy)? I gdzie można to sprawdzić co taki błąd oznacza? Nie chce żeby użytkownicy wiedzieli co poszło nie tak.

3. Czy taki zapis

[PHP] pobierz, plaintext 
<?php
		  $result = mysql_query($query)
				or die (&#092;"Zapytanie nieudane! BŁĄD: \".mysql_error());
 
?>
[PHP] pobierz, plaintext 

można zastąpić innym? Np takim:

[PHP] pobierz, plaintext 
<?php
		  $result = mysql_query($query);
				  if (!$result) then wyswietl_ladny_blad();
 
?>
[PHP] pobierz, plaintext 

a)Czy można napisać funkcję bez parametrów wyswietl_ladny_blad() , która będzie tworzyła napis w stylu "Wystąpił błąd: KOD BŁDU: $kod_błędu"?
b ) Jak to zrobić?
c)I czy można ją zapisać w pliku include.php.inc (w tym pliku mam nazwe gosta, nazwę bazy i hasło do bazy)?

[ActivePlayer]

poczytaj o wyjątkach w php5... i poczytaj tez

mysql" title="Zobacz w manualu PHP" target="_manual
mysql_errno" title="Zobacz w manualu PHP" target="_manual

powinno wystarczyc.

[mike]

Odnośnie "profesjonalności" :
1.

[PHP] pobierz, plaintext 
<?php
$query = &#092;"BEGIN WORK\";
$result = mysql_query($query);
?>
[PHP] pobierz, plaintext 

W sytuacji gdzie zapytanie jest bardzo skomplikowane - owszem, ale nie lepiej:

[PHP] pobierz, plaintext 
<?php
$result = mysql_query( 'BEGIN WORK' );
?>
[PHP] pobierz, plaintext 

2.
Standardy kodowania
3.
Staraj sie nie używać " tam gdzie można '.

2. i 3. to takie drobnostki.
Ale to są indywidualne sprawy programisty.

Dodatkowo to co napisał @ActivePlayer.

[ActivePlayer]

profesjonalnosc kody to pojecie względne... moglbys dzielic to na fukncje, klasy, uzywac db layerow i innych wymyslnych metod... czasem smieje sie z tego bo przylapalem siebie sam, na tym ze zamiast 2 linijek kodu, dołączam jakies biblioteki z nieba wzięte... pamiętaj, jesli Twój kod jest łątwy w utrzymaniu i edycji, do tego szybki i malo podatny na bledy, to jest to dobry kod.

[-Guest-]

Dzięki Panowie. Przeczytałem. Ale problem z tą funkcją pozostał. Wiem już jak pobrać numer błędu. Chodziło mi o to, czy jest na stronie mysql wykaz błędów.
Czyli klient coś zapisuje do bazy, zapisanie nie udało się a on zobaczy tylko błąd:1048 i tyle - bez opisu. A ja sobie biorę taki kod i sprawdzam na stroni co on oznacza.

Druga sprawa.
Czy sam warunek jest poprawnie zapisany

[PHP] pobierz, plaintext 
<?php
		  $result = mysql_query($query);
				  if (!$result) then wyswietl_ladny_blad();
 
?>
[PHP] pobierz, plaintext 

I jak z tą funkcą. Nie chce korzystać z brzydkiego zapisu echo "Błąd ble ble ble"
tylko napisać funkcję wyswielt_ladny_blad(), która sormatuje tabelkę na środku strony a w niej ładny napis np "Operacja nie udała się. Wystąpił błąd: .....". Żeby za każdym razem nie powielać tych samych kilku linijek a wywoływać funkcję z innego pliku..
Da się coś takiego zrobić?

[ActivePlayer]

warunek jest ok. poczytaj jeszcze o tym jak poprawnie pisac przejrzysty(ort?) kod. mike_mech dal linka. jak chcesz dawać ładny błąd to w funkcji wyswietl_ladny_blad() zapisuj nazwe i ID błedu do pliku//bazy//gdziekolwiek, a userowi wyswietlasz tylko ze 'wystąpiły problemy techniczne'. aha i odradzam uzywanie dorgi mailowej do raportowania o błędach mysql.... padnie Ci serwer mysqla i zanim sie obejrzysz bedziesz mial 20000 listow z raportami bledów. niezbyt miłe

[-Code46-]

poczytaj jeszcze o tym jak poprawnie pisac przejrzysty(ort?) kod. mike_mech dal linka. jak chcesz dawać ładny błąd to w funkcji wyswietl_ladny_blad() zapisuj nazwe i ID błedu do pliku//bazy//gdziekolwiek, a userowi wyswietlasz tylko ze 'wystąpiły problemy techniczne'. aha i odradzam uzywanie dorgi mailowej do raportowania o błędach mysql.... padnie Ci serwer mysqla i zanim sie obejrzysz bedziesz mial 20000 listow z raportami bledów. niezbyt miłe

Ok

warunek jest ok.

Wiesz co w między czasie próbowałem sam. I chcaiłem przetestować. Do tej pory miałem tak:

[PHP] pobierz, plaintext 
<?php
 
  $query = &#092;"SELECT COUNT(login) FROM konto WHERE kto='0'\";
	  $result = mysql_query($query)
	  or die (&#092;"Zapytanie nieudane! BŁĄD: \".mysql_error());
 
?>
[PHP] pobierz, plaintext 

i chciałem żeby po każdym udanym odczycie wypisał komunikat "ZAPYTANIE UDAŁO SIE", więc przerobiłem to na taki kod:

[PHP] pobierz, plaintext 
<?php
 
  $query = &#092;"SELECT COUNT(login) FROM konto WHERE kto='0'\";
  $result = mysql_query($query);
 
   if ($result) then echo &#092;"Zapytanie udane\";
 
?>
[PHP] pobierz, plaintext 

Ale parser zatrzymuje wykonywanie skryptu i widzę białą stronę. Dlatego obawiam się czy zapis jest poprawny...

[ActivePlayer]

[PHP] pobierz, plaintext 
<?php
$query = &#092;"SELECT COUNT(login) FROM konto WHERE kto='0'\";
$result = mysql_query($query);
if ($result){
   echo &#092;"Zapytanie udane\";
}
else{
   pokaz_ładny_błąd();
   die();
}
 
?>
[PHP] pobierz, plaintext 

to 'then' to jakies basicowe czy c++sowe ?

Ten post edytował ActivePlayer 19.04.2005, 21:41:42

[-Code46-]

to 'then' to jakies basicowe czy c++sowe ?

Hehe żeczywiście złe nawyki. A jeszcze jedno - wiesz jak taką funkcje napisać?

numer 1 - oznacza że nie ma takiej bazy
2 - nie ma takiego rekordu
..
..
itd.

Piszę jedną funkcę którą wrzucam do jakiegoś pliku (możę byc plik config.inc.php - w nim zapisuje parametry połączenia z bazą?), żeby nie powielać kilka razy tego samego kodu.
W funkcji mam np taki kod:

[PHP] pobierz, plaintext 
<?php
 
 
		   <br><br><br><br><br><br><br><br><br>
		   <table width=&#092;"80%\" align=\"center\" border=\"0\" cellpadding=\"14\">
				  <tr>
					  <td><h2 class=&#092;"error\">$KOMUNIKAT</h2></td>
				  </tr>
		   </table> 
?>
[PHP] pobierz, plaintext 

A zamiast pisać np

[PHP] pobierz, plaintext 
<?php
 
 
  $query = &#092;"SELECT COUNT(login) FROM konto WHERE kto='0'\";
  $result = mysql_query($query)
	 or die (&#092;"Zapytanie nieudane! BŁĄD: \".mysql_error()); 
 
?>
[PHP] pobierz, plaintext 

pisałbym

[PHP] pobierz, plaintext 
<?php
 
 $query = &#092;"SELECT COUNT(login) FROM konto WHERE kto='0'\";
  $result = mysql_query($query);
  if (!$result) wypiszblad($kodbledu);
 
?>
[PHP] pobierz, plaintext 

[ActivePlayer]

to 'then' to jakies basicowe czy c++sowe ?

Hehe żeczywiście złe nawyki. A jeszcze jedno - wiesz jak taką funkcje napisać?

numer 1 - oznacza że nie ma takiej bazy
2 - nie ma takiego rekordu
..
..
itd.

Piszę jedną funkcę którą wrzucam do jakiegoś pliku (możę byc plik config.inc.php - w nim zapisuje parametry połączenia z bazą?), żeby nie powielać kilka razy tego samego kodu.
W funkcji mam np taki kod:

[PHP] pobierz, plaintext 
<?php
 
 
           <br><br><br><br><br><br><br><br><br>
           <table width=&#092;"80%\" align=\"center\" border=\"0\" cellpadding=\"14\">
                  <tr>
                      <td><h2 class=&#092;"error\">$KOMUNIKAT</h2></td>
                  </tr>
           </table> 
?>
[PHP] pobierz, plaintext 

A zamiast pisać np

[PHP] pobierz, plaintext 
<?php
 
 
  $query = &#092;"SELECT COUNT(login) FROM konto WHERE kto='0'\";
  $result = mysql_query($query)
     or die (&#092;"Zapytanie nieudane! BŁĄD: \".mysql_error()); 
 
?>
[PHP] pobierz, plaintext 

pisałbym

[PHP] pobierz, plaintext 
<?php
 
 $query = &#092;"SELECT COUNT(login) FROM konto WHERE kto='0'\";
  $result = mysql_query($query);
  if (!$result) wypiszblad($kodbledu);
 
?>
[PHP] pobierz, plaintext 

ale po co userowi przedstawiac kod błędu. lepiej zapisz sobie go gdzies, np w pliku. a user niech zobaczy jakis statyczny napis informujący o problemach z serwerem czy cokolwiek.