我目前有两个类层次结构:一个用于处理错误,另一个用于记录错误。
我想知道
1)将错误处理委托(delegate)给与错误处理程序分开的类是否有利
和
2)我所拥有的是否是一个好的设计。
这是我所拥有的:
class ErrorHandler {
public __construct(ErrorLogger $ErrorLogger) {
...
$this->setErrorPageURL('/error.php');
}
public function setErrorPageURL($errorPageURL);
public function handle() {
...
$this->ErrorLogger->log($errorNumber, $errorMessage, $errorFile, $errorLine);
header("Location: $this->errorPageURL");
}
}
class ErrorLogger {
abstract protected function log($errorNumber, $errorMessage, $errorFile, $errorLine);
// Overridable.
protected function getExtraLogMessage() {
return null;
}
}
class EmailerErrorLogger extends ErrorLogger {
public function __construct($toEmailAddress, $fromEmailAddress) {
...
}
protected function log($errorNumber, $errorMessage, $errorFile, $errorLine) {
// collect error information and various pieces of information
mail(...);
}
}
register_shutdown_function(array(new ErrorHandler(new EmailerErrorLogger()), 'handle'));
最佳答案
我认为这种结构是有道理的,因为您的错误处理程序除了记录错误外还必须执行某些其他工作。
记录器本身可以实现chain of responsibility pattern