PHP如果条件

Question

我很尴尬。

我有三个函数来检查用户当前登录是什么：

public function isAuthor(User $user) 
{ 
    return $user->getId() === $this->getDestination(); 
} 

public function isSupervisor(User $user) 
{ 
    return $user->getId() === $this->getFirstApprover(); 
} 

public function isSecondApprover(User $user) 
{ 
    return $user->getId() === $this->getSecondApprover(); 
} 

然后，我想里面添加我的行动的条件来检查，如果用户是任何上述三人。如果他不是其中之一，应该拒绝访问。用户有时可能不止一个，但大多只有三个中的一个。

我是这样的第一想法，但显然它不能正常工作

if (!$object->isAuthor($this->getUser()) || !$object->isSupervisor($this->getUser()) || !$object->isSecondApprover($this->getUser())) { 
    throw new AccessDeniedException(); 
} 

会有什么检查，如果用户是他们中的一个最好的办法？我应该创建一个全新的功能吗？

我应该使用这样的事：

if (!$object->isAuthor($this->getUser())) { 
    throw new AccessDeniedException(); 
} elseif (!$object->isSupervisor($this->getUser())) { 
    throw new AccessDeniedException(); 
} 

我可以请有从别人的一些想法和投入？因为我现在很困惑。 还在这里一个新手

Answer 1

您的逻辑将工作，这是只是一个“反向”布尔逻辑，很复杂编辑按照。它有一个错误，使用& &而不是||。

一种替代：

if (! ( $object->isAuthor($this->getUser()) || 
     $object->isSupervisor($this->getUser()) || 
     $object->isSecondApprover($this->getUser())) 
{ 
    throw new AccessDeniedException(); 
} 

另一种选择，你可以在 “对象” 类写一个函数：

public function hasAccessLevelX(User $user) 
{ 
    return in_array($user->getId(), [ 
      $this->getDestination(), 
      $this->getFirstApprover(), 
      $this->getSecondApprover() 
    ]); 
} 

if (!$object->hasAccessLevelX($this->getUser())) { 
    throw new AccessDeniedException(); 
} 

我会用后者。

Answer 2

使用下面的代码：

常用功能

public function userLogin(User $user) 
{ 
    $userId = $user->getId(); 
    if($userId == $this->getDestination() || 
     $userId == $this->getFirstApprover() || 
     $userId == $this->getSecondApprover()) 
    { 
     return TRUE;  
    } 
    return FALSE; 
} 

使用/通用功能的呼叫

if ($object->userLogin($this->getUser()) == FALSE) { 
    throw new AccessDeniedException(); 
}