常见的5个PHP编码小陋习以及优化实例讲解
发布:smiling 来源: PHP粉丝网 添加日期:2022-04-14 09:27:28 浏览: 评论:0
在做过大量的代码审查后,我经常看到一些重复的错误,以下是纠正这些错误的方法。
在循环之前测试数组是否为空
- $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']
- }<br>
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
- }
Tags: PHP编码小陋习
- 上一篇:php中创建字符串的变量实例讲解
- 下一篇:最后一页
推荐文章
热门文章
最新评论文章
- 写给考虑创业的年轻程序员(10)
- PHP新手上路(一)(7)
- 惹恼程序员的十件事(5)
- PHP邮件发送例子,已测试成功(5)
- 致初学者:PHP比ASP优秀的七个理由(4)
- PHP会被淘汰吗?(4)
- PHP新手上路(四)(4)
- 如何去学习PHP?(2)
- 简单入门级php分页代码(2)
- php中邮箱email 电话等格式的验证(2)