我目前有两个类层次结构:一个用于处理错误,另一个用于记录错误。

我想知道

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

09-11 22:01