Как избавиться от try-catch в контроллерах?

Question

[Di Lee]

В REST-API проекте на Symfony есть куча контроллеров, где в каждом есть try-catch
Понятно, что это неправильно потому, что это повторяемость кода
Как избавиться от такого ***кода и вынести try-catch куда-то? Возможно вынести в отдельный метод либо есть другие удачные решения?

Пример роут метода контроллера:

/**
* @Route("/comment/create", name="comment_create", methods={"POST"})
*/
public function __invoke( Request $request ): JsonResponse {
  try {
    $data = $this->initRequestParams($request);
    $someService->doSomething($data);
  } catch (\Throwable $exception) {

    return new JsonResponse(['error' => 'Страница не найдена'], Response::HTTP_NOT_FOUND);
  }

  return new JsonResponse(null, Response::HTTP_CREATED);
}

Comments

[Максим Федоров]

удачное решение — когда обертка у вас прямо по месту, как в примере... хлоп и дебажить/рефакторить прямо тут и будете

[Дмитрий]

ну ум только велосипед приходит

Answer 1

[Максим]

Смотря для чего вы делаете эти try catch. Это не всегда повторяемость иногда действительно нужна, например, чтобы перебросить своё исключение. Пойти можно разными способами. Но а простой - это вам нужно просто их убрать и сделать Listener или Subscriber, который будет обрабатывать ошибки и отдавать их в нужном формате в api. Если правильно понял для чего нужно. Пример.

Comments

[Di Lee]

Спасибо! То что надо!

Answer 2

[Григорий Васильков]

сделать мидлвар/слушатель, который вокруг контроллера, но просто его в коде не видно, он как правило часть фреймворка. или в index.php обернуть require что-то в try catch в одном единственном месте

try catch это же "поставить следилку, закрыть следилку, что делать с отслеживаемым". Не пиши их и их не будет. Респонсы ифами генерируй "if (! $content) return new JsonResponse();". А то что вы хотите эксепшен выбросить как JSON это не про конкретный трайкетч, это про "обработка ошибок в рамке папки с контроллерами по имени api/v1/"

Comments

[Di Lee]

Вопрос в чистоте кода, чтобы не было повторяемости, потому просто не писать - это не вариант.
Ну и также искалось продуманное решение, чтобы не изобретать колесо.
То есть да, как вы заметили, нужна именно обработка ошибок для конкретного типа контроллеров.
Решение с Listener или Subscriber наиболее подходящее думаю. Нашел уже в нескольких источниках такой подход к данной проблеме.

[Григорий Васильков]

Di Lee, да, я обычно пишу в Handler.php (это если Laravel), в ResponseListener.php (если Symfony) или index.php (если самопал) в блоке `catch` функцию, которая берет текущий роут, смотрит в какой он "папке" и по ней решает - Json выдавать, xml и так далее пишу типа

$mapResponse = [
  'api/v1/json' => JsonResponse::class,
  'api/v1/xml' => XmlResponse::class,
];

$responseClass = null;
foreach ($mapResponseClass as $routeBeginsWith => $class) {
  if (0 === strpos($currentRoute->pattern, $routeBeginsWith)) {
    $responseClass = $class;
    break;
  }
}

$response = new $responseClass; // и он выпадет здесь в ошибку если я забыл что-то указать и просто указываю ещё что-то в мапе.

[Di Lee]

Григорий Васильков, ResponseListener.php - это же из папки vendor? Если так, то это рисковый путь что-то там исправлять

[Григорий Васильков]

Di Lee, не не, это я сам создаю папку Listeners и туда файл ResponseListener.php где обработка всех респонсов с программы.