php专区

推荐：《PHP视频教程》

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

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

$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
        // ...
    }
}

这不是 PHP 特有的情况，不过我经常碰到此类情况。你可以通过提前返回来减少缩进。

所有主要方法处于第一个缩进级别

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[&＃39;user&＃39;]) && isset($items[&＃39;user&＃39;][&＃39;id&＃39;]) {
    // process with $items[&＃39;user&＃39;][&＃39;id&＃39;]
}

我们经常需要检查变量是否已定义，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[&＃39;user&＃39;], $items[&＃39;user&＃39;][&＃39;id&＃39;])) {
    // process with $items[&＃39;user&＃39;][&＃39;id&＃39;]
}

四：echo和sprintf方法一起使用

$name = "John Doe";
echo sprintf(&＃39;Bonjour %s&＃39;, $name);

这段代码可能在微笑，但是我碰巧写了一段时间。而且我仍然看到很多！不用结合echo和sprintf，我们可以简单地使用printf方法。

$name = "John Doe";
printf(&＃39;Bonjour %s&＃39;, $name);

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

$items = [
    &＃39;one_key&＃39; => &＃39;John&＃39;,
    &＃39;search_key&＃39; => &＃39;Jane&＃39;,
];

if (in_array(&＃39;search_key&＃39;, array_keys($items))) {
    // process
}

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

$items = [
    &＃39;one_key&＃39; => &＃39;John&＃39;,
    &＃39;search_key&＃39; => &＃39;Jane&＃39;,
];

if (array_key_exists(&＃39;search_key&＃39;, $items)) {
    // process
}

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

if (isset($items[&＃39;search_key&＃39;])) {
    // process
}

原文地址：https://dev.to/klnjmm/5-bad-habits-to-lose-in-php-2j98

译文地址：https://learnku.com/php/t/49583

以上就是你可能要纠正这5个PHP编码小陋习！的详细内容，更多请关注 第一PHP社区 其它相关文章！