PHPの関数に対して論理エラーを返すベストプラクティスは何ですか

Question

ユーザーに従業員のアクセス許可があるかどうかをチェックしたいと思います。

function hasEmployeePermission($employeeID, $userKey) 
{ 
    $usersID = DB::table('users')->where('key', $userKey)->value('id'); 

    if($userID != null) { 
     $employeeID = DB::table('employees')->where('user_id', $userID)->value('id'); 

     if($mployeeID != null) 
      return true; 
     else 
      return false; 
    } 
    return false; 
} 

私は、戻り値は例外を投げるような、より表現力になりたいです。論理的なエラーで例外を投げるのはベストプラクティスではないと思います。私はエラーを返すようにコードを修正する方法を知りたがりません。

Answer 1

単純なエラークラスを作成します。あなたはこのような何かを行うことができます

Class myError{ 
    public $error = true; 
    public $message = null; 
    function __construct($error,$message) { 
     $this->error = $error; 
     $this->message = $message; 
    } 
} 

その後、

if($mployeeID != null) 
     return new myError(true,"no permission"); 
    else 
     return new myError(false,"has permission"); 

は、例えば、クラスに追加、より多くの機能があるかもしれませんエラーのどこかにログを記録する

Answer 2

あなたの関数が失敗した理由を知りたいのであれば、enumを使うことをお勧めします。ここで

は一例です：

abstract class EmployeeErrors 
{ 
    const WrongID = 1; 
    const NoPermissions = 2; 
    // etc. 
} 

function hasEmployeePermission($employeeID, $userKey) 
{ 
    $usersID = DB::table('users')->where('key', $userKey)->value('id'); 

    if($userID != null) { 
     $employeeID = DB::table('employees')->where('user_id', $userID)->value('id'); 

     if($mployeeID != null) 
      return 0; 
     else 
      if ($userKey == null) 
       return EmployeeErrors::WrongKey; 
      else ... 
    } 
    return EmployeeErrors::WrongID; 
}