TP模型中是返回還是拋出異常

自己寫的在模型中返回一個(gè)數(shù)組由控制器根據(jù)參數(shù)響應(yīng)給視圖,代碼如下

1:瀏覽器post請求控制器;
2:控制器實(shí)例化用戶模型
3:模型中接受到全部的數(shù)據(jù)存入$Data變量
4:利用驗(yàn)證器驗(yàn)證數(shù)據(jù)合法性,如果驗(yàn)證不通過返回status為:0;并且返回msg:錯(cuò)誤信息
5:模型總save新增數(shù)據(jù)
6:新增成功返回1:提示新增成功新增失敗返回0:表示新增失敗:

控制器代碼:

//roue:member/index/userAdd
public function userAdd(){
    $Result = (實(shí)例化會員模型)->add();
    //中間可能還有其他操作 
    return ajaxReturn($Result['status'],$Result['msg']);
}

模型代碼

public function add(){
    $Data = input('');
    $Validate= new UserAddValidate($Data);
    if(false === $Result){
    //如果嚴(yán)重失敗會返回false 
    //返回狀態(tài)0,內(nèi)容為:msg 
        return ['status',=>0,'msg'=>$Validate->getError()];
    }
    //執(zhí)行新增
    $res = $this->save($Data);
    if($res){
        return ['status'=>1,'msg'=>'添加會員成功'];
    }else{
            return ['status'=>0,'msg'=>'添加會員失敗'];
    }
}

這個(gè)是自己琢磨的但是感覺好麻煩,每次都要各種的if判斷
所以我就去看別人的代碼,看到一個(gè)大神些的利用的是tp5的自定義異常處理類,代碼寫法如下

控制器代碼:

//roue:member/index/userAdd
public function userAdd(){
   return (實(shí)例化會員模型)->add();
}

模型代碼

public function add(){
    $Data = input('');
    $Validate= new UserAddValidate($Data);
    if(false === $Result){
        throw new HttpException([
        'status' => 0,
        'msg' => $Validate->getError();
        ]);
    }
    $res = $this->save($Data);
    if($res){
     throw new HttpException([
         'status' => 1,
         'msg'= > '添加會員成功',
     ]);
        
    }else{
     throw new HttpException([
         'status' => 0,
         'msg'= > '添加會員失敗',
     ]);
    }
}

HttpException 是TP5里面可以自定義的一個(gè)異常處理類最終結(jié)果就是拋出異常,異常中的內(nèi)容是json后的結(jié)果,
感覺這種相對來說很好用,但是不知道為什么大多數(shù)人的代碼都沒用用這種呢?
或者還有什么更好的辦法嗎?

以上代碼全是在瀏覽器中手寫的,可能會有一點(diǎn)點(diǎn)的誤差!

回答

編輯回答

旖襯

你只是想返回一個(gè)結(jié)果，但使用 throw 一般是拋出異常。
很多時(shí)候返回 status 為0，其實(shí)并不是異常。
我覺得這些時(shí)候使用 throw 是不合適的。
所謂的異常，應(yīng)該是“它本來應(yīng)該是這樣的，但結(jié)果卻不是”，才需要拋出異常。
比如你要處理一張圖片，傳的參數(shù)指向的文件存在，文件后綴也存在，但讀取數(shù)據(jù)后卻不能正確處理，這時(shí)拋個(gè)異常是可以理解的，但我個(gè)人覺得這還不夠好。

因?yàn)槲揖陀龅竭^，使用 thinkPHP 3.2 的圖片處理類，遇到不能處理的圖片，直接拋出了異常，但其實(shí)對我來說，如果處理圖片有問題，是可以直接跳過去的。

也就是說：只有“這一步如果出錯(cuò)，后面的代碼執(zhí)行就完全沒有意義或者完全不能執(zhí)行”的時(shí)候才去拋出異常，否則只需要返回結(jié)果，給使用者（可能是控制器）去處理就好了。

對于樓主的需求，ThinkPHP 是有內(nèi)置的方法的，比如控制器中有 success() 和 error() 兩種方法來滿足樓主的需要，當(dāng)然，我個(gè)人比較喜歡 ThinkPHP 3.2的處理方式，比如簡潔，ThinkPHP 5中的這兩個(gè)方法添加了不必要的參數(shù)，寫起來倒挺麻煩的。
但這兩個(gè)方法只在控制器中有，模型中默認(rèn)是沒有的，當(dāng)然，如果你喜歡的話，也可以加到模型中去。
但我個(gè)人同樣不太推薦。處理返回結(jié)果或頁面跳轉(zhuǎn)的任務(wù)還是交給控制器會比較好。

模型中，其實(shí)可以自己定義個(gè)類似的方法，比如：

function succ($data="",$status=1){
      return array(
       'status'=>$status,
       'info'=>$data,
    );
}

function error($msg=""){
  return succ($msg,0);
}

你在模型中直接 return succ($data);或 return error($msg);即可。

控制器中根據(jù)返回的數(shù)組，再使用$this->success()或$this->error();當(dāng)然，如果你在使用ThinkPHP 5,也像我一樣比較喜歡 ThinkPHP 3.2 的風(fēng)格，這兩個(gè)方法你是可以改成3.2的風(fēng)格的。

說回樓主和你看到的大神的代碼，基本思想都差不多，只不過你直接返回一個(gè)數(shù)組，而“大神”卻直接拋出異常，我不喜歡這位大神的風(fēng)格。
但你和“大神”都喜歡寫重復(fù)的 "status'=>1,"msg"=>......，我則比較喜歡把它裝進(jìn)函數(shù)或方法中，畢竟這兩個(gè)操作是使用非常頻繁的。

關(guān)于代碼中較多 if/else 的情況，我一般使用兩種方法解決：

短的判斷可以用三目運(yùn)算符代替；
很多時(shí)候 if 時(shí)使用了return ，后面就可以不寫 else 了。

補(bǔ)充：

概括一下：

模型中建議使用 return ，而不是拋出異常，return 的數(shù)據(jù)可以自己封裝個(gè)函數(shù)或方法；(當(dāng)然，要結(jié)合業(yè)務(wù)的具體需求！)
拋出異常意味著程序的終止，在非必須的情況下不要這樣做；
控制器中直接使用 success 和 error 或 ajaxReturn 方法即可。

但如果把樓主看到的“大神”的代碼從模型中移到控制器的話，應(yīng)該是合理的（除了重復(fù)的代碼有點(diǎn)多外）。
我個(gè)人是不太喜歡使用 throw，但我查看了一下 ThinkPHP 5 和 ThinkPHP 3.2 的源碼，其中的 success和 error 方法除了使用參數(shù)復(fù)雜性不同外，還有一點(diǎn)區(qū)別就是 3.2直接使用 exit()結(jié)束程序，而 5.0 版本中使用的是 throw 一個(gè)異常類。

應(yīng)該有理由相信，使用 throw 可能會比如說 exit 更好。
于是，我找到一種說法：