做PHP开发已经有快一年的时间了，在这一年的时间中，学习了很多生产环境中的技巧，学习了很多东西，期间也阅读了一些优秀的源码和关于代码的书，对写代码这一块有了一定的思考，也看过很多别人写的好的代码和坏的代码，这里说说自己的感悟和改进吧。
本篇博客直说自己的感悟，在写代码时，我给自己立下的规则，这样可以让代码清晰可读并少走一些坑。这些简单的规则虽然没有设计模式看起来那么激动人心，但是，平常注意可以让代码看起来很清爽。
1. 不要在对象外使用未声明的变量
这个问题其实表述起来可能不容易理解。这个问题是因为PHP语言本身的特点决定的。由于PHP是一个弱类型的动态脚本语言，所以很多情况下，给了这个语言本省很宽松的条件让开发者去编写代码。但是往往这些便利也会变为坑，所以在使用一些动态语言很方便的写法的时候，尤其要注意。
下面我们先声明一个类，暂且叫这个类为用户类，这个User类的背景设定为，框架自带，不允许修改，并且隐藏在框架深处，不容易发现，实际案例可以参考laravel框架的Request类，代码如下：
class User { public $username; public $password; public $otherInfo = []; public function readUserInfo() { return [ 'username' => $this->username, 'password' => $this->password, ]; } public function addAtri($info) { array_push($this->otherInfo, $info); } }
这样的代码看似中规中矩，但是接下来，我们需要对这个类进行操作：
$user = new User(); $user->userRealName = "hello world";
这样的代码在PHP中是完全可以运行的，并且不会报错，但是这样的代码会对之后的一些事情做为干扰。我们现在假定，上边的代码是在PHP web项目中是一个拦截器，或者叫做中间件也可以，然后我们在controller中会会使用到这个类的实例，并且使用到这个中间件中添加的这个变量，如下：
class WebOperate { public function doOprate(User $user) { $user->userRealName = "hello world"; next($user); } }
这里设定的场景是，WebOperate是一个中间件，所有的Controller都会走这个中间件后到达Controller，之后，在处理相应的Controller的功能，接下来，Controller会将中间件的实例注入进来，供控制器使用，而中间件开发人员不是很在意其的存在：
class IndexController { public function index(User $user) { return $user->userRealName; } }
而这样的代码是可以完美运行的，接下来，开发人员想要的实另一个User类，这个User类中添加一些其他功能，正如之前所说，这个类在框架深处并且很难找到，且不允许修改，因为其他功能使用了这个类，所以，我们只有继承并添加方法。根据开发经验，开发人员会认为User类中存在这个userRealName变量，所以就造成了这个写法：
首先是基于这个User衍生出来的Teacher类：
class Teacher extends User { public function sayHello() { return "hello world"; } }
这样，我们的Teacher就可以sayhello了，但是，这个时候，在我们的Controller中还想知道老师的真实姓名，怎么办？根据经验，我们可以将注入的类换成Teacher并且返回真实姓名：
class IndexController { public function index(Teacher $user) { return $user->userRealName; } }
那么这下问题来了，其实User类中并没有这个类，所以这个变量根本没有数值，但是根据经验，是中间件已经赋值过一次了，所以我们应该可以直接使用，但是并没有这个数值，我们开始看源码发现，继承的User类中根本不存在这个变量，那么这个变量之前为什么可以使用呢，因为在中间件中，给User的实力付了值。
所以我们的不能这样直接使用未声明的变量，在一个类中。
我们应该这样写：
class WebOperate { public function doOprate(User $user) { $user->addAtri([ 'userRealName' => 'hello world', ]); next($user); } }
这样的中间件，在调用的时候继承类也可以使用同样的方法，很简单并且很不容易出现坏的味道。
2. 类or数组
其实这个问题同时也衍生出了另外的问题，就是函数返回值的问题。
首先，我明确表示，一个函数做多种类型的返回值是我个人感觉是不好的，在动态语言中虽然很常见，很多PHP的原生方法也有这样的，但是，在生产中使用这样的方式会造成函数返回的不确定性，我们需要作出很多判断来证明我们的结论，但是，如果返回值类型只有一种，我们就可以直接判断返回值就好了。
就像如下代码：
public function addNewUser() { $res = $this->addData(); if ($res) { return true; } else { return [ 'error' => 1, 'errormsg' => "没有添加成功" ]; } }
这样的代码在作为调用者往往会多一次判断，如下：