Как написать функцию проверки пользователя на права админа?

Question

[semki096]

Получилось так (но чувствую что этот код не очень хороший). Посоветуйте, правильно ли я написал, если нет - то как лучше это переписать? А также по безопасности, если есть замечания буду благодарен.

public function userIsAdmin() {
        if(isset($_SESSION['user'])){
            if($_SESSION['user'] == 1){
                return true;
            }
            else {
                return false;
            };
        } else {
            return false;
        };
    }

Comments

[Максим Грищенко]

ОФФТОП:
если сделали return, то не обязательно делать условие else
грубо говоря так чище:

public function userIsAdmin() {
    if(isset($_SESSION['user'])){
        if($_SESSION['user'] == 1){
            return true;
        }

        // сюда код дойдет, если не равен одному, но сессия сеть
        return false;
    }

    return false;
}

[Дмитрий]

А можно вопрос? Зачем после фигурных скобок точка с запятой?

[Rsa97]

public function userIsAdmin() {
  return (($_SESSION['user'] ?? 0) === 1);
}

Answer 1

[Антон Шаманов]

Вначале нужно почитать теорию, гугли - RBAC, ACL