[php5] Co wrzucić do klasy obsługi błędów? Opcje

[Joachim Peters]

Witam,

Napisałem sobie mini klasę do obsługi błędów, jednak nie wiem co do niej mogę jeszcze wrzucić. Myślałem, o zapisywaniu logów, ale czy warto?

[PHP] pobierz, plaintext 
<?php
class Error {
	 private $errors = array();
 
	public function add($type = null, $text = null) {
		if(empty($text)) {
			switch($type) {
				case 'select':
					$this->errors[] = 'Wystąpił błąd podczas pobierania danych z bazy!<br />';
				break;
				case 'insert':
					$this->errors[] = 'Wystąpił błąd podczas wysyłania danych do bazy!<br />';
				break;
				case 'update':
					$this->errors[] = 'Wystąpił błąd podczas aktualizowania danych w bazie!<br />';
				break;				
				case 'delete':
					$this->errors[] = 'Wystąpił błąd podczas usuwania danych w bazie!<br />';
				break;
				default:
					$this->errors[] = false;
			} 
		} else {
			$this->errors[] = $text . '<br />';			
		}
	}
 
	private function display() {
		// echo $this->errors;
	}
}
?>
[PHP] pobierz, plaintext 

Co jeszcze można do niej dodać?

Pozdrawiam

[Ludvik]

Ja bym zaczął od tego, co można z tej klasy usunąć - obrzydliwego switcha. A wystarczy zrobić to trochę bardziej obiektowo.

[PHP] pobierz, plaintext 
<?php
class Error {
	public function __construct($msg) {
		$this->msg = $msg;
	}
 
	public function __toString() {
		return $this->msg;
	}
 
	protected $msg = '';
}
 
class DBSelectError extends Error {
	public function __construct() {
		parent::__construct('Wystąpił błąd podczas pobierania danych z bazy!');
	}
}
 
class ErrorLogger {
	public function add(Error $e) {
		$this->errors[] = $e;
	}
 
	protected $errors;
}
?>
[PHP] pobierz, plaintext 

Proste i działa lepiej niż switch...

[UDAT]

Ja bym zaczął od tego, co można z tej klasy usunąć - obrzydliwego switcha. A wystarczy zrobić to trochę bardziej obiektowo.

[PHP] pobierz, plaintext 
<?php
class Error {
	public function __construct($msg) {
		$this->msg = $msg;
	}
 
	public function __toString() {
		return $this->msg;
	}
 
	protected $msg = '';
}
?>
[PHP] pobierz, plaintext 

A nie prościej użyć klasy Exception ( wbudowanej w PHP ) ?