当前位置:首页 > PHP教程 > php应用 > 列表

常见的5个PHP编码小陋习以及优化实例讲解

发布:smiling 来源: PHP粉丝网  添加日期:2022-04-14 09:27:28 浏览: 评论:0 

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

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

  1. $items = []; 
  2. // ... 
  3. if (count($items) > 0) { 
  4.   foreach ($items as $item) { 
  5.     // process on $item ... 
  6.   } 

foreach 以及数组函数 (array_*) 可以处理空数组。

不需要先进行测试可减少一层缩进

  1. $items = []; 
  2. // ... 
  3. foreach ($items as $item) { 
  4.   // process on $item ... 

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

  1. function foo(User $user) { 
  2.   if (!$user->isDisabled()) { 
  3.     // ... 
  4.     // long process 
  5.     // ... 
  6.   } 

这不是 PHP 特有的情况,不过我经常碰到此类情况。你可以通过提前返回来减少缩进。

所有主要方法处于第一个缩进级别:

  1. function foo(User $user) { 
  2.   if ($user->isDisabled()) { 
  3.     return
  4.   } 
  5.  
  6.   // ... 
  7.   // 其他代码 
  8.   // ... 

多次调用 isset 方法

你可能遇到以下情况:

  1. $a = null; 
  2. $b = null; 
  3. $c = null; 
  4. // ... 
  5.  
  6. if (!isset($a) || !isset($b) || !isset($c)) { 
  7.   throw new Exception("undefined variable"); 
  8.  
  9. // 或者 
  10.  
  11. if (isset($a) && isset($b) && isset($c) { 
  12.   // process with $a, $b et $c 
  13.  
  14. // 或者 
  15.  
  16. $items = []; 
  17. //... 
  18. if (isset($items['user']) && isset($items['user']['id']) { 
  19.   // process with $items['user']['id'] 

我们经常需要检查变量是否已定义,php 提供了 isset 函数可以用于检测该变量,而且该函数可以一次接受多个参数,所以一下代码可能更好:

  1. $a = null; 
  2. $b = null; 
  3. $c = null; 
  4. // ... 
  5.  
  6. if (!isset($a$b$c)) { 
  7.   throw new Exception("undefined variable"); 
  8.  
  9. // 或者 
  10.  
  11. if (isset($a$b$c)) { 
  12.   // process with $a, $b et $c 
  13.  
  14. // 或者 
  15.  
  16. $items = []; 
  17. //... 
  18. if (isset($items['user'], $items['user']['id'])) { 
  19.   // process with $items['user']['id'] 
  20. }<br> 

echo 和 sprintf 方法一起使用

$name = "John Doe";

echo sprintf('Bonjour %s', $name);

看到这段代码你可能会想笑,不过我的确这样写了一段时间,而且我仍然会看到很多这样写的!其实 echo 和 sprintf 并不需同时使用,printf 就可以完全实现打印功能。

$name = "John Doe";

printf('Bonjour %s', $name);

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

  1. $items = [ 
  2.   'one_key' => 'John'
  3.   'search_key' => 'Jane'
  4. ]; 
  5.  
  6. if (in_array('search_key'array_keys($items))) { 
  7.   // process 

我经常看到的最后一个错误是 in_array 和 array_keys 的联合使用。所有这些都可以使用 array_key_exists 替换。

  1. $items = [ 
  2.   'one_key' => 'John'
  3.   'search_key' => 'Jane'
  4. ]; 
  5.  
  6. if (array_key_exists('search_key'$items)) { 
  7.   // process 

我们还可以使用 isset 来检查值是否不是 null。

  1. if (isset($items['search_key'])) { 
  2.   // process 
  3. }

Tags: PHP编码小陋习

分享到: