Как правильно делать возврат из метода?

Question

[bedolazhka]

Я рассуждаю, например: мой метод должен вернуть или true или false.
А что вернёт метод, если по условию есть или true или действие?

protected function redirect($http = null)
    {
        session_regenerate_id();
        if (is_null($http)) {
            $redirect = $_SERVER['HTTP_REFERER'] ?? '/';
        } else {
            $redirect = $http;
        }
        header('location: ' . $redirect);
        exit();
    }

protected function checkToken(): bool
    {
        if (filter_input(INPUT_POST, 'do') == session_id()) {
            return true;
        }
        $this->redirect(); // как видим тут у нас редирект
        return false; // тогда нужно ли тут возвращать булев?
    }

// применение
if (filter_has_var(INPUT_POST, 'do') and $this->checkToken()) {}

Comments

[Дмитрий]

По идеи редирект из чека надо выносить, если чек использовать для проверки

[bedolazhka]

Дмитрий, я задолбаюсь тогда во всех отправках отдельно редирект колбасить, если не прошла проверка :(

[Дмитрий]

bedolazhka, ну так ведь не правильно, метод долже либо сделать редирект, либо проверить. Тот чек не может быть в условии

Answer 1

[Евгений]

Метод checkToken() делает 2 дела, поэтому у вас не получается решить что он должен возвращать.

Comments

[bedolazhka]

protected function checkToken()
    {
        if (filter_input(INPUT_POST, 'do') == session_id()) {
            return true;
        }
        $this->redirect();
    }

?

[Дмитрий]

bedolazhka, лучше от обратного, if (!filter_input ...
return false;

[bedolazhka]

Дмитрий, может так? ну если в возврат упираться

protected function checkToken($button_name): bool
    {
        if (filter_has_var(INPUT_POST, $button_name)) {
            if (filter_input(INPUT_POST, $button_name) == session_id()) {
                return true;
            }
            $this->redirect();
        }
        return false;
    }

// применение
if ($this->checkToken('do')) {}

[Дмитрий]

bedolazhka, зачем два if когда можно объеденить, после возврата редирект работать не будет, скорее и после редиректа ни чего не придёт

[Евгений]

bedolazhka, и снова метод решает две задачи...

[bedolazhka]

Евгений, работает правильно - сбрасывает повторную отправку формы

[FanatPHP]

На самом деле даже больше чем два. Еще работает с инпутом.

[bedolazhka]

FanatPHP,

<button class="btn btn-sm btn-success" name="do" <!--сюда при нажатии на кнопку вставляется value="id сессии"-->><i class="fas fa-sign-in-alt"></i> Войти</button>

Answer 2

[#ямыnginx]

Для метода redirect($http = null) поставьте тип возвращаемой информации - void.
То есть, укажите, что, в буквальном смысле - "ничего". Потому как у Вас даже return'a нет, значит ничего и не возвращается.
protected function redirect($http = null) : void
P.S. Работает с версии PHP 7.2+ (если не ошибаюсь)