Где грамотнее делать try/catch?

Question

[Floydreme]

Здравствуйте! Ситуация:
Есть обычный CRUD, реализованный в виде многослойной архитектуры. Столкнулся с небольшим непониманием, где стоит обрабатывать разного рода ошибки (не найдена сущность в БД / неверная роль / etc.) - в BLL или же на уровне клиента.
Я пришел к мысли, что самым оптимальным вариантом будет на уровне клиента:

• BLL по факту - прослойка между клиентов и БД. Клиент может изменяться, а если делать try/catch в сервисах, то некоторые специфические для какой-то одной платформы ошибки (например, веб), могут оказаться некорректны на другой
  
• Если опираться на такое решение, то мы, грубо говоря, делегируем сервису вытащить какие-то данные. Если есть какая-то ошибка, я смогу ее отвалидировать на уровне клиента в зависимости от платформы
  

Скажите, пожалуйста, я прав или нет?

P.S под клиентом подразумеваются контроллеры

Comments

[WbICHA]

Вообще-то ошибки должны обрабатываться и там, и там.

[Floydreme]

WbICHA, допустим, есть endpoint для поиска сущности по id, я вызвал метод в сервисе, он обратился к репозиторию и тот выкинул ошибку. Зачем делать повторение кода и в сервисе, и на клиенте, если по факту ошибка одна, и на клиенте я должен реализовать новое поведение? В чем бенефит от валидации на уровне бизнес логики? Не очень понимаю(

[WbICHA]

Floydreme, в том, что у тебя фронт не будет зависеть от ошибок самой БД. Обновится твоя БД однажды и там, допустим, часть ошибок начнут по другому выводится и вот у тебя уже всплывают бековские проблемы на фронте.
А если тебе надо будет поменять БД? Это переписывать бэк И фронт, а не только бэк.
Потом дело дойдёт до тайпскрипта и удачи тебе типизировать ошибки БД, в то время как при нормальном подходе у тебя будут твои личные ошибки с необходимой тебе структурой.

Ах, да, ещё банальное желание передать какую-то инфу вместе с ошибкой и вот у тебя уже разные принципы обработки ошибок.

[Floydreme]

WbICHA, да, это звучит валидно
Спасибо большое

Answer 1

[Владислав Вознесенский]

Можно использоваться оба варианта. Часто, а точнее всегда, зависит от ситуации.
К примеру, вы хотите в репозитории сделать запрос на сторонний сервис (библиотекой got или axios). Ответ может быть какой угодно, включая ошибки построения неправильного запроса. В таком случае лучше обернуть данный блок кода в try/catch.
На все контроллеры, которые так или иначе взаимодествуют с бд, можно повесить try/catch на уровне контроллера.
В своих проектах, обычно, я оборачиваю все контроллеры в try/catch и передаю ошибку в последний middleware, где есть общий обрабочтик, который в зависимости от ошибки отдаёт пользователю соответсвующий ответ

Крайне рекомендую глянуть best practice для ноды, так как там как раз часто упоминается способ обработки ошибок.