欧美bbbwbbbw肥妇,免费乱码人妻系列日韩,一级黄片

如何避免PHP實(shí)例代碼中的一些壞代碼

 更新時(shí)間:2018年07月08日 16:29:14   作者:風(fēng)為南_  
本篇文章給大家分享了如何在PHP實(shí)例代碼中發(fā)現(xiàn)壞代碼以及如何修復(fù)的問(wèn)題,有興趣的朋友參考下。

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

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

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

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

下面我們先聲明一個(gè)類(lèi),暫且叫這個(gè)類(lèi)為用戶(hù)類(lèi),這個(gè)User類(lèi)的背景設(shè)定為,框架自帶,不允許修改,并且隱藏在框架深處,不容易發(fā)現(xiàn),實(shí)際案例可以參考laravel框架的Request類(lèi),代碼如下:

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ī)中矩,但是接下來(lái),我們需要對(duì)這個(gè)類(lèi)進(jìn)行操作:

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

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

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

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

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

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

首先是基于這個(gè)User衍生出來(lái)的Teacher類(lèi):

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

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

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

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

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

我們應(yīng)該這樣寫(xiě):

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

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

2. 類(lèi)or數(shù)組

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

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

就像如下代碼:

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

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

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ù)都會(huì)有這一套出現(xiàn),不僅代碼不美觀,而且很臃腫。

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

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

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

1)返回int類(lèi)型的數(shù),然后通過(guò)這個(gè)int類(lèi)型的數(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)用者就可以很簡(jiǎn)單的使用方法并給出提示了:

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

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

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

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

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

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í)候,盡量使用同種類(lèi)型的變量,盡量減少使用數(shù)組返回。

相關(guān)文章

最新評(píng)論