因为 @公子 公子指出了我在一个 回复 中的错误，我才开始逐渐意识到自己已经开始养成了一些很不好的编程习惯，比如嵌套循环，比如 N 多个条件嵌套，导致 } 符号堆积，以至于代码可读性极差。之前只重视了功能的实现与否，没有忽视了代码的可读性和性能方面的体现。
这边罗列下我所知道的一些不好的编程习惯：
1.嵌套循环
2.过多的条件嵌套
3.用户提交数据不进行过滤
4.函数体过长
5.变量命名清晰
我相信有不少 PHPer 都读过 《PHP 最佳实践》，或许我们大家可以集思广益，总结一个《PHP“最差”实践》，这样也可以当做一个警示录，时刻提醒自己优化代码，希望 PHP 开发经验丰富的前辈们能不吝赐教。:)
常规的别人都会说，我来说点激进的吧
极客观点
项目管理
HarmonyOS
$arr = array();
for($i = 0; $i < count($arr);$i ++)
{
//code here.
}
避免上面这种写法，因为循环每一次都会重新计算一次数组长度，可以这样写：
$arr = array();
$length = count($arr);
for($i = 0; $i < $length;$i ++)
{
//code here.
}
拿自己举例，一个实现合并二维数组相同 key 的写法：
我的 ugly 写法：
$arr1 = array(
array('num'=>5,'period'=>3),
array('num'=>10,'period'=>3),
array('num'=>15,'period'=>9)
);
$arr2 = array();
foreach($arr1 as $k1 => $v1) {
if(empty($arr2)) {
$arr2[] = $v1;
} else {
foreach ($arr2 as &$v2) {
if($v1['period'] == $v2['period']) {
$v2['num'] += $v1['num'];
} else {
$arr2[] = $v1;
}
}
}
}
1.不必要的嵌套循环
2.变量命名不清晰
3.} 堆叠使得代码可读性极差
@公子 的规范写法：
rr = array(
array('num'=>5,'period'=>3),
array('num'=>10,'period'=>3),
array('num'=>15,'period'=>9)
);
$temp = array();
foreach($arr as $item) {
list($n, $p) = array_values($item);
$temp[$p] = array_key_exists($p, $temp) ? $temp[$p]+$n : $n;
}
$arr = array();
foreach($temp as $p => $n)
$arr[] = array('num'=>$n, 'period'=>$p);
print_r($arr);
1.拆开循环
2.利用 PHP 自带的数组函数简化了代码
3.代码可读性较强
之前翻译过一个针对新手的建议帖子 一共有三篇
其中前两篇有比较多的对于不好用法的分析和解决方案
http://ashitaka.me/%E8%AF%91/2014/01/05/40-php-tips-part-i/
http://ashitaka.me/%E8%AF%91/2014/01/06/40-php-tips-part-ii/
如果你用netbeans8.0的话,超过2还是3层的嵌套,全局变量的不经判断直接使用,一个方法写的太长(应该对功能拆分).等等,都会直接给出警告提示.一个好的IDE对提高自己的代码书写规范性和可阅读新还是很有帮助的.
慎用&
例子：
$arr = array('a'=>1, 'b'=>2);
foreach($arr as &$v)
{
if($v == '2')
{
$v = 3;
}
}
$v = 444;
打印$arr['b'] 的值为 444;
可以改成如下：
$arr = array('a'=>1, 'b'=>2);
foreach($arr as $k => $v)
{
if($v == '2')
{
$arr[$k] = 3;
}
}2 回答3.1k 阅读✓ 已解决
1 回答1.4k 阅读✓ 已解决
1 回答1k 阅读✓ 已解决
1 回答1.3k 阅读✓ 已解决
2 回答1.2k 阅读
1 回答1.2k 阅读
1 回答1k 阅读
1,拼凑SQL,如下
2, 迟迟不肯return,代码挤在一起
3, 全局变量global, 以下的
msg函数大量调用....4,超长嵌套
5, 过多使用
@抑制错误,不使用异常处理, 类的方法没有明确声明访问权限.太多了..不好弄...