E_NOTICE ?== E_DEBUG，避免 isset() 和 @ 使用更复杂的 error_handler

Question

有哪些更好的方法可以避免应用程序逻辑中出现过多的 isset()，并保留在需要时查看调试消息 (E_NOTICE) 的能力？

首先假设：E_NOTICE 不是错误，它是用词不当，实际上应该是 E_DEBUG。然而，虽然这对于未设置的变量来说是正确的（PHP 仍然是一种脚本语言），但某些文件系统函数等也会抛出它们。因此，最好在启用 E_NOTICE 的情况下开发。

然而，并非所有调试通知都是有用的，这就是为什么它是 的常见（不幸的）PHP 习惯用法在整个应用程序逻辑中引入 isset() 和 @。 isset/empty 当然有许多有效的用例，但总的来说，它似乎是语法盐，并且实际上会阻碍调试。

这就是为什么我目前使用 error_reporting 书签和愚蠢的开/关开关：

// javascript:(function(){document.cookie=(document.cookie.match(/error_reporting=1/)?'error_reporting=0':'error_reporting=1')})()

if (($_SERVER["REMOTE_ADDR"] == "127.0.0.1")
    and $_COOKIE["error_reporting"])
{
    error_reporting(E_ALL|E_STRICT);
}
else {/* less */}

但是，这仍然给我带来了一个问题，即一旦启用，就有太多的通知需要搜索。作为解决方法，我可以利用 @ 错误抑制运算符。与 isset() 不同，它不会完全终止调试选项，因为自定义错误处理程序仍然可以收到抑制的 E_NOTICE。因此，这可能有助于将预期的调试通知与潜在的问题分开。

但这同样不能令人满意。因此就有了这个问题。有谁使用或知道更复杂的 PHP 错误处理程序。我正在想象这样的事情：

• 输出未过滤的错误/警告/通知（使用 CSS 绝对定位？）
• 和 AJAX-whatnot 来允许客户端检查和抑制
• ，但也保存了预期的过滤列表和“”批准”通知或警告。

当然，某些框架必须已经有这样的用户错误处理程序。

• 基本上我对警告/通知管理感兴趣。
• 完全抑制 E_NOTICE 确实不是所希望的。
• E_NOTICES 需要。只是数量少一些而已。默认情况下突出显示我可能关心的内容，而不是预期的内容。
• 如果我在没有 ?order= 参数的情况下运行，则会出现预期的通知。这是预料之中的，我不需要多次告知。
• 然而，在完全调试模式下，我确实希望通过所述调试通知的存在（或更有趣的是不存在）来查看未定义变量的存在。 ->我认为这就是他们的目的。避免 isset 会带来语言隐式打印语句。
• 还要认识到这是关于适合普通 PHP 表单处理语义的用例，而不是必须严格的应用程序领域。

天哪，请有人帮忙重写这个。冗长的解释失败。

Answer 1

开发一个从不发出任何 E_NOTICE 的大型 PHP 应用程序是可能的。您所要做的就是避免所有可以发出通知的情况，其中绝大多数是未初始化的变量和不存在的数组键。不幸的是，这与您避免使用 isset() 以及扩展 array_key_exists() 的愿望相冲突，因为它们是为处理该确切问题而设计的。

充其量，您可以通过仔细构建框架来最大程度地减少它们的使用。这通常意味着（例如）一个输入层，它被告知期望什么 GET 变量以及默认缺失的变量。这样，特定于页面的代码将始终具有可供查看的值。总的来说，这是一项有价值的技术，可以应用于各种 API。但我质疑这是否应该成为一个高度优先的设计目标。

与其他一些语言不同，PHP 区分不存在的变量和通常包含“空”值（通常为 null）的变量。它可能是早期版本的设计工件，但它仍然存在，因此您无法真正避免它。

Answer 2

我仅对 $_GET 和 $_SERVER 变量使用 isset()，其中数据来自外部控制我的应用程序。当我没有时间编写适当的 OOP 解决方案来避免它时，我会在其他情况下使用它，但我确信在大多数（如果不是全部）地方都可以避免它。例如，最好使用类而不是关联数组，这样您就不需要检查数组键是否存在。

我的建议是：

• 避免使用@运算符。
• 使用Xdebug。首先，它打印关于每个通知/警告的易于阅读且易于注意到的消息，并且它在异常上打印非常有用的堆栈跟踪（您可以将其配置为打印出每个方法参数和每个局部变量（< code>xdebug.collect_params=4 和 xdebug.show_local_vars=on 配置参数），其次，它可以使用 xdebug.scream 禁用 @ 运算符。 =1 配置值。您也可以使用 Xdebug 进行分析和代码覆盖率分析，这是您的开发计算机上的必备工具
• 。我也在使用 FirePHP，因为它与 Firebug，并且能够将消息打印到 Firebug 控制台，因此它也可用于AJAX 调试。
• 通过自定义错误处理程序，您可以捕获并过滤任何错误并进行处理。警告，您可以将它们记录到文件中或使用 FirePHP 显示它们，或者您可以使用例如 jGrowl 或 Gritter 以便在网页上很好地显示它们。

我正在使用 PHP 手册 中示例的修改版本：

<?php
//error_reporting(0);
set_error_handler("errorHandler");

function errorHandler($errno, $errstr, $errfile, $errline)
{
    echo "errorHandler()<br />\n";

    // filter out getImageSize() function with non existent files (because I'am avoiding using file_exists(), which is a costly operation)
    if ( mb_stripos($errstr, 'getimagesize') !== false )
        return true;

    // filter out filesize() function with non existent files
    if ( mb_stripos($errstr, 'filesize') !== false )
        return true;

    // consoleWriter is my class which sends the messages with FirePHP
    if (class_exists('consoleWriter'))
        consoleWriter::debug(array('errno'=>$errno, 'errstr'=>$errstr, 'errfile'=>$errfile, 'errline'=>$errline, 'trace'=>debug_backtrace()), "errorHandler");

    switch ($errno) {
    case E_USER_ERROR:
        $out .= "<b>FATAL_ERROR</b> <i>$errno</i> $errstr<br />\n";
        $out .= "Fatal error on line $errline in file $errfile";
        echo "</script>$out";   // if we were in a script tag, then the print is not visible without this
        //writeErrorLog($out);

        echo "<pre>";
        var_export(debug_backtrace());
        echo "</pre>";

        exit(1);
        break;

    case E_USER_WARNING:
        $out .= "<b>WARNING</b> <i>$errno</i> $errstr<br />\n";
        $out .= "On line $errline in file $errfile<br />\n";
        break;

    case E_USER_NOTICE:
        $out .= "<b>NOTICE</b> <i>$errno</i> $errstr<br />\n";
        $out .= "On line $errline in file $errfile<br />\n";
        break;

    default:
        $out .= "<b>Unknown</b> <i>$errno</i> $errstr<br />\n";
        $out .= "On line $errline in file $errfile<br />\n";
        break;
    }

    if (!class_exists('consoleWriter'))
        echo $out;

    //writeErrorLog($out);
    //addJGrowlMessage($out);

    // Don't execute PHP internal error handler
    return true;
}

function testNotice($a)
{
    echo $a;
}
testNotice();

还有一个建议是不要使用仅 php 文件末尾的结束 ?> 标记，因为它可能会在禁用输出缓冲的配置上导致 headers already sent 错误默认情况下。

Answer 3

好吧，如果您等待 PHP 7，您将可以访问 空合并三元 运算符，除了拥有现有的最酷的操作员名称（我将我的下一个孩子命名为“Null Coalesce”）之外，它还可以让你做到这一点：

$var = $some_array[$some_value] ?? "default value";

它取代了无处不在的（而且丑陋的）

$var = isset( $some_array[$some_value] ) ? $some_array[$some_value] : "default_value";

Answer 4

尝试 xdebug - http://www.xdebug.org/docs/stack_trace

大量的 isset 检查不会伤害你，

事实上，它鼓励在使用变量之前声明变量

Answer 5

我认为遵循最佳实践并不是浪费时间。确实如此，通知不是错误，但通过正确的变量声明和验证，您的代码可能会更具可读性和安全性。
但编写一个用户定义的错误处理程序 使用 debug_backtrace 使用正则表达式对 E_NOTICE(8) 进行排序。

Answer 6

我也有类似的愿望。所以我开始使用自定义错误处理程序。

http://php.net/manual/en/function.set-error -handler.php

然后，您可以创建自己的过滤器/机制来显示/记录错误/通知。

干杯!

Answer 7

PHP 确实在这个问题上被破坏了，导致代码的可读性降低。 “null”的意思是“未定义”——很简单。

当我遇到这个导致代码不可读的问题时，我会这样做：

/**
 * Safely index a possibly incomplete array without a php "undefined index" warning.
 * @param <type> $array
 * @param <type> $index
 * @return <type> null, or the value in the index (possibly null)
 */
function safeindex($array, $index) {
  if (!is_array($array)) return null;
  return (isset($array[$index])) ? $array[$index] : null;
}

// this might generate a warning
$configMenus = $config['menus'];  

// WTF are you talking about!!  16 punctuation marks!!!
$configMenus = (isset($config['menus']) ? $config['menus'] : null;

// First-time awkward, but readible
$configMenus = safeindex($config, 'menus');

在此处交叉发布此答案。这对垃圾邮件检查有帮助吗？

Answer 8

我认为避免 isset() 的最佳方法是在使用变量之前定义它们。我不喜欢 isset() 并不是因为它丑陋，而是因为它促进了糟糕的编程实践。

至于错误处理本身，我将所有这些信息都放入服务器日志中。我还在命令行上使用 php -l 来预先对程序进行语法检查。默认情况下，我会为用户发送漂亮的消息。

您可以研究各种框架之一，看看它们是否适合您。我看过的大多数都具有错误处理例程，使事情比 PHP 提供的现成的更容易。

编辑：
@mario - 我对你的评论的回复太长了:-)。我不提倡定义类型或采用某种严格的格式，如 Java 或 C。我只是提倡在使用变量的上下文中声明变量。 （ $foo = null; 与将变量留空不同）。

我认为在很多情况下这更多是全局变量的问题，尤其是用于获取 GET 和 POST 数据的超级全局变量。我真的希望 PHP 能够放弃超级全局变量，转而使用一个用于获取输入数据的类。像这样的东西（超级简单，但是嘿，你想要一些具体的东西::)）

<?php
class PostData {
     private $data;

     public function __construct() {
          $this->data = $_POST;
          unset($_POST);
     }

     public function param($name, $value = null) {
          if( $value !== null ) {
               $this->data[$name] = $value;
          }

          if( isset( $this->data[$name] ) ) {
               return $this->data[$name];
          }
          return null;
      }
}
?>  

包含该类，然后你可以从 param() 方法获取和设置 POST 数据。这也是将验证合并到输入数据中的好方法。作为奖励，无需检查 isset() 的所有内容（它已经是了）。

Answer 9

现在这是一个过时的答案，但当时我最初使用了灵活的日志调度程序， https://github.com/ grosser/errorhandler （不完全是我正在寻找的 IIRC，但至少比完全抑制和部分抑制之间的交替更复杂一点。）

无论如何，我同时使用 $_GET->int["input"] 包装器适用于最常见的情况。这只是一个简单的 ArrayAccess 包装器，隐式捕获不存在的变量，从而可以更轻松地进行通知恢复。 （只是一个副产品。主要用于立即过滤。）

对于另一个项目，我什至使用预处理器宏 IFSET@($var)，以允许启用/禁用或日志重定向取决于构建参数。