推荐：《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);这段代码可能在微笑，但是我碰巧写了一段时间。而且我仍然看到很多！不用结合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}原文地址：https://dev.to/klnjmm/5-bad-habits-to-lose-in-php-2j98
译文地址：https://learnku.com/php/t/49583