将错误转换为异常：设计缺陷？

Question

我最近遇到了一些代码，它们使用自定义错误处理程序将任何 PHP 错误转换为通用应用程序异常。还定义了一个自定义异常处理程序，如果它在特定的错误代码范围内，它将记录异常。示例：

class AppException extends Exception
{

}

function error_handler($errno, $errstr, $errfile, $errline)
{
    throw new AppException($errstr, $errno);
}

function exception_handler($exception)
{
    $min = ...;
    $max = ...;

    if ($exception->getCode() >= $min && $exception->getCode() <= $max)
    {
        // log exception
    }
}

set_error_handler('error_handler');
set_exception_handler('exception_handler');

$a[1]; // throws exception

问题是我看到了类似的东西：

try
{
    do_something();
}
catch (AppException $exception)
{
}

这意味着实际编程错误和“异常”行为之间没有区别。在进一步挖掘后，我发现部分代码是围绕 PHP 错误代表“异常”行为的想法而设计的，例如：

...

function my_function($param1, $param2)
{
    // do something great
}

try
{
    my_function('only_one_param');
}
catch (AppException $exception)
{
}

这最终会混淆错误和应用程序界面的设计。

您对以这种方式处理错误有何看法？是否值得将 PHP 的原生错误转化为异常？在上述情况下，围绕这个想法设计代码库时，你会怎么做？

Answer 1

就个人而言，我一直这样做。唯一的区别是，在我的error_handler 函数中，我首先检查错误是否为E_NOTICE，如果不是则仅抛出（无论如何我都会记录通知）...

我会将 AppException 更改为扩展 ErrorException... 之类的东西：PhpRuntimeErrorException extends ErrorException 仅用于 PHP 错误...原因是它更具可读性（更容易说出什么PhpRuntimeErrorException 不需要弄清楚它被抛出的位置）。另一个原因是ErrorException 将存储生成行/文件/等信息，它不会存储在其他地方（因为回溯从throw 行开始）...

那么，你可以“尝试”这样的代码：

try {
    $f = fopen('foo.bar', 'r');
    $ret = '';
    while ($data = fread($f)) {
        $ret .= process($data);
    }
    fclose($f);
    return '';
} catch (PHPRuntimeErrorException $e) {
    throw new RuntimeException('Could not open file');
} catch (ProcessException $e) {
    fclose($f);
    throw new RuntimeException('Could not process data');
}
return $ret;

我还让我的默认异常处理程序生成一个 500 服务器错误页面。那是因为任何异常都应该被捕获，如果没有，那就真的是服务器错误......

只是我的经验和意见...

Answer 2

PHP 社区对此进行了一些争论。我相信一般的想法是错误是内部 PHP 函数抛出的东西，是你真正需要修复的编码问题；而异常是您的应用程序代码抛出的东西，您可能只需要“处理”。

但是，一些较新的 SPL 扩展（往往更面向对象）确实使用异常来处理以前可能引发错误的事情，因此存在一些灰色区域。

还有一个 PHP 核心类 ErrorException - http://www.php.net/manual/en/class.errorexception.php 看起来比你的代码示例更容易使用，如果这是你想走的路线。

Answer 3

我将所有内容都转换为异常，包括通知。没有理由允许异常继续正常进行。未定义的变量或偏移量？这是个问题。

https://github.com/KyleWolfe/PHPErrorNet