[PHP]Wyjątki PHP, na przykładzie Opcje

[wNogachSpisz]

Witam

Jestem w trakcie pisania biblioteki - ujednoliconego API - służącej do obsługi usług rozwiązywania captch. Nie muszę chyba nikogo przekonywać, że dobrze zaprojektowany system wyjątków to podstawa. Niestety nigdy się w to nie bawiłem, tzn. często używam wyjątków, ale nigdy nie próbowałem definiować własnych klas wyjątków. Przeczyłem ten artykuł: http://blogs.msdn.com/b/kcwalina/archive/2...ierarchies.aspx
Uznałem że zaproponowany tam podział na "usage exception" oraz "system exception" wydaje się rozsądny.
Moj kod wygląda następująco:

[PHP] pobierz, plaintext 
class Decaptcha_Exception extends Exception
{}
class Decaptcha_Exception extends Exception
{}
/*
	99: unknown error
 */
 
	class DecaptchaUsage_Exception extends Decaptcha_Exception
	{}
	/*
		101: invalid login or password
		102: captcha file not found / file empty
		103: not logged in
	 */
 
	class DecaptchaSystem_Exception extends Decaptcha_Exception
	{}
 
		class DecaptchaSystemLogical_Exception extends DecaptchaSystem_Exception
		{}
		/*
			201: unable to decode captcha (too hard or malformed image)
			202: banned
			203: insufficient funds
			204: service overload
			205: invalid response
			206: connection failed/lost/timeout
			207: network error
		 */
 
		class DecaptchaSystemFailure_Exception extends DecaptchaSystem_Exception
		{}
		/*
			301: php extension not found
			302: php function not found
		 */
[PHP] pobierz, plaintext 

(przepraszam za tabulatory ale odzwierciedlają one hierarchę).

Pytanie: Co należałoby tu zmienić, czy np. "202: insufficient funds" pasuje bardziej do "SystemLogical_Exception" czy może "Usage_Exception". Czy to co zrobiłem ma jakikolwiek sens? (IMG:style_emoticons/default/smile.gif)

Ten post edytował wNogachSpisz 26.02.2013, 19:10:13

[Crozin]

Chyba odwrotnie, LogicalException to błąd użycia, a RuntimeException, błąd programu.

Nie, LogicException to błędy w samym kodzie, np.:

[PHP] pobierz, plaintext 
<?php
 
$command->stop(); // LogicException: Komenda nigdy nie została wystartowana, więc nie może zostać zatrzymana
 
$command->start();
$command->stop(); // OK
[PHP] pobierz, plaintext 

RuntimeException to zaś wszystkie błędy, które wynikły ze względu na otrzymane z zewnątrz dane (od użytkownika, z pliku czy z sieci).

Tak czy inaczej, podział ten wydaje gorszy od zaproponowanego przez P. Krzysztofa. Podam przykład: "network error" problem z siecią nie powstaje z powodu błędnego inputa, ani tego że program jest źle napisany, tylko.. no wlasnie, z przyczyn niezależnych :-]

Nie zgodziłbym się. Jeżeli próbujesz skorzystać np. z socketu, który nie jest w ogóle podpięty (zbindowany) możemy mówić o LogicException - bo jako programista powinieneś go zbindować. Jeżeli zaś problem pojawił się z powodu wprowadzenia poprawnej, lecz nieistniejącej nazwy hosta, albo akurat internet nam padł jest to RuntimeException. Jeżeli jakiś błąd wynika z przyczyn niezależnych niemal zawsze będzie to RuntimeException.

Nigdy nie potrzebowałem, ale dzisiaj uznałem że czas na małe rozwarstwienie. Warto zauważyć, że zostawiam sobie fallback, mianowicie, wszystkie wyjątki można rozpoznać bez podziału na klasy po ich unikatowych kodach błędu.

Podobnie jak autor artykułu czy Ty sam, nie jestem zwolennikiem tworzenia dziesiątek klas dla wyjątków. Zauważ, że u Ciebie spokojnie można wykorzystać wbudowane wyjątki PHP do osbługi przynajmniej części. Resztę z nich mógłbyś zapewne obsłużyć przy pomocy dużo "płytszej" hierarchii.

Hierarchia wyjątków zawsze będzie stanowiła dosyć poważny problem, ale przy raptem 13 wyjątkach jakie tutaj zaprezentowałeś odwzorowywanie hierarchii z C#/.NET nie jest dobrym pomysłem. Szczególnie, że pokrywa się częściowo z SPL-owską.