必须纠正得5个PHP编码小陋习

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

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

10年架构师领你架构-成长之路-（附面试题（含答案））

（腾讯T3-T4）打造互联网PHP架构师教程目录大全，只要你看完，薪资立马提升2倍（持续更新）

点击与我交流企鹅群.

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']
}

感谢大家一直来支持，这是我准备的1000粉丝福利

【1000粉丝福利】10年架构师分享PHP进阶架构资料，助力大家都能30K

点击与我交流企鹅群.

我们经常需要检查变量是否已定义，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
}

大厂2000道面试题（含答案）

PHP面试题汇总，看完这些面试题助力你面试成功，工资必有20-25K

点击与我交流企鹅群.

喜欢我的文章就关注我吧，持续更新中.....

以上内容希望帮助到大家，很多PHPer在进阶的时候总会遇到一些问题和瓶颈，业务代码写多了没有方向感，不知道该从那里入手去提升，对此我整理了一些资料，包括但不限于：分布式架构、高可扩展、高性能、高并发、服务器性能调优、TP6，laravel，YII2，Redis，Swoole、Swoft、Kafka、Mysql优化、shell脚本、Docker、微服务、Nginx等多个知识点高级进阶干货需要的可以免费分享给大家，需要的可以点击进入暗号：知乎。

转自链接：dev.to/klnjmm