wyjatki - problem Opcje

[Rafiki]

Witam.

Stworzylem plik extends w ktorym mam:

[PHP] pobierz, plaintext 
<?php
class system_Exception extends Exception {	
}
?>
[PHP] pobierz, plaintext 

nic wielkiego ...

Procz tego mam klase do obslugi mysqla, z cacheowaniem i inne 'gadzety'. W klasie tej w niektorych miejscach generowane sa wyjatki min w konstruktorze ktory ma za zadanie wywolanie polaczenia z baza, gdy polaczenie sie nie powiedzie generowany jest wyjatek w taki sposob:

[PHP] pobierz, plaintext 
<?php
throw new system_Exception('Błąd w nawiązaniu połączenia z bazą danych - ' . mysqli_connect_error($this -> connect) . ' - ' . mysqli_connect_errno($this -> connect));
?>
[PHP] pobierz, plaintext 

Niby wszystko ok... teraz mam index.php

[PHP] pobierz, plaintext 
<?php
include_once('config.php');
include_once('libs/exceptions.class.php');
include_once('libs/sql.class.php');
 
try
{
	$sql = & new sql('localhostaaa', 'user', 'password', 'dbname');
} catch (Exception $Ex) {
	include($site['dir'] . 'templates/' . $site['template'] . '/Exception.php');
} catch (system_Exception $sys_Ex) {
	include($site['dir'] . 'templates/' . $site['template'] . '/system_Exception.php');
}
?>
[PHP] pobierz, plaintext 

Dane przy wywolaniu klasy sa specjalnie zle podane aby zostal wychwycony wyjatek... ale wyjatek przechwyca catch z Exception a nie z system_Exception mimo ze w klasie do obslugi sql wywolywalem system_Exceptions.

System Exceptions ma sie roznic tym od zwyklego wyjatku ze ma wywalac bledy w 'jadrze' systemu na surowo na bialym tle z dokladnym opisem, linia bledu itd itd... zas zwykly wyjatek ma wyswietlac sam opis ktory ma sie wyswietlac w jakiejs okreslonej szacie graficznej strony, ale niestety nie dziala gdyz system_Exceptions pokazuje sie tak jak powinien sie pokazywac zwykly Exceptions ( bynajmiej tak mi sie wydaje (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Prosze o pomoc i z gory dziekuje.

[Cysiaczek]

Kwestia dziedziczenia.
Klasa system_Exception jest jednocześnie specjalizacją klasy Exception i jako taki jej obiekt jest przechwytywany jako pierwszy.
Po prostu przechwytywanie Exception daj do ostatniego catch i będzie działać,

Pozdrawiam.

Ten post edytował Cysiaczek 14.08.2006, 19:10:20

[mike]

Przykład: http://forum.php.pl/index.php?s=&showt...st&p=264066

[hwao]

Podstawy wyjątków, warto żebyś przeczytał... patrząc na to co piszesz.

[Rafiki]

Przeczytałem juz wczesniej, ale nie zwróciłem uwagi piszac skrypt ze Exception w tym przypadku musi byc na koncu.

Widzac co piszesz... ahah (IMG:http://forum.php.pl/style_emoticons/default/thumbsupsmileyanim.gif)

Ten post edytował Rafiki 15.08.2006, 11:46:02

[hwao]

Exception zawsze musi byc na samym końcu

[Cysiaczek]

@hwao - nie, nie musi być na końcu. Jeśli chcesz po prostu przechwycić błędy, a nie interesuje Cię jaki to błąd, to możesz śmiało 'przekleić' Exception na początek. (IMG:http://forum.php.pl/style_emoticons/default/laugh.gif)

Pozdrawiam.

[hwao]

@hwao - nie, nie musi być na końcu. Jeśli chcesz po prostu przechwycić błędy, a nie interesuje Cię jaki to błąd, to możesz śmiało 'przekleić' Exception na początek. (IMG:http://forum.php.pl/style_emoticons/default/laugh.gif)

Pozdrawiam.

To wtedy jaki jest sens pisanie pozostałych (tych pod exception?)? (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[Cysiaczek]

No w takim układzie to żaden (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)
Pytanie jednak brzmi: co zrobić, gdy chcemy w jednakowy sposób zareagowac na jakiekolwiek wyjatki?

[PHP] pobierz, plaintext 
<?php
//ano. Możemy przekleić Exception
try{
$ob=new Application();
}
catch (Exception $e){
$logger->notify($e->getMessage());
$logger->emailMe();
}
catch(specExc $e){}
catch(specExc2 $e){}
catch(specExc3 $e){}
?>
[PHP] pobierz, plaintext 

Co prawda przydatne raczej tylko przy testach...ale można (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Pozdrawiam

[mike]

W takiej sytuacji to też kiepskie wyjście.

Lepiej zachować normalną kolejnośc bloków catch, czyli od najszczegółowszego do Exception.

W takiej sytuacji kiedy chcemy na każdy wyjątek zareagować tak samo, to możesz dpisać troche kodu do klsy po której wszystkie dziedziczą.

Nie będziesz musiał zmieniać wszystkich try...catch w swojej aplikacji.

[dr_bonzo]

Manual: php przeglada kolejno wszystkie catch'e i po napotkaniu pierwszego z typem wyjatku bedacym klasa/nad-klasa aktualnie obslugiwanego wyjatku wchodzi do tego catch'a{}.

Cysiaczek:
juz lepiej to usun, bo wyglada jak bys nie przejmowal sie wystepujacymi wyjatkami. W tym przypadku wystarczy jeden catch z Exception

[PHP] pobierz, plaintext 
<?php
catch(specExc $e){}
catch(specExc2 $e){}
catch(specExc3 $e){}
?>
[PHP] pobierz, plaintext 

----------
edit: mike_mech

W takiej sytuacji kiedy chcemy na każdy wyjątek zareagować tak samo, to możesz dpisać troche kodu do klsy po której wszystkie dziedziczą.

Reakcja na wyjatek jest jego obsluga w catch'u a nie wewnatrz obiektu wyjatku :/

[Cysiaczek]

:roll2: Ja tylko pokazuję, że można... nie mówię, że to ma jakieś uzasadnienie projektowe (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Zdrówka. (IMG:http://forum.php.pl/style_emoticons/default/laugh.gif)

Ten post edytował Cysiaczek 15.08.2006, 15:13:05

[mike]

----------
edit: mike_mech
Reakcja na wyjatek jest jego obsluga w catch'u a nie wewnatrz obiektu wyjatku :/

Tak wiem, ale można zrobić:

[PHP] pobierz, plaintext 
<?php
try
{
}
catch( MyException $objException ) // MyException extends Exception a cała reszta extends MyException
{
	$objException->logError();
}
 
?>
[PHP] pobierz, plaintext 

Hmm, ale takie teoretyzowanie już nie za daleko prowadzi (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[dr_bonzo]

Ja zrozumialem ze chodzilo ci o zajecie sie wyjatkiem przy jego tworzeniu, w konstruktorze tegoz wyjatku (z wywolaniem parent::__construct()) -- zapisaniem go do logu, itp.

[mike]

Ja zrozumialem ze chodzilo ci o zajecie sie wyjatkiem przy jego tworzeniu, w konstruktorze tegoz wyjatku (z wywolaniem parent::__construct()) -- zapisaniem go do logu, itp.

O to też.
Zapomniałem dopisać :/