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.避免同时关联三张以上的数据表,连表过多影响效率