因为 @公子 公子指出了我在一个 回复 中的错误，我才开始逐渐意识到自己已经开始养成了一些很不好的编程习惯，比如嵌套循环，比如 n 多个条件嵌套，导致 } 符号堆积，以至于代码可读性极差。之前只重视了功能的实现与否，没有忽视了代码的可读性和性能方面的体现。
这边罗列下我所知道的一些不好的编程习惯：
1.嵌套循环
2.过多的条件嵌套
3.用户提交数据不进行过滤
4.函数体过长
5.变量命名清晰
我相信有不少 phper 都读过 《php 最佳实践》，或许我们大家可以集思广益，总结一个《php“最差”实践》，这样也可以当做一个警示录，时刻提醒自己优化代码，希望 php 开发经验丰富的前辈们能不吝赐教。:)
回复内容： 因为 @公子 公子指出了我在一个 回复 中的错误，我才开始逐渐意识到自己已经开始养成了一些很不好的编程习惯，比如嵌套循环，比如 n 多个条件嵌套，导致 } 符号堆积，以至于代码可读性极差。之前只重视了功能的实现与否，没有忽视了代码的可读性和性能方面的体现。
这边罗列下我所知道的一些不好的编程习惯：
1.嵌套循环
2.过多的条件嵌套
3.用户提交数据不进行过滤
4.函数体过长
5.变量命名清晰
我相信有不少 phper 都读过 《php 最佳实践》，或许我们大家可以集思广益，总结一个《php“最差”实践》，这样也可以当做一个警示录，时刻提醒自己优化代码，希望 php 开发经验丰富的前辈们能不吝赐教。:)
1,拼凑sql,如下
$sql = insert into `#@__archives`(id,typeid,typeid2,sortrank,flag,ismake,channel,arcrank,click,money,title,shorttitle, color,writer,source,litpic,pubdate,senddate,mid,voteid,notpost,description,keywords,filename,dutyadmin,weight) values ('{$this->art['id']}','{$this->art['typeid']}','{$this->art['typeid2']}','{$this->art['sortrank']}','{$this->art['flag']}', '{$this->art['ismake']}','{$this->art['channelid']}','{$this->art['arcrank']}','{$this->art['click']}','{$this->art['money']}','{$this->art['title']}', '{$this->art['shorttitle']}','{$this->art['color']}','{$this->art['writer']}','{$this->art['source']}','{$this->art['litpic']}','{$this->art['pubdate']}', '{$this->art['senddate']}','{$this->art['adminid']}','{$this->art['voteid']}','{$this->art['notpost']}','{$this->art['description']}', '{$this->art['keywords']}','{$this->art['filename']}','{$this->art['adminid']}','{$this->art['weight']}');;
2, 迟迟不肯return,代码挤在一起
function readhosts(){ global $config; $hosts = array(); $fp = fopen(host_file, 'r'); if($fp){ while(!feof($fp)){ $line = trim(fgets($fp)); if($line!='' && substr($line, 0, 1)!='#'){ //将多个空格替换为一个，增加容错率 $line = preg_replace(/\s(?=\s)/,\\1,$line); @list($host,$user,$password,$time) = explode(' ', $line); $timeline = mktime((int)$time); if($host && $user && $password){ if($timeline $host, 'dbuser' => $user, 'dbpwd' => $password, 'dbname' => 'mysql', 'dbprefix' => 'dede_', 'dbcharset' => 'gbk', ); msg('服务器'.$host.'加入发布列队'); }else{ msg('服务器'.$host.'没有到发布的时间'); } } } } echo \r\n; } fclose($fp); return $hosts;}
3, 全局变量global, 以下的msg函数大量调用....
function sendtosite($site){ global $db,$log,$config; if($log->get_value($site['dbname'])==='finish') return; echo =====================\r\n; msg('开始连接到数据库 '.$site['dbname']); if($db->select_db($site['dbname'])){ msg('连接成功，收集站点信息'); if($db->hastable('archives')){ msg('找到文章数据表，开始发布'); }else{ msg('没有找到文章数据表，可能表前缀不正确，跳过这个站点'); return; } }else{ msg('连接失败，跳过这个站点'); return; }
4,超长嵌套
if(count($site_channel)>0) { $keylink = ''.gbktoutf8($site_name).''; msg('找到'.count($site_channel).'个文章栏目'); echo \r\n; foreach ($site_channel as $channel){ if($log->get_value($site['dbname'].'_'.$channel['id'])==='finish'){ msg($site['dbname'].'栏目id为'.$channel['id'].'今日已发布完成，跳过'); continue; } $sendnum = readintfromstr($config['send_num']); if($sendnum get_value($site['dbname'].'_'.$channel['id'])){ continue; } $keyword_title_num = readintfromstr($config['keyword_title_num']); if($keyword_title_num>$sendnum) $keyword_title_num = $sendnum; $keyword_titles = array(); for($i=0;$iget_value($site['dbname'].'_'.$channel['id']);$iart_add($art)){ $backupfile=dir_backup.substr($txtfile, strlen(dir_txtfile)); if(!file_exists(dirname($backupfile))) mkdirs(dirname($backupfile)); rename($txtfile, $backupfile); msg('添加成功，备份txt文件'); $log->add($site['dbname'].'_'.$channel['id'], $i); $log->save(); }else{ msg('文章添加失败'); } }else{ msg('向 ['.$channel['typename'].'] 中添加文章失败，跳过'); continue; } }else{ msg('没有找到可用的txt文件'); break; } echo \r\n; } $log->add($site['dbname'].'_'.$channel['id'], 'finish'); $log->save(); } $log->add($site['dbname'], 'finish'); $log->save(); }else{ msg('没有找到文章栏目，跳过这个站点'); echo \r\n; }
5, 过多使用@抑制错误,不使用异常处理, 类的方法没有明确声明访问权限.
class db { private $config = array( 'dbhost' => '', 'dbname' => '', 'dbuser' => '', 'dbpwd' => '', 'dbprefix' => '', 'dbcharset' => '' ); public $linkid = null; public $isconnect = false; function __construct($config) { $this->config = $config + $this->config; $this->connect(); } function connect(){ if($this->config['dbhost'] && $this->config['dbname']){ $this->linkid = @mysql_connect($this->config['dbhost'], $this->config['dbuser'], $this->config['dbpwd']); if(!$this->linkid){ //连接错误，直接退出 //echo mysql_error().\r\n; return false; } if (@mysql_select_db($this->config['dbname'])) { if($this->config['dbcharset'] != ''){ @mysql_query(set names '{$this->config['dbcharset']}', $this->linkid); } } $this->isconnect = true; } return $this->isconnect; }}
太多了..不好弄...
https://github.com/justjavac/php-best-practices-zh_cn
http://youngsterxyf.github.io/2013/06/01/php-best-practices/
http://segmentfault.com/a/1190000000443795
http://wulijun.github.io/php-the-right-way/
http://thisinterestsme.com/php-best-practises/
没仔细看，可能有重复。
常规的别人都会说，我来说点激进的吧
除了全局变量之外，全局function也不允许自己定义一个php文件里要么只有一个class声明，要么不存在class声明接上，必须用autoloader来加载类，而且规则越简单越好，psr标准最好不允许修改public static $var参数是对象时必须在签名中声明类型，声明的类型中没有的成员不允许用不允许给对象动态添加属性。唯一的例外是stdclass必须使用单入口pattern，而且web server配置的documentroot下只能有index.php一个php文件
$arr = array();for($i = 0; $i
避免上面这种写法，因为循环每一次都会重新计算一次数组长度，可以这样写：
$arr = array();$length = count($arr);for($i = 0; $i
之前翻译过一个针对新手的建议帖子 一共有三篇
其中前两篇有比较多的对于不好用法的分析和解决方案
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/
拿自己举例，一个实现合并二维数组相同 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.代码可读性较强
如果你用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; }}
某些字符串函数与数组函数搞不清哪个是needle哪个是haystack...
js里有一种嵌套回调，层级太深了，也是巨难理解的。不过写过js的人都习惯了……
你贴的代码，之所以难读，好像和缺少换行和注释也有关系。