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

PHP

转载自 PHP 论坛:https://learnku.com/php/t/49583

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

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


$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

}
讨论请前往专业的 PHP 论坛:https://learnku.com/php/t/49583
阅读 535

推荐阅读
PHP / Laravel / 全栈
用户专栏

PHP 和 Laravel 的精华文章分享。

4338 人关注
188 篇文章
专栏主页