如何最佳化php代碼結構

每次調用方法後，都需要驗證傳回值來決定返回還是繼續執行，以下代碼如何調整會好些？

/** * 執行商務邏輯 * @param $action 執行方法 * @param $allParams * @return array|bool */public static function parseMore($action, $allParams){    // 使用者登入請求資料解析    $inputs = self::userLoginParse($allParams);    //驗證    if(self::$_errorNo != StatusCode::STATUS_TRUE) return array();    if($inputs === false) return false;    // 調用業務方法    $result = call_user_func(array("Frontier", strtolower($action)), $inputs);    //驗證    if(self::$_errorNo != StatusCode::STATUS_TRUE) return array();    if($result === false) return false;    return array(        $action => $result    );}

回複內容：

每次調用方法後，都需要驗證傳回值來決定返回還是繼續執行，以下代碼如何調整會好些？

/** * 執行商務邏輯 * @param $action 執行方法 * @param $allParams * @return array|bool */public static function parseMore($action, $allParams){    // 使用者登入請求資料解析    $inputs = self::userLoginParse($allParams);    //驗證    if(self::$_errorNo != StatusCode::STATUS_TRUE) return array();    if($inputs === false) return false;    // 調用業務方法    $result = call_user_func(array("Frontier", strtolower($action)), $inputs);    //驗證    if(self::$_errorNo != StatusCode::STATUS_TRUE) return array();    if($result === false) return false;    return array(        $action => $result    );}

這個情境下應該使用異常而不是傳回值。

包括userLoginParse和那個call_user_func的動態方法，看上去都在用self::$_errorNo傳遞錯誤狀態，這是連C/C++都不推薦的糟糕模式（全域變數傳遞錯誤碼）

這段代碼或許應該長這樣

public static function parseMore($action, $allParams){    try {        $inputs = self::userLoginParse($allParams);    } catch(EmptyInputException $e) {        return array();    }    $result = call_user_func(array("Frontier", strtolower($action)), $inputs);    return array(        $action => $result    );}

或許應該長這樣

public static function parseMore($action, $allParams){    $inputs = self::userLoginParse($allParams);    if($inputs === null) {        return array();    }    $result = call_user_func(array("Frontier", strtolower($action)), $inputs);    return array(        $action => $result    );}

取決於那個return array();的分支和userLoginParse的業務

我通常判斷一個函數返回bool是否合理是這樣判斷的

a)bool和其他類型混合：一定不合理。比如題主這種情境應該用異常，比如查詢XXX返回對象/false 應該用null
b)看函數名字，如果不能改叫isXXX，hasXXX等回答是否的疑問句，八成有問題

我會考慮如下2個方法：
1、把檢查的地方剝離出來，單獨寫個方法，用來檢查$_errorNo和傳回值
2、在調用的方法裡，拋出異常而不是return false，然後外面catch住，根據異常的code來判斷返回false還是空數組

