请纠正这 5 个 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['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);

看到这段代码你可能会想笑,不过我的确这样写了一段时间,而且我仍然会看到很多这样写的!其实echosprintf 并不需同时使用,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_arrayarray_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
}
本文中的所有译文仅用于学习和交流目的,转载请务必注明文章译者、出处、和本文链接
我们的翻译工作遵照 CC 协议,如果我们的工作有侵犯到您的权益,请及时联系我们。

原文地址:https://dev.to/klnjmm/5-bad-habits-to-lo...

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

本帖已被设为精华帖!
本文为协同翻译文章,如您发现瑕疵请点击「改进」按钮提交优化建议
讨论数量: 7
Junwind

foreach循环的数组判断,更应该判断是否是数组和该数据是否存在

3年前 评论
Marrigan 3年前
陈十一 3年前
秦晓武 3年前
celaraze

isset()empty() 可以互换使用吗?一直习惯用 empty() 来判断是否空

3年前 评论
雪风 3年前
MuYan 3年前
xiaomifdsa 3年前
秦晓武 3年前
MuYan 3年前
秦晓武 3年前
Violet_Ice紫冰 3年前

一般foreach都要操作数组内元素的下标吧,这样如果foreach循环体内有这样的代码,在你foreach的数组出现为空的情况下,不就报未定义索引错误了吗,所以我觉得foreach之前判断是否为空,还是有一些存在的必要的

3年前 评论
秦晓武 3年前
秦晓武

“echo sprintf”,这个我用了一年。。。看了文章才知道

3年前 评论

@秦晓武 学过C语言,看到sprintf应该也能想到printf啊

3年前 评论
秦晓武 3年前
if (isset($items['search_key'])) {
    // process
}

我之前写的是 empyt($items['search_key']) 然后有个领导,非要让我用 isset 好吧,他说的算,谁让他是领导呢

3年前 评论
UKNOW

赞一个

2年前 评论

讨论应以学习和精进为目的。请勿发布不友善或者负能量的内容,与人为善,比聪明更重要!