10个PHP开发者最容易犯的错误

最新推荐文章于 2019-07-04 22:18:42 发布

转载最新推荐文章于 2019-07-04 22:18:42 发布 · 148 阅读

php 专栏收录该内容

46 篇文章

订阅专栏

本文探讨了PHP中isset()函数的常见误用及其在变量存在性和值判断上的局限性，同时介绍了array_key_exists()作为替代方案的优势。此外，文章还分析了在循环中执行查询的效率问题，提出了优化查询策略，避免对数据库的重复请求。

常见错误 #2：误解 `isset()` 的行为

尽管名字叫 isset，但是 isset() 不仅会在变量不存在的时候返回 false，在变量值为 null 的时候也会返回 false。

这种行为比最初出现的问题更为棘手，同时也是一种常见的错误源。

看看下面的代码：

$data = fetchRecordFromStorage($storage, $identifier);
if (!isset($data['keyShouldBeSet']) {
    // do something here if 'keyShouldBeSet' is not set
}

开发者想必是想确认 keyShouldBeSet 是否存在于 $data 中。然而，正如上面说的，如果 $data['keyShouldBeSet'] 存在并且值为 null 的时候， isset($data['keyShouldBeSet']) 也会返回 false。所以上面的逻辑是不严谨的。

我们来看另外一个例子：

if ($_POST['active']) {
    $postData = extractSomething($_POST);
}
 
// ...
 
if (!isset($postData)) {
    echo 'post not active';
}

上述代码，通常认为，假如 $_POST['active'] 返回 true，那么 postData 必将存在，因此 isset($postData) 也将返回 true。反之， isset($postData) 返回 false 的唯一可能是 $_POST['active'] 也返回 false。

然而事实并非如此！

如我所言，如果$postData 存在且被设置为 null， isset($postData) 也会返回 false 。也就是说，即使 $_POST['active'] 返回 true， isset($postData) 也可能会返回 false 。再一次说明上面的逻辑不严谨。

顺便一提，如果上面代码的意图真的是再次确认 $_POST['active'] 是否返回 true，依赖 isset() 来做，不管对于哪种场景来说都是一种糟糕的决定。更好的做法是再次检查 $_POST['active']，即：

if ($_POST['active']) {
    $postData = extractSomething($_POST);
}
 
// ...
 
if ($_POST['active']) {
    echo 'post not active';
}

对于这种情况，虽然检查一个变量是否真的存在很重要（即：区分一个变量是未被设置还是被设置为 null）；但是使用 array_key_exists() 这个函数却是个更健壮的解决途径。

比如，我们可以像下面这样重写上面第一个例子：

$data = fetchRecordFromStorage($storage, $identifier);
if (! array_key_exists('keyShouldBeSet', $data)) {
    // do this if 'keyShouldBeSet' isn't set
}

另外，通过结合 array_key_exists() 和 get_defined_vars()，我们能更加可靠的判断一个变量在当前作用域中是否存在：

if (array_key_exists('varShouldBeSet', get_defined_vars())) {
    // variable $varShouldBeSet exists in current scope
}

PHP isset函数作用

参考链接：php isset()与empty()的使用

isset函数是检测变量是否设置。

格式：bool isset( mixed var [, mixed var [, ...]] )

返回值：

若变量不存在则返回FALSE

若变量存在且其值为NULL，也返回FALSE

若变量存在且值不为NULL，则返回TURE

同时检查多个变量时，每个单项都符号上一条要求时才返回TRUE,否则结果为FALSE

如果已经使用unset()释放了一个变量之后，它将不再是isset()。若使用isset()测试一个被设置成NULL的变量，将返回FALSE。同时要注意的是一个NULL字节（"\0"）并不等同于PHP的NULL常数。

警告：isset()只能用于变量，因为传递任何其它参数都将造成解析错误。若想检测常量是否已设置，可使用defined()函数。

$a = array ('test' => 1, 'hello' => NULL);
var_dump( isset ($a['test') ); // TRUE
var_dump( isset ($a['foo') ); // FALSE
var_dump( isset ($a['hello') ); // FALSE 
// 'hello' 等于 NULL，所以被认为是未赋值的。
// 如果想检测 NULL 键值，可以试试下边的方法。
var_dump( array_key_exists('hello', $a) ); // TRUE

常见的错误 #4：在循环中执行查询

如果像这样的话，一定不难见到你的 PHP 无法正常工作。

$models = [];
 
foreach ($inputValues as $inputValue) {
    $models[] = $valueRepository->findByValue($inputValue);
}

这里也许没有真正的错误，但是如果你跟随着代码的逻辑走下去，你也许会发现这个看似无害的调用$valueRepository->findByValue() 最终执行了这样一种查询，例如：

$result = $connection->query("SELECT `x`,`y` FROM `values` WHERE `value`=" . $inputValue);

结果每轮循环都会产生一次对数据库的查询。因此，假如你为这个循环提供了一个包含 1000 个值的数组，它会对资源产生 1000 单独的请求！如果这样的脚本在多个线程中被调用，他会有导致系统崩溃的潜在危险。

因此，至关重要的是，当你的代码要进行查询时，应该尽可能的收集需要用到的值，然后在一个查询中获取所有结果。

一个我们平时常常能见到查询效率低下的地方（例如：在循环中）是使用一个数组中的值 (比如说很多的 ID )向表发起请求。检索每一个 ID 的所有的数据，代码将会迭代这个数组，每个 ID 进行一次SQL查询请求，它看起来常常是这样：

$data = [];
foreach ($ids as $id) {
    $result = $connection->query("SELECT `x`, `y` FROM `values` WHERE `id` = " . $id);
    $data[] = $result->fetch_row();
}

但是 只用一条 SQL 查询语句就可以更高效的完成相同的工作，比如像下面这样：

$data = [];
if (count($ids)) {
    $result = $connection->query("SELECT `x`, `y` FROM `values` WHERE `id` IN (" . implode(',', $ids));
    while ($row = $result->fetch_row()) {
        $data[] = $row;
    }
}

因此在你的代码直接或间接进行查询请求时，一定要认出这种查询。尽可能的通过一次查询得到想要的结果。然而，依然要小心谨慎，不然就可能会出现下面我们要讲的另一个易犯的错误...

原文链接：https://blog.youkuaiyun.com/luyaran/article/details/79785688