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(); 
}