嵌套太多

                         
if
(!
empty
(
$data
) { 
    
if
(
is_array
(
$data
)) { 
        
foreach
(
$data
as
$item
) { 
            
// Do something
        
}
        
return
$data
;
    
}
} 
else
{ 
    
return
null;
}
        
      

这里的问题就是代码嵌套太多层了，如果foreach里面有很多东西的话更难看

改成：

                         
if
(
empty
(
$data
) || !
is_array
(
$data
)) { 
    
return
null;
}
foreach
(
$data
as
$item
) { 
    
//Do something
}
return
$data
;
        
      

其实上面一个代码一定是最符合人类思维才会这样写的，而且不得不承认很容易会写出这样的代码

后面的那个代码把阅读顺序稍微修改了下，不大影响阅读，也能优化代码，不错的。

函数返回数据的验证

                         
if
(
$ver
>= 10901601) { 
    
$suggestIds
= SuggestModel::getSuggestIds(
$word
);
    
if
(!
empty
(
$suggestIds
)) { 
        
$this
->doAction(
$suggestIds
);
    
} 
else
{ 
        
$searchIds
= 
$this
->getSearchIds(
$word
);
        
if
(!
empty
(
$searchIds
)) { 
            
$this
->doAction(
$searchIds
);
        
}
    
}
}
        
      

上面一段代码，当去Suggest中获取数据失败的时候，就去Search中获取ID，但最后实际都是做一样的操作doAction

那么可以优化：

                         
if
(
$ver
< 10901601) { 
    
return
;
}
$ids
= 
array
();
$ids
= SuggestModel::getSuggestIds(
$word
);
if
(
empty
(
$ids
)) { 
    
$ids
= 
$this
->getSearchIds(
$word
);
}
$this
->doAction(
$ids
);
        
      

首先由于php是若类型的，所以需要养成的好习惯：

1 一个封闭函数应该返回什么类型的数据就返回什么类型的数据

比如这里的getSearchIds的函数，如果内部原先是期望返回array的，那么如果里面的数据为空的话，请直接返回array()，不要返回null或者false。

2 一个封闭的函数最开头需要做预防性的行为

比如这里的doAction($ids) 最先需要对ids如果为空数据的话如何处理

这里的getSearchIds和doAction应该是这样的：

                         
function
getSearchIds(
$word
) { 
    
if
(
empty
(
$word
)) { 
        
return
array
();
    
}
    
//DoSomething
    
return
$ids
;
}
funciton doAction(
$ids
) { 
    
if
(
empty
(
$ids
)) { 
        
return
false;
    
}
    
// DoSomeThing
    
foreach
(
$ids
as
$id
) { 
        
// Do SomeThing
    
}
    
return
true;
}
        
      

可以看到，这样做如果ids是array(), 会被过滤掉，即使没有过滤，foreach也不会发出warnging。

返回格式映射

php的controller中做的最多的是什么，一定是会说是返回格式的映射

一个controller中的逻辑大概是：

step1: 获取参数

step2: 根据参数获取数据

step3: 将数据重新组装返回

如果你的php是一个接口，那么这个第三步的东西估计是天天需要做的

所以治本的方法就该是学习rest风格，先定义好一种通用的数据，然后做个通用的映射逻辑（这个是可以做的，因为数据库的格式其实是固定的）

通用代码

通用代码是经常遇到的一个问题，显而易见，如果一份代码经常使用，那么就应该把它封装起来，然后做成一个通用的。

但是呢，我的观点是不能“滥通用”。意思就是不能每每看一个东西就认为是通用的。

比如有个观点“所有配置文件都应该放在统一的地方”。

这个我认为是不必要的，只有使用一次的配置数据就应该放在使用的地方，这样能保证代码的可读性

比如这里的：

                         
function
getManualAppsIds()
{ 
    
$config
= 
'/data/conf/test.json'
;
    
$content
= 
file_get_contents
(
$config
);
    
$manuals
= json_decode(
$content
, true);
    
return
manuals;
}
        
      

config这个变量，有人就会习惯放在其他代码里面，或者通用。但是实际上，这个几乎是永远不会再用，如果把它放到一个所谓“通用”的地方，读代码的话会出现阅读障碍。

反而是不美了。

本文转自轩脉刃博客园博客，原文链接：http://www.cnblogs.com/yjf512/p/3446368.html，如需转载请自行联系原作者