Правильная обработка исключений от внешнего апи symfony?

Question

[nyatmeat15]

Здравствуйте. Есть определенное апи, с которым работаю через guzzle. Обработка с апи разбита на 3 уровня. Контроллер, менеджер, сервис апи.
Хотелось бы уведомлять об ошибке в контроллере, что конкретно произошло не так в апи. Должен ли я обернуть блок вызова менеджера, который вызывает апи прям в контроллере, или обработать как то иначе, через какие то модели. Например сервис апи возвращал бы результат валиден или нет, и обрабатывать уже в менеджере?

Comments

[galliard]

Первое, что вам стоит научиться сделать - это научиться грамотно выражать свои мысли. Вот из того что вы написали нихрена не понятно от слова "совсем". Расшифровывать ваш поток сознания будут лишь те, у кого ну очень много свободного времени.

Answer 1

[Decadal]

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

Вот пример из моего кода:

try {
    $results = $core->execute($coreConfig);
}
catch (BreakpointException $e) {
    throw $e;
}
catch (StrategiesException $e) {
    $this->mark("Expected error:".$e->getMessage());
    return $this->formattingResult(ProcessRecord::STATUS_ERROR,$e->getMessage());
}
catch (\Exception $e) {
    $this->mark("Unexpected error:".$e->getMessage()."[".$e->getFile()."] : ".$e->getLine());
    return $this->formattingResult(ProcessRecord::STATUS_UNEXPECTED_ERROR,$e->getMessage());
}

правда у меня это лежит в сервисе, но в принципе, зависит от логики работы.

Comments

[nyatmeat15]

Только вот на каком уровне лучше бросать исключение? На уровне апи все ограничивается по сути кодами http и бывает кодом ошибки. Можно рассматривать сервис апи как репозиторий. Бросать ли в нем исключения для контроллера?

[Decadal]

nyatmeat15, зависит от того, что делает ваш контроллер. У меня нет информации, каким именно целям он у вас служит. Это контроллер админки, внутренней системы? тогда нужна одна политика обработок исключений. Это контроллер апишки? Нужна другая политика. Контроллер консольного приложения? третья.
Если у вас вредные пользователи которые хотят видеть только красивые ошибки то вам придётся обойтись логированием, а в контроллере повесить общий отлов ошибок \Exception (а еще лучше настроить в конфиге обработчик ошибок для этого контроллера целиком), и редиректить всех на красивую одинаковую страницу "сорян братан что-то пошло не так".

[Decadal]

nyatmeat15 , вдогонку к написанному выше, у экшенов контроллера есть разные предназначения. Разумеется при создании\изменении чего-либо (POST, PATCH, PUT), нужно выводить ошибки валидации, полученные из валидатора. У нас на зенде (вроде в симфони похожая тема) за это отвечают формы, которые знают какие поля где отваливаются, и это отображается в красивеньком лейбле прямо напротив неправильно заполненного поля.
А вот если это GET или, в меньшей степени, DELETE то там нужно либо навешивать flash messages с редиректом на родительскую страницу, либо просто редирект на страницу "ошибки".