具体如下：

1、避免使用魔数

if($age<18){
}

这个18不太明白为什么要这样子。

可以将28定义在一个变量里面，这个变量命名表明了这个值的含义

$adult_age = 18;//成年的分界点年龄
if($age<$adult_age){
}

2、函数的返回结果：不要使用一个变量来存储返回结果

一旦你知道了返回结果，就应该马上返回。这样做的好处，是可以减少错误。

3、函数带有很多的参数。不要超过三个

如果参数很多，尽量聚合成一个model传递进去。比如一个数组，一个实例都可以。

太多的参数为什么影响方法的稳定性呢？

比如修改就会变得很麻烦。

我现在明白了。这个方法需要新加一个参数传递进去，那么原来调用这个方法的代码，就得跟着改变。

在做接口的时候，经常会遇到类似的问题。

function forgot($userName, $email, $email_url, $format = 'json')

上面是三个参数。

原来别人调用这个函数是，

forgot($userName, $email, $email_url, $format = 'json');

现在需求要变化了，需要新增加一个参数。怎么办?

原来的代码就要跟着修改。方法并不稳定。要么就重新开一个方法适应新的需求。要么就修改调用方的调用代码。

但是有种办可以避免这种问题

把原来传递参数做成数组形式，如下：

forgot($params=array(),$format = 'json');

聚合成一个数组。这样可以加任意多个参数。

如何理解：php引擎的内置函数，带有多个参数的情况呢？

设计缺陷？

4、方法的参数中含有布尔参数。

这意味着，这个方法不是完成一个单一的目标。违背了单一职责。增加了复杂性。