Также строчки 67 и 68 $query->email = $body['email'] ?? '';

Question

Также строчки 67 и 68 $query->email = $body['email'] ?? '';

$query->password = $body['password'] ?? '';

Опять же валидация в том же самом методе происходит. Лучше всего ее вынести в отдельный метод или класс, как мне кажется

Строчка 70.

$user = $this->users->fetch($query);

Можно добавить метод fetchOrFail, который будет сразу проверять, если такой пользователь. И если его нет, то отправлять ошибку

#backend #php #programming #russian

0

26.08.2021

4 ответов

9 просмотров

Максим Автор вопроса

Volodymyr Melko
fetchOrFail - ларавельщики подїехали? =)

Да, я на ларавель работаю

0

26.08.2021

Дмитрий Елисеев

> Строки 67 и 68 Это присваивание в будущем уйдёт в вызов сериалайзера. И, как сказал выше, подход из Laravel с выносом заполнения и валидации наружу здесь слабоприменим.

0

26.08.2021

Дмитрий Елисеев

> Строка 70 В случае этого экшена нельзя просто сделать throw NotFoundException, так как это полностью прервёт экшен, не отрендерив форму. Здесь в случае ненахождения нужно вывести ошибку валидации над формой. Для этого придётся код всё равно оборачивать в try { fetchOrFail } catch { error = ... }, что не удобнее проверки на null.

0

26.08.2021

Volodymyr Melko · Accepted Answer

Volodymyr Melko

fetchOrFail - ларавельщики подїехали? =)

0

26.08.2021

170 похожих чатов

Также строчки 67 и 68 $query->email = $body['email'] ?? '';

4 ответов

Похожие вопросы