Kiedy używać wyjątków? Opcje

[zbig13]

Witam!

Jestem w trakcie tworzenia obiektowego skryptu w php5. Zacząłem od własnego sterownika bazy danych. Napisałem taki kod:

[PHP] pobierz, plaintext 
<?php
 
class DBMySQL
{
 
	private static $dbcon;
	private static $dbhandle;
	private static $query;
	private static $liczbaZapytan;
	private static $queryResult;
	private static $temp;
 
	public function __construct()
	{
		self::$zapytan = 0;
		self::$dbcon = FALSE;
	}
 
	public static function connect($host, $user, $pswd)
	{
		self::$dbcon = @mysql_connect($host, $user, $pswd);
		if (!self::$dbcon) {
			throw new Exception('Nie udało się połączyć z bazą danych.');
		}
		return self::$dbcon;
	}
 
	public static function selectDb($dbname)
	{
		self::$dbhandle = @mysql_select_db($dbname);
		if (!self::$dbhandle) {
			throw new Exception('Nie udało się wybrać bazy.');
		}
		return self::$dbhandle;
	}
 
	public static function query($query)
	{
		self::$liczbaZapytan++;
		self::$query = $query;
		return mysql_query(self::$query);
	}
 
	public static function fetchArray($result)
	{
		self::$queryResult = $result;
		self::$temp = mysql_fetch_array(self::$queryResult);
		return self::$temp;
	}
 
	public static function numRows($result)
	{
		self::$queryResult = $result;
		return mysql_num_rows(self::$queryResult);
	}
 
	public static function realEscapeString($string)
	{
		return mysql_real_escape_string($string, self::$dbcon);
	}
 
	public static function startTransaction()
	{
		self::query('BEGIN');
	}
 
	public static function acceptTransaction()
	{
		self::query('COMMIT');
	}
 
	public static function rollbackTransaction()
	{
		self::query('ROLLBACK');
	}
 
	public static function returnQueries()
	{
		return $liczbaZapytan;
	}
 
}
 
?>
[PHP] pobierz, plaintext 

W dwóch sytuacjach użyłem rzucania wyjątków w przypadku błedu (łączenie z bazą i wybieranie bazy). Chciałem też dać obsługe wyjątków do innych funkcji, np. query(), ale jakoś nie wyobrażam sobie później takiego wykonywania zapytań:

[PHP] pobierz, plaintext 
<?php
try {
	DBMySQL::query("SELECT * FROM tabela");
} catch (Exception $e) {
	echo $e->getmessage();
}
?>
[PHP] pobierz, plaintext 

Jest jakiś sposób na rzucanie wyjątków, żeby nie trzeba było ich później za każdym razem obsługiwać w kodzie? I czy warto rzucać wyjątki w przypadku takiej funkcji jak query(), lub numRows()?

Ten post edytował zbig13 27.03.2006, 07:56:12

[Prph]

Nie rozumiem, dlaczego query to zle miejsce na wyjatek. Jak dla mnie idealne.

Mozesz tobic tak:

[PHP] pobierz, plaintext 
<?php
 
 
$wynik = DB::query('zapytanie');
if(!$wynik)
  return fasle;
 
$wiersz = DB::fetch();
 
..
..
 
idt.
 
?>
[PHP] pobierz, plaintext 

albo, majac wyjatki w query:

[PHP] pobierz, plaintext 
<?php
 
	try
	{
	  $this->_oDatabase->insert('session', $aToInsert);
 
	  $sQuery = 'SELECT id FROM session WHERE phpsessid = "'.$this->_sPHPSessionId.'"';
	  $this->_oDatabase->query($sQuery);
 
	  $aRow = $this->_oDatabase->fetch();
	  $this->_iNativeSessionId = $aRow['id'];
 
	  $this->_sPHPSessionId = $sPHPSessionId;
	}
	catch (Exception $oException)
	{
	   throw $oException;
	}
 
?>
[PHP] pobierz, plaintext 

Czy to nie jest wygodne?

[zbig13]

Nie rozumiem tych Twoich przykładów. Mi chodzi po prostu o to, żeby w tej funkcji:

[PHP] pobierz, plaintext 
<?php
	public static function query($query)
	{
		self::$liczbaZapytan++;
		self::$query = $query;
		return mysql_query(self::$query);
	}
?>
[PHP] pobierz, plaintext 

dodać rzucenie wyjątku jeśli nie uda się wykonać zapytania. Mogę zrobić tak jak w funkcji DBMySQL::connect(), ale wtedy przy każdym wykonaniu zapytanie będe musiał pisać:

[PHP] pobierz, plaintext 
<?php
try {
	DBMySQL::query("SELECT * FROM tabela");
} catch (Exception $e) {
	echo $e->getmessage();
}
?>
[PHP] pobierz, plaintext 

A wolałbym prościej:

[PHP] pobierz, plaintext 
<?php
DBMySQL::query("SELECT * FROM tabela");
?>
[PHP] pobierz, plaintext 

[dr_bonzo]

A wolałbym prościej: ....

przeciez i tak bedziesz musial sprawdzic czy zaputanie sie powiodlo, np.

[PHP] pobierz, plaintext 
<?php
if ( DBMySQL::query("SELECT * FROM tabela") )
{
...
}
else
{
 ...
} 
?>
[PHP] pobierz, plaintext 

nie musisz lapac wyjatkow po kazdym zapytaniu, tylko otaczasz try/catch od laczenia sie z baza, poprzez wybieranie bazy, zapytania az dorozlaczenia. No chyba ze potrzebujesz to zrobic inaczej.

[zbig13]

Ok juz rozumiem. Dzięki za pomoc (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[SongoQ]

Dorzuc tam transakcjem, logowanie czasow zapytania i explain

[zbig13]

Mam jeszcze jedno pytanie w związku z tym rzucaniem wyjątków.
Lepiej sprawdzać w metodzie klasy czy zapytanie się powiodło i w razie błędu wyrzucić wyjątek, czy może nie sprawdzać tego w metodzie klasy a po prostu w skrypcie sprawdzac czy wynik zapytania != false? Bo jak sprawdzam w metodzie, to cały skrypt mam objęty instrukcją try...catch i jak wystąpi jeden wyjątek to cały skrypt zostanie zatrzymany, a głupio trochę zatrzymać cały system jak tylko jedno zapytanie się nie powiedzie.
Więc jak lepiej?

  Dorzuc tam transakcjem, logowanie czasow zapytania i explain

A co to są transakcje? (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

[SongoQ]

A co to są transakcje?

W google znajdziesz. Dziwie sie ze w ogole zapytales, bo to jest podstawa relacyjnych baz danych. To tak jak bys pytal co to FOR w C

[zbig13]

Ok, już dodałem (kod w moim pierwszym poście) tylko nie wiem czy jest to przydatne w skrypcie, bo przecież zanim się doda jakieś dane do bazy to trzeba to sprawdzić wszystkimi możliwymi sposobami (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) więc raczej nie ma potrzeby cofania zapytań. (przynajmniej mi się nigdy coś takiego nie zdarzyło)

A masz jakieś sugestie co do pierwszego pytania? (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[Prph]

Jest bardzo wiele sytuacji w ktorzych korzystasz z transakcji. Sprzojrz na przyklad:

Kod

zapytanie = wstaw do bazy nowa glaerie;
if(baza->wykonaj(zaptanie))
{
  if(!utworz_katalog(strona/katalogi/uzytownik/nowa_galeria))
  {
    // okazalo sie nie nie mogl go utworzyc.
    zapytanie = usun z bazy wczesniej dodana galerie.
    if(!baza->wykonaj(zaptanie))
    {
       no i spojnosc danych poszla sie..;)
    }
  }
}

Z wykorzystaniem stansakcji

Kod

start_transakcji;

zapytanie = wstaw do bazy nowa glaerie;
if(baza->wykonaj(zaptanie))
{
  if(!utworz_katalog(strona/katalogi/uzytownik/nowa_galeria))
  {
    // okazalo sie nie nie mogl go utworzyc.
    abort;
  }
  else
    commit;
}

commit mozna w zasadzie pominac, bo zazwyczaj w bazie ustawuone jest autocommit i na koncy dane i tak zostana zapisane w bazie.

Pozdrawiam.

[zbig13]

No dobra przekonałeś mnie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

A co do tej funkcji query to chyba nie będe jednak dodawał w niej rzucania wyjątków, bo to troche bezsensu że cały skrypt się zatrzyma w przypadku błędu w zapytaniu.

@SongoQ Logowanie czasów zapytania i explain postaram się dodać w najbliższym czasie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Jeszcze raz dzięki wam za pomoc.

Pozdrawiam,
Zbig

[SongoQ]

Mozesz ewentualnie zrobic cos w rodzaju debug zapytania. Ustawiasz sobie jakis config gdzie podajesz max dla zapytania i np przy wersji debug wyswietlasz sobie zapytania ktore przeszly zadany czas a ktore nie, ktore sie wywalaja itd. Tam samo mozesz postapic z explain. Jak sobie zorganizujesz to juz zalezy tylko od Ciebie. Dla mnie np takie rzeczy sa bardzo wazne.

[Vengeance]

Bo jak sprawdzam w metodzie, to cały skrypt mam objęty instrukcją try...catch i jak wystąpi jeden wyjątek to cały skrypt zostanie zatrzymany, a głupio trochę zatrzymać cały system jak tylko jedno zapytanie się nie powiedzie.

Jak sama nazwa wskazuje, wyjątek służy do informowania o sytuacjach wyjątkowych, czyli takich po wystąpieniu których aplikacja musi być wstrzymana (ewentualnie jakiś blok operacji w aplikacji).

Dlatego stwierdzenie, że "trochę głupio jest zatrzymać system" jest samo w sobie bardzo głupie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Chyba rozumiesz?

[zbig13]

Tak rozumiem, dlatego zrezygnowałem z wyjątków w funkcji query (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[mike]

Bo jak sprawdzam w metodzie, to cały skrypt mam objęty instrukcją try...catch i jak wystąpi jeden wyjątek to cały skrypt zostanie zatrzymany, a głupio trochę zatrzymać cały system jak tylko jedno zapytanie się nie powiedzie.

Jak sama nazwa wskazuje, wyjątek służy do informowania o sytuacjach wyjątkowych, czyli takich po wystąpieniu których aplikacja musi być wstrzymana (ewentualnie jakiś blok operacji w aplikacji).

Dlatego stwierdzenie, że "trochę głupio jest zatrzymać system" jest samo w sobie bardzo głupie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Chyba rozumiesz?

~Vengeance mylisz się.
Wyjątek jest to nagła styuacja, na którą trzeba zareagować.

A przerwanie działania systemu/aplikacji to najgorsza reakcja.

Nie można sobie (a przynajmniej nie powinno) zatrzymać działania w fowolnej chwili bo wyskoczył jakiś wyjątek, no właśnie po to wyskoczył żeby nam powiedzieć że musimy zrobić coś inaczej niż planowaliśmy.

To zależy od natury wyjątku, ale powinniśmy ładnie dalej poprowadzić aplikacją na tory boczne, które zaprowadzą użytkownika do ładnego ekranu powiadamiającym go o błędzie. Wcześniej możemy wykonać wiele operacji: logować wyjątek, wysyłać maila do admina, ... .

Kill process to bardzo zły wybór.

[SongoQ]

To zależy od natury wyjątku, ale powinniśmy ładnie dalej poprowadzić aplikacją na tory boczne, które zaprowadzą użytkownika do ładnego ekranu powiadamiającym go o błędzie.

Wyjatek moze rowniez sluzyc jako "redukcja" warunkow, wiec dokladnie zejscie na tory boczne jest tylko w wiekszosci przypadkow.

[zbig13]

To konkretnie jak lepiej robić? :
- Tam gdzie to możliwe umieszczać wyjątki
czy
- Unikać wyjątków i zastępować je warunkami

[Vengeance]

mike_mech: Chodziło mi raczej oto, że stosowanie wyjątku dla np.:

[PHP] pobierz, plaintext 
<?php
 
$request->Get('action');
 
?>
[PHP] pobierz, plaintext 

Gdzie parametr action nie istnieje jest moim zdaniem głupie. Nie lubię, gdy ktoś załkowicie usuwa ify() na rzecz wyjątków.

[DeyV]

@ Vengeance - ale już w przypadku
system::createAction( 'news' );
wyrzucenie wyjątku może okazać się nazzwyczaj przydatne.
Zawsze wtedy masz możliwośc załadowania innej akcji, zrobienie czegoś innego.

Podobnie sytuacja wygląda np. z autoryzacją - nie masz uprawnień, to się zaloguj, i nie ważne jest to, w którym momencie system to zauważył. Ważne, że to dostrzegł, i w zwiazku z tym może coś z tym zrobić.

Apropo wyjątków i transakcji - mi osobiście bardzo podoba sie taki przykład:

[PHP] pobierz, plaintext 
<?php
 
 
try{
   $Db->startTrans();
 
   $Db->Execute( pytanie );
   $Db->Execute( pytanie2 );
   $Db->Execute( pytanie3 );
 
   system::robCos();
   
   $Db->commitTrans();
}
catch( ExceptionDB $e ){
   $Db->abortTrans();
}
 
?>
[PHP] pobierz, plaintext 

I niech ktoś w czytelniejszy sposób zapisze transakcje (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

[Vengeance]

Ale czy ja mówię, że wyjątki są złe? Przykłady które podałeś są jak najbardziej dobre... i pokazują jak wyjątki są przydatne. Ale czy cieszyłbyś się, gdyby każda funkcja php zamiast np. false w razie błędu zwracała wyjątek? Ja nie... bo nie do tego są wyjątki by stosować je wszędzie gdzie się da. A że temat brzmi "kiedy używać wyjątków" chciałem zwrócić na to uwagę autora