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

来源：cnblogs　　作者：八重樱　　时间：2020/11/9 16:11:44　　对本文有异议

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

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

$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);

看到这段代码你可能会想笑，不过我的确这样写了一段时间，而且我仍然会看到很多这样写的！其实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
}

原文链接：http://www.cnblogs.com/a609251438/p/13937721.html

友情链接：直通硅谷　点职佳　北美留学生论坛

PHP热门文章

收藏了8年的PHP优秀资源，都给你整理好了

php对微信支付回调处理的方法（合集）

聊聊TCP连接耗时的那些事儿

php如何比较两个浮点数是否相等详解

PHP实现PDO操作mysql存储过程示例

富文本编辑器新增导入word功能，自动转为html，可直接显示图片

curl put方式上传文件

thinkphp3.2模块名如何不区分大小写?

php去掉字符串的最后一个字符

WebSocket 长连接及超时问题解决

PHP转Go系列 | ThinkPHP与Gin框架之API接口签名设计实践

MagicArray:像php一样，让Go业务代码不再卷！

2023年的PHP项目部署笔记。什么？还有人用PHP？

[CTF/Web] PHP 反序列化学习笔记

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

将代码内容封装到一个 if 语句汇总

多次调用 isset 方法

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

PHP热门文章

PHP推荐文章