Как сократить данный код?

Question

[JTester]

Здравствуйте! Подскажите пожалуйста, как сократить данный код, тип чтобы там методы не повторялись каждый раз и т.д

if ($user->isModerator()) {
    $user->updateStatus(self::SUPPORT);

    $a = $this->get($user, 'status', []);

    echo 'Вы модератор';
}
elseif ($user->isSupport()) {
    $user->updateStatus(self::MODERATOR);

    $a = $this->get($user, 'id', []);

    echo 'Вы поддержка';
}

Comments

[FanatPHP]

что означает действие $user->updateStatus(self::SUPPORT);?

[JTester]

FanatPHP, изменить статус на суппорт

[FanatPHP]

Почему пользователю меняется статус на суппорт, но пишется, что он модератор?

[FanatPHP]

для чего нужно $a = $this->get($user, 'status', []);?

Answer 1

[Антон Р.]

Вынести их в отдельную функцию.

Comments

[JTester]

Кого вынести в отдельную функцию? Они и так в отдельной функции, куда ещё?

Answer 2

[Вечно Крайний]

Вынести в отдельный метод можно типа checkRole() и узнать какая роль у юзера, а не спрашивать каждый раз юзер админ? юзер саппорт? и т.д А так код вополне понятен) Если только у Вас не 100500 ролей)

Answer 3

[edward_freedom]

Можно сделать вместо вызова функции при заполнении массива, указывать функцию и в цикле проверять до возвращения true, чтобы не вызывать все функции сразу

$options = [
    [
        "condition" => isModerator(),
        "execute" => function () {
            echo 'isModerator';
        }
    ],
    [
        "condition" => isSupport(),
        "execute" => function () {
            echo 'isSupport';
        }
    ],
];


array_walk(array_filter($options, function ($item) {
    return $item["condition"];
}), function ($item) {
    $item["execute"]();
});

Comments

[Lander]

Сократил так сократил. :) Шучу. :)

[JTester]

Lander, пхахаххах, топ