PHP - 检查返回false;

Question

只是一个简单的问题 - 而且我确信它非常基本!

我有以下代码:

function checkThings($foo, $bar) {
    ...
    if ($valid) {
        return $results;
    } else {
        return false;
    }
}

另一方面,我目前正在做

$check = checkThings($foo, $bar);
if ($check === false) {
    echo "Error";
} else {
    echo $check;
}

写下面的内容是一样的吗？

$check = checkThings($foo, $bar);
if (!$check) {
    echo "Error";
} else {
    echo $check;
}

如果两者都正确,哪种方法是首选方法？

谢谢 :)

Answer 1

三等运算符是类型敏感的.所以当你检查:

if ($check === false)

...只有$check布尔值为"false" 才会成立.Wheras

if ($check == false)

...不是专门检查布尔值false,而是"falsey"值.PHP中的False等于零,null为"falsey",为空字符串("" == false == null,但是"" !== false !== null).所以:

$check = 0;
if ($check == false)

......评估为真.

该!前缀运算符相当于==.因此,当零需要是来自布尔值false的离散值时,!和==运算符是不够的.

最佳做法是使您的条件检查尽可能具体.这不仅是为了避免您的逻辑中的潜在疏忽,而且还为未来的开发人员提供更易于维护的代码.想象一下遇到一行这样的代码:

if (check_something($variable)) {
  // do stuff
}

我可以假设check_something函数返回一个布尔值为true.但是,除非我去挖开check_something功能,它可能还可以返回一个非空字符串,数字...谁知道!这样做要清楚得多:

if (check_something($variable) === true) {
  // do stuff
}

现在,仅仅通过观察,我知道该check_something函数应该返回一个真值.我可能不知道这个函数是做什么的,但至少它确切地说明它返回的是什么.您在其他地方看到的另一个常见示例:

if (!$_GET['value']) {
  // do something
}

这是一个宠儿.条件陈述应始终清楚地比较事物.所以,你想做:

if (array_key_exists('value', $_GET) !== false && $_GET['value'] === '1') {
  // do something
}

现在,您可以告诉我,我不仅要检查查询字符串参数是否存在,还要查看它是否等于特定值.

总之,单!前缀运算符和==运算符很少有用,并且总是不明确的.最好的方法是以一种文档本身的方式编写代码,或者将其翻译成人类语言来表达逻辑.使用一个直接比较!==或===在适当情况下,是一个好习惯,好做法,并最有可能产生在你的代码意想不到的效果.

顺便说一句，这是一个特定于`php`（可能还有其他松散类型的语言）的问题，因为它非常愿意将值从一种类型自动转换为另一种类型。在强类型语言（例如Java、C#、C++）中，大多数程序员更喜欢简洁的语法，例如`!value`（“非值”）而不是`value == false` 或`value === false`。在这些语言中，如果 `value` 不是布尔类型，编译器会报错，所以简洁的形式是安全的。`"not value"` 在逻辑和人类语言中都有明确的含义；只是`php` 的“松散”使得这种形式不好使用。 (3认同)

Answer 2

===错误检查类型,(!$ check)计算值,并且对于返回的值(例如null,''(空字符串)和零(0))将失败

===是最正确的