做PHP開發(fā)已經(jīng)有快一年的時(shí)間了，在這一年的時(shí)間中，學(xué)習(xí)了很多生產(chǎn)環(huán)境中的技巧，學(xué)習(xí)了很多東西，期間也閱讀了一些優(yōu)秀的源碼和關(guān)于代碼的書，對寫代碼這一塊有了一定的思考，也看過很多別人寫的好的代碼和壞的代碼，這里說說自己的感悟和改進(jìn)吧。

本篇博客直說自己的感悟，在寫代碼時(shí)，我給自己立下的規(guī)則，這樣可以讓代碼清晰可讀并少走一些坑。這些簡單的規(guī)則雖然沒有設(shè)計(jì)模式看起來那么激動人心，但是，平常注意可以讓代碼看起來很清爽。

1. 不要在對象外使用未聲明的變量

這個(gè)問題其實(shí)表述起來可能不容易理解。這個(gè)問題是因?yàn)镻HP語言本身的特點(diǎn)決定的。由于PHP是一個(gè)弱類型的動態(tài)腳本語言，所以很多情況下，給了這個(gè)語言本省很寬松的條件讓開發(fā)者去編寫代碼。但是往往這些便利也會變?yōu)榭樱栽谑褂靡恍﹦討B(tài)語言很方便的寫法的時(shí)候，尤其要注意。

下面我們先聲明一個(gè)類，暫且叫這個(gè)類為用戶類，這個(gè)User類的背景設(shè)定為，框架自帶，不允許修改，并且隱藏在框架深處，不容易發(fā)現(xiàn)，實(shí)際案例可以參考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);  }}

這樣的代碼看似中規(guī)中矩，但是接下來，我們需要對這個(gè)類進(jìn)行操作：

$user = new User();$user->userRealName = "hello world";

這樣的代碼在PHP中是完全可以運(yùn)行的，并且不會報(bào)錯(cuò)，但是這樣的代碼會對之后的一些事情做為干擾。我們現(xiàn)在假定，上邊的代碼是在PHP web項(xiàng)目中是一個(gè)攔截器，或者叫做中間件也可以，然后我們在controller中會會使用到這個(gè)類的實(shí)例，并且使用到這個(gè)中間件中添加的這個(gè)變量，如下：

 class WebOperate {   public function doOprate(User $user) {     $user->userRealName = "hello world";     next($user);   } }

這里設(shè)定的場景是，WebOperate是一個(gè)中間件，所有的Controller都會走這個(gè)中間件后到達(dá)Controller，之后，在處理相應(yīng)的Controller的功能，接下來，Controller會將中間件的實(shí)例注入進(jìn)來，供控制器使用，而中間件開發(fā)人員不是很在意其的存在：

 class IndexController {   public function index(User $user) {     return $user->userRealName;   } }

而這樣的代碼是可以完美運(yùn)行的，接下來，開發(fā)人員想要的實(shí)另一個(gè)User類，這個(gè)User類中添加一些其他功能，正如之前所說，這個(gè)類在框架深處并且很難找到，且不允許修改，因?yàn)槠渌δ苁褂昧诉@個(gè)類，所以，我們只有繼承并添加方法。根據(jù)開發(fā)經(jīng)驗(yàn)，開發(fā)人員會認(rèn)為User類中存在這個(gè)userRealName變量，所以就造成了這個(gè)寫法：

首先是基于這個(gè)User衍生出來的Teacher類：

 class Teacher extends User {   public function sayHello() {     return "hello world";   } }

這樣，我們的Teacher就可以sayhello了，但是，這個(gè)時(shí)候，在我們的Controller中還想知道老師的真實(shí)姓名，怎么辦？根據(jù)經(jīng)驗(yàn)，我們可以將注入的類換成Teacher并且返回真實(shí)姓名：

 class IndexController {   public function index(Teacher $user) {     return $user->userRealName;   } }

那么這下問題來了，其實(shí)User類中并沒有這個(gè)類，所以這個(gè)變量根本沒有數(shù)值，但是根據(jù)經(jīng)驗(yàn)，是中間件已經(jīng)賦值過一次了，所以我們應(yīng)該可以直接使用，但是并沒有這個(gè)數(shù)值，我們開始看源碼發(fā)現(xiàn)，繼承的User類中根本不存在這個(gè)變量，那么這個(gè)變量之前為什么可以使用呢，因?yàn)樵谥虚g件中，給User的實(shí)力付了值。

所以我們的不能這樣直接使用未聲明的變量，在一個(gè)類中。

我們應(yīng)該這樣寫：

class WebOperate {  public function doOprate(User $user) {    $user->addAtri([      'userRealName' => 'hello world',    ]);    next($user);  }}

這樣的中間件，在調(diào)用的時(shí)候繼承類也可以使用同樣的方法，很簡單并且很不容易出現(xiàn)壞的味道。

2. 類or數(shù)組

其實(shí)這個(gè)問題同時(shí)也衍生出了另外的問題，就是函數(shù)返回值的問題。

首先，我明確表示，一個(gè)函數(shù)做多種類型的返回值是我個(gè)人感覺是不好的，在動態(tài)語言中雖然很常見，很多PHP的原生方法也有這樣的，但是，在生產(chǎn)中使用這樣的方式會造成函數(shù)返回的不確定性，我們需要作出很多判斷來證明我們的結(jié)論，但是，如果返回值類型只有一種，我們就可以直接判斷返回值就好了。

就像如下代碼：

public function addNewUser() {    $res = $this->addData();    if ($res) {      return true;    } else {      return [        'error' => 1,        'errormsg' => "沒有添加成功"      ];    }  }

這樣的代碼在作為調(diào)用者往往會多一次判斷，如下：

public function index() {    $res = $this->addNewUser();    if (is_array($res) && isset($res['error'])) {      return isset($res['errormsg']) ? $res['errormsg'] : "未知錯(cuò)誤";    }    return "成功";  }

這樣的代碼幾乎每一次調(diào)用完成這個(gè)函數(shù)都會有這一套出現(xiàn)，不僅代碼不美觀，而且很臃腫。

這樣的代碼需要改善，首先限制住函數(shù)的返回值。比如，我們只讓這個(gè)函數(shù)返回bool類型的數(shù)：

public function addNewUser() {  $res = $this->addData();  if ($res) {    return true;  } else {    return false;  }}

但是，顯然，很多時(shí)候，我們要的不是簡單的真價(jià)值，所以，我們會選擇返回更多信息，這個(gè)時(shí)候，我們可以有三種處理方式。

1）返回int類型的數(shù)，然后通過這個(gè)int類型的數(shù)去判斷處理結(jié)果，我們可以添加上映射關(guān)系：

class Operate{  public $operateRes = [    0 => '成功',    1 => '添加失敗',    2 => '未知錯(cuò)誤',  ];      public function addNewUser() {    $res = $this->addData();    if ($res) {      return 0;    } else if ($res > 1) {      return 1;    }    return 2;  }  }

這樣方法的調(diào)用者就可以很簡單的使用方法并給出提示了：

$opera = new Operate();$res = $opera->addNewUser();return $opera->operateRes[$res];

給出統(tǒng)一的返回值類型的時(shí)候就完全不需要判斷返回值類型而且可以設(shè)置一個(gè)規(guī)范返回提示。

2）我們也可以使用數(shù)組

3）數(shù)組給人不缺定性，因?yàn)楹芏鄷r(shí)候，數(shù)組里可以認(rèn)為的少寫一些元素，如果少寫了，程序直接報(bào)錯(cuò)，很不好。

所以第三種方式就是建議將固定格式的返回，寫成一個(gè)類，做返回的時(shí)候，使用這個(gè)類：

class Operate{  public function addNewUser() {    $res = $this->addData();    $result = new Result();    if ($res) {      $result->errno = 0;      $result->errmsg = "成功";    } else if ($res > 1) {      $result->errno = 1;      $result->errmsg = "失敗";    }    $result->errno = 2;    $result->errmsg = "未知錯(cuò)誤";    return $result;  }  }class Result {  public $errno;  public $errmsg;}

這樣的返回，保證了所有變量的存在，同樣可以減少一次判斷。

所以，綜合以上，在我們返回結(jié)果的時(shí)候，盡量使用同種類型的變量，盡量減少使用數(shù)組返回。