isset($var) 与 @$var

Question

这是使用 PHP 错误抑制的好做法还是可接受的方法？

if (isset($_REQUEST['id']) && $_REQUEST['id'] == 6) {
  echo 'hi';
}

if (@$_REQUEST['id'] == 6) {
  echo 'hi';
}

编辑：
我也是这么想的。 代码（和想法）来自朋友。
谢谢你证明我是对的。 :)

Answer 1

使用@ 抑制错误只会抑制错误的显示，而不是创建。 因此，如果您不先检查isset()，则会从错误中获得很小的性能损失。

Answer 2

使用错误抑制并不是一个很好的做法。 使用 $_REQUEST 甚至都不是一个好习惯。 只需使用 isset() 或 !empty() 或其他什么，不要偷懒。

还有一件事，在使用 isset() 时关闭括号是一个“好习惯”:)

Answer 3

不，在我看来，这并不是一种真正可以接受的做法。 除了看起来很草率之外，即使使用错误抑制，自定义错误处理程序仍然会被触发。

该手册提供了更多完全避免使用它的理由：

目前，“@”错误控制运算符前缀甚至会禁用将终止脚本执行的严重错误的错误报告。 除此之外，这意味着如果您使用“@”来抑制某个函数的错误，并且它不可用或输入错误，则脚本将立即死在那里，没有任何迹象表明原因。

Answer 4

我总是使用 isset() 因为它更具体。 另外，我会使用更具体的超全局变量，所以使用 $_POST、$_GET、$_SESSION。 清楚你的代码可以避免以后的麻烦:)

这是我如何运行我的检查：

if(isset($_POST['id']) && $_POST['id'] == '6')
{
     // do stuff
}

这是非常彻底的检查，因为它会检查帖子是否存在，然后我的变量是否是帖子的一部分，最后如果这两个通过，它会检查我的变量是否等于 6。

Answer 5

除了不是一个好的做法，因为@可以在调用堆栈中咀嚼真正重要的错误，性能损失是微不足道的。

让我们用一个基准来验证这一点。

<?php
error_reporting(-1);

$limit = 10000;

$start = microtime(true);
for ($i = 0; $i < 10000; $i++) {
    echo !isset($_GET['aaa']) ? '' : $_GET['aaa'];
}
$total = 1000000 * (microtime(true) - $start)/$limit;
echo "With isset: $total μs\n";

$start = microtime(true);
for ($i = 0; $i < 10000; $i++) {
    echo @$_GET['aaa'];
}
$total = 1000000 * (microtime(true) - $start)/$limit;
echo "With @: $total μs\n";

在我最近的电脑上，它输出：

With isset: 0.295 μs
With @: 0.657 μs

μs 是百万分之一秒。 这两种方法都需要接近百万分之一秒的一半。

可以说，但是如果我这样做成百上千次，会有什么不同吗？ 如果您必须执行!isset()一百万次，那么您的程序已经花费了大约 0.3 秒的时间！ 这意味着您一开始就不应该这样做。

然而，对于比简单数组更复杂的任何事物， @是一种不好的做法，因此即使您知道性能差异微不足道，也不要使用它。