Question

我有一个类，在大多数方法之前（不是全部），我检查一个id是否有效。类中每个方法的目的是检查各种用户详细信息，每个方法返回true或false，以便其他类可以检查事物。

public function doSomething($userId){

    $valid = false;

    if(empty($userId)){

        return $valid;
    }

    //do other stuff which may turn valid true

    return $valid;
}

因此，为了整理课程，我已将此检查从它出现的所有方法中移到了自己的方法上。

public function doSomething($userId){

    $valid = false;

    $this->idCheck($userId);

    //do other stuff

    return $valid;
}

private function idCheck($userId){

    if(empty($userId)){

        return false;
    }
    return true;
}

如果id检查失败，我怎么能得到方法 - doSomething返回？是否可以在此方法中进一步检查：

if(!$this->idCheck($userId))return false;

或者有更好的方法吗？

Answer 1

你是以正确的方式做到的。如果ID无效，只需返回您的功能。否则 - 那就更好了，在构造函数中检查你的ID！

public function __construct($userID)
{
    if(!empty($userID))
    {
        //do constructing things
    }
}

正如DarkBee所提到的，你可以将你的idCheck功能重写为：

private function idCheck($userID)
{
    return empty($userID);
}

Answer 2

我会做这样的事情：

private function idCheck($userId)
 {

     $status=true;
    if(empty($userId))
       {

        $status=false;
       }

       return $status;


 }

从方法返回true / false

2 个答案: