Является ли такой подход плохим в работе с исключениями, если в методе render сразу же возвращать ответ в формате json?

Question

[Александр Яремко]

Не является ли такой подход плохим в работе с исключениями, если в методе render сразу же возвращать ответ в формате json, вот пример:

namespace App\Exceptions\Telegram;

use DomainException;
use Illuminate\Http\Response;

class InvalidDataTelegramException extends DomainException
{
    public function render()
    {
        return response(
            ['message' => __('auth.incorrect_telegram')],
            Response::HTTP_UNPROCESSABLE_ENTITY,
        );
    }
}

Comments

[Ипатьев]

Когда вы задаете вопрос, вам русским языком пишут, Сформулируйте вопрос так, чтобы сразу было понятно, о чём речь.

Answer 1

[iljaGolubev]

Исключение без стандартных message, code. Фикс json респонз. Не делайте так.
Это сейчас вам достаточно всегда получать json. Но что будет при вызове из консоли? а в логах?

Как минимум - релиазовать Illuminate\Contracts\Support\Responsable.
А если Symfony\Component\HttpKernel\Exception\HttpExceptionInterface релиазовать или просто наследовать от HttpException то в вашем случае будет достаточно

throw new InvalidDataTelegramException(Response::HTTP_UNPROCESSABLE_ENTITY, __('auth.incorrect_telegram');

Illuminate\Foundation\Exceptions\Handler сделает тоже, что вы и даже больше.

Comments

[Ипатьев]

Я тоже сначала хотел на это заругаться, но потом увидел что речь идет о методе render. Насколько я понимаю, при вызове из консоли этот метод вызываться не будет. А в логах будет как раз и code, и message

[iljaGolubev]

Ипатьев, в общем то да... соглашусть что однозначно "плохом" не является.
только вот имхо форсить json - выстрел в ногу через X месяцев (но это не точно).
Для обработки исключений есть "замечательный" App\Exceptions\Handler->map.

Answer 2

[Андрей]

Имхо, таки рендерить исключения для клиента лучше или в контроллере, или в команде, где там у вас точка входа.
Иначе, если у вас исключение знает, как его обрабатывать - то какое же это исключение?