请纠正这 5 个 PHP 编码小陋习

作者: wxfeng 分类: php 发布时间: 2022-01-05 10:58    阅读 31 次

在做过大量的代码审查后,我经常看到一些重复的错误,以下是纠正这些错误的方法。

在循环之前测试数组是否为空

$items = [];
// …
if (count($items) > 0) {
    foreach ($items as $item) {
        // process on $item …
    }
}

foreach 以及数组函数 (array_*) 可以处理空数组。不需要先进行测试,可减少一层缩进。

$items = [];
// …
foreach ($items as $item) {
    // process on $item …
}

提前返回来减少缩进

所有主要方法处于第一个缩进级别,将代码内容封装到一个 if 语句汇总

function foo(User $user) {
    if (!$user->isDisabled()) {
        // …
        // long process
        // …
    }
}
//改进后
function foo(User $user) {
    if ($user->isDisabled()) {
        return;
    }
    // ...
    // 其他代码
    // ...
}

多次调用 isset 方法
你可能遇到以下情况:

$a = null;
$b = null;
$c = null;
// …
if (!isset($a) || !isset($b) || !isset($c)) {
    throw new Exception("undefined variable");
}
// 或者
if (isset($a) && isset($b) && isset($c) {
    // process with $a, $b et $c
}
// 或者
$items = [];
//…
if (isset($items['user']) && isset($items['user']['id']) {
    // process with $items['user']['id']
}

我们经常需要检查变量是否已定义,php 提供了 isset 函数可以用于检测该变量,而且该函数可以一次接受多个参数,所以以下代码可能更好:

$a = null;
$b = null;
$c = null;
// …
if (!isset($a, $b, $c)) {
    throw new Exception("undefined variable");
}
// 或者
if (isset($a, $b, $c)) {
    // process with $a, $b et $c
}
// 或者
$items = [];
//…
if (isset($items['user'], $items['user']['id'])) {
    // process with $items['user']['id']
}

echo 和 sprintf 方法一起使用

$name = "John Doe";
echo sprintf('Bonjour %s', $name);

看到这段代码你可能会想笑,不过我的确这样写了一段时间,而且我仍然会看到很多这样写的!其实 echo 和 sprintf 并不需同时使用,printf 就可以完全实现打印功能。

$name = "John Doe";
printf('Bonjour %s', $name);

通过组合两种方法检查数组中是否存在键

$items = [
'one_key' => 'John',
'search_key' => 'Jane',
];
if (in_array('search_key', array_keys($items))) {
// process
}

我经常看到的最后一个错误是 in_array 和 array_keys 的联合使用。所有这些都可以使用 array_key_exists 替换。

$items = [
'one_key' => 'John',
'search_key' => 'Jane',
];
if (array_key_exists('search_key', $items)) {
// process
}

我们还可以使用 isset 来检查值是否不是 null。

if (isset($items['search_key'])) {
// process
}

————————————————
原文作者:Summer
转自链接:https://learnku.com/php/t/49583
版权声明:著作权归作者所有。商业转载请联系作者获得授权,非商业转载请保留以上作者信息和原文链接。

如果觉得我的文章对您有用,请随意赞赏。您的支持将鼓励我继续创作!

发表评论

您的电子邮箱地址不会被公开。 必填项已用*标注