Forum PHP.pl

Jak w temacie, czy myslicie ze tworzenie specjalnej grupy obiektow do obslugi roznorodnych zdarzen: informacji debug, wyjatkow, bledow jest dobrym pomyslem?

W praktyce wygladaloby to tak:

$events = new EventListener();
$events->registerHandler('Exception', new ExceptionHandler());

// Zlapany w bloku catch() wyjatek:
$events[] = $e;

Powiadamiamy w ten sposob EventListenera o zajsciu nowego zdarzenia. Nastepnie zostaje wykonany lancuszek klas odpowiedzialnych za obsluge danego typu zdarzenia.

Co sadzicie o tym?

Pozdrawiam,

Czy nie chodzi ci o to samo, czym było by wykorzystanie funkcji set_exception_handler() ?

Nie, bo imho zdarzenia nie musza miec nic wspolnego z wyjatkami...

A może wzorzec Observer ? Sama nazwa, jak i zastosowanie spełnią chyba twoje wymagania.

Serafin - podoba mi się to rozwiązanie

Kilka luźnych uwag:

1) To jest chyba wykorzystanie interfejsu ArrayAccess?

2) Jak Observer, to warto (jeżeli się da) wykorzystać kod z SPL. Nic odkrywczego nie wymyślili, ale może uda się wypracować jakiś standard/

3) Bardzo by się przydał lazy loading, bo przecież ExceptionHandler przeważnie nie będzie wywoływany. Albo handle/proxy (jak w WACT lub phiend.handle), albo delegate/invoker (jak w WACT lub niepublikowanej jeszcze części phiend.handle).

4) Zmieniłbym nazewnictwo, bo EventListener sugeruje, że jest to Observer, a tymczasem jest to drugi element tego wzorca - Observable. Robiony przeze mnie pakiet phiend.logger ma klasę Logger, którza przyjmuje wiadomości/zdarzenia i kieruje do listenerów. Efekt jest chyba podobny...

@Hawk Wątpię aby to byl ArrayAccess. Ja osobiscie tez wykorzystuje statyczny rejestr w ktorym zapisuje komunikaty -np. nie mozna sie polaczyc z baza i jechac tylko na cache o ile sie da.

Co handlerow - tez mam swoje - inicjowane wraz z jadrem systemu - choc zastanawiam sie czy jest jakas metoda ktora pozwoli na to aby ominac wywolanie handler jesli w aplikacji nie zajdzie zaden wyjatek Logger tez jest a majac do dyspozycji Reflection Class mozna robic naprawde fajne rzeczy

@NuLL: Twoje watpliwosci sa nieuzasadnione - chyba nie uwazasz na lekcji

[PHP] pobierz, plaintext 
<?php
 
$events = new EventListener();
$events->registerHandler('Exception', new ExceptionHandler());
$events[] = $e;
 
?>
[PHP] pobierz, plaintext 

Ciekawe jak inaczej?

Ale ja nie chce rejestrowac tego handlera dopoki nie jest mi potrzebny - ba, najlepiej zadnego handlera nie inicjowac. Tu robisz wywolanie jawnie inicjujac aplikacje - ja chce tego uniknac. Pomysl juz mam
Pozatym pomysl z tablica jest poroniony. Lepiej wg mnie.

[PHP] pobierz, plaintext 
<?php
 
Logger::Register(&#092;"/e NoSuchComponent\");
 
?>
[PHP] pobierz, plaintext 

Przelaczniki ala DOS i to mi sie podoba

Zmienilem nieco idee. Pokazuje wiec kod:

[PHP] pobierz, plaintext 
<?php
 
// Prosty obserwator;)
final class DebugPrint implements Observer {
 
	private $subject;
 
	public function register(Subject $s) {
		$this->subject = $s;
		$s->attach($this);
	}
 
	public function unregister(Subject $s) {
		if($this->subject) {
			$this->subject->detach($this);
		}
	}
 
	public function update(Subject $s) {
		$this->subject = $s;	
		 $call = debug_backtrace();
		 $call = $call[2];
		 echo '<p style=\"font-family: Verdana; font-size: 10px; padding: 0px; margin: 0px;\"><b style=\"color: red\">Debug</b>: ' . $this->subject->getMsg() . ' <!--<a href=\"source.php?file='.$call['file'].'&line='.$call['line'].'\" target=\"_blank\">'.basename($call['file']) . ' @ line '.$call['line']. '</a>--></p>';
	}
}
// Obserwowane zdarzenie
class Debug implements Subject, ISingleton {
	protected $msg;
	private $observers;
 
	public function attach(Observer $o) {
		$this->observers[] = $o;
	}
 
	public function detach(Observer $o) {
		foreach($this->observers as &$observer) {
			if($observer === $o) {
				unset($observer);
			}
		}
	}
 
	public function notify() {
		if(count($this->observers)) {
			foreach($this->observers as $observer) {
				$observer->update($this);
			}
		}
	}
 
	public static function &singleton() {
		static $instance;
 
		if(!($instance instanceof Debug)) {
			$instance = new Debug();
		}
		return $instance;
	}
 
	public function getMsg() {
		return $this->msg;
	}
 
	public function info($msg) {
		$this->msg = $msg;		
		$this->notify();
	}
}
 
// ErrorHandler;)
final class ErrorHandler extends Debug {
 
	public function __construct() {
		error_reporting(E_ALL);
	}
 
	public static function &singleton() {
		static $instance;
 
		if(!($instance instanceof ErrorHandler)) {
			$instance = new ErrorHandler();
		}
 
		return $instance;
	}
 
	public function info($errno, $errstr, $errfile, $errline) {
		$this->msg = 'PhpError: ' . $errstr  . ' [<u>' . basename($errfile) . '</u> at line <u>' . $errline . '</u>]';		
		$this->notify();
	}
}
// Inicjacja wszystkiego
 
		$debug = Debug::singleton();
		$ehandle = ErrorHandler::singleton();
 
		set_error_handler(array($ehandle, 'info'));
 
		/* Debug print to output */			   
		$d = new DebugPrint();
		$d->register($debug);
		$d->register($ehandle);
 
		/* Debug logging into file observer */		
		$f = new FileLog();
		$f->register($debug);  
 
		$debug->info(&#092;"Application start\");
 
?>
[PHP] pobierz, plaintext 

Jak widac skorzystalem z wzorca Obserwator. To rozwiazanie jest teraz w miarę szybkie i dosyc skuteczne. Postaram sie zaimplementowac cos jeszcze.

Pozdrawiam,

// Edit

Hawk: Co do lazy loading, masz racje obserwatorzy czy tez exceptionhandler nie zawsze beda potrzebni, w takim razie mozna nieco zmodyfikowac cala idee by zamiast obiektow przekazywac tylko nazwy odpowiednich obserwatorow.

Ten post edytował serafin 14.07.2005, 08:32:03

@NuLL: Reflection API sprawdziłoby się raczej tylko w przypadku statycznych handlerów klas - tzn jesli masz handler np. XYZ::onDoSomething(). W tym wypadku moznaby albo za kazdym razem leciec reflectionem po klasachw wposzukiwaniu obsługi zdarzeń, albo (tylko raz) stworzyć mapę klas z danymi handlerami.

Ale co jeśli obiektów danego typu jest dużo? Na dodatek lokalnych w jakiejś funkcji? W tym wypadku chyba właśnie lepiej zastosować obserwatorów(chyba nawet trzeba) i po kolei ich rejestrować, tak jak zrobił serafin. Chyba nie ma innego wyjścia

Może się mylę, poprawcie jakby co

hymm..., ja myślę, żę obserwator nie jest tu niezbędny, ale bardzo ułatwia pracę. Choć sam debug zrobił bym inaczej (wyświetlanie komunikatów na końcu) i oddzielny obserwator dla wyjątków i błędów. Ogólnie myślę, że jest wiele sposobów prowadzenia debugera i jeżeli chodzi o uniwersalność "statyczne klasy", które mają metody i właściwości statyczne są lepsze i nie trzeba znać wzorca obserwatora. Aczkolwiek eventy mi się podobają, proste i w miarę skuteczne.

serafin: zastanawiam się po co te register() i unregister() w Obserwatorze. Chyba przejrzyściej by wyglądało, gdybyś to w obiekcie obserowanym registrował obserwator, a nie odwrotnie.

Vengeance: rejestrujesz w Obserwatorze Podmiot ktory ma obserwowac. Wiec logiczny jest taki kierunek rejestracji

A co logicznego jest w tym, że w Obserwatorze rejestruje Podmiot do obserwacji... z czego ten Obserwator tak na prawde potem rejestruje się w Podmiocie :]

Nie wydaje ci sie to drogą "na około" ?

Możesz wyciąć w tych metodach dodawanie do stosu podmiotów, choć ich rejestrowanie jest pomocne. A jak rejestruje się wiele podmiotów pod jednym obserwatorem to jest po prostu zbawienie.

Reflection API jest fantastyczne w przypadku wywalania na ekran informacji o błędzie. Ja np. wyciągam definicję każdej metody z backtrace, włącznie z wszystkimi przełącznikami (final, static, itd), z nazwami i typami argumentów itd. W wolnym czasie chciałem zrobić parsowanie phpdoc w celu wyciągnięcia dodatkowych informacji o argumentach i zwracanym typie (i oczywiście opisu). Jak zrobię, będę mógł zaprezentować phiend.logger . Chyba że ktoś chce się tym za mnie pobawić...

hawk: przejrzalem twojego phienda2. I'm quite impressed.

Bardzo spodobal mi sie plugin loader i sposob w jaki go zaimplementowales (zwlaszcza Handles i Lazy Loading)

P.S. Nie obrazisz sie jak skorzystam z kilku twoich pomyslow ;> ?

Czy czasem Lazy Loader nie odnosił się bardziej to logiki biznesowej niż zarządzaniem obiektami ?!

Lazy loading odnosi sie do zarzadzania obiektami. Obiekt tworzony jest wylacznie wtedy kiedy jest potrzebny, nie wczesniej. Podobnie dziala chyba mechanizm dynamicznego wiazania w javie, obiekt tworzony jest w momencie kiedy jego instancja jest potrzebna.

pozdrawiam,