php 代码审查中的问题总结
1.IF条件删减
1 if (isset($searchDate) && true !== checkNull($searchDate)) { 2 $flag = checkDates($searchDate); 3 if (!$flag) { 4 return wrongBack(lang('Time format error')); 5 } else { 6 $beginTimeStr = strtotime($searchDate); 7 $endTimeStr = $beginTimeStr + 86400; 8 $endTime = date('Y-m-d', $endTimeStr); 9 $where['pod.BEGIN_TIME'] = array('between', "$searchDate,$endTime"); 10 } 11 }
if条件可以省略,isset可以直接判断null 且可连续写入参数,精简代码如上 可优化 为 if(isset($searchData,$searchDate))
2.禁止循环中嵌套查询,避免数据库多次扫表减少查询速度减少服务器压力
3.避免同时关联三张以上的数据表,连表过多影响效率