Всем привет) вопрос приложение порты-сервисы-адаптеры в каких случаях можно кидать исключения собственноручно, а

Question

Всем привет) вопрос приложение порты-сервисы-адаптеры в каких случаях можно кидать исключения собственноручно, а

в каких это нини?
я знаю что контролировать флоу приложения с помощью исключений это не очень хорошо
вот пример того что есть в приложении:
в миддлвейре есть фильтр эксепшенов, который ловит разные типы исключений и преобразовывает их в разные http респонзы с кодами (например общий эксепшн это 500, кастомный NotFoundException это 404, InvalidRequestException это 400, ошибка/краш в адаптерах - UpstreamException(название неточное) - это 502, и тд)
в портах не кидаются исключения
в сервисах не кидаются исключения , и в основном не перехватываются (но иногда все же перехватываются)
в адаптерах обертка httpClient-a кидает апстрим исключение если вернулся !response.IsSuccessStatusCode (к этому у меня нет вопросов)
и в адаптерах обрабатывается отдельно случай, когда данные не найдены (внешний сервис вернул код 404 или 200 с пустым телом) и кидается NotFoundException - вопросы к этому моменту. Далее, либо это исключение не ловится и апи возвращает 404 после отлова фильтром эксепшенов (например эндпоинт ищет юзера по айди, тогда мы кидаем UserNotFoundException), либо это исключение ловится в сервисах и игнорируется (например мы меняем описание книги, и хотим отправить ее автору нотификейшн на имейл, но автор/имейл не найден), тогда мы логируем warning и апи возвращает 200 или 202 или еще ченибудь делает, но не возвращает 404.
Насколько это плохо? Почему?
Этот подход работает около года, очень удобный, не вызывает боли у членов команды (старых и новых), ну и в целом мне нравится как исключения преобразовываются в коды и мне не нужно копипастить обработку и возврат разных кодов в каждом методе каждого контроллера. На продакшене десятки тысяч пользователей из разных стран, ежедневная активность заоблачная, но с перформансом проблем никаких нету (это на случай аргумента о том что эксепшены тяжелые). Стоит ли париться?

#net #programming #russian

0

04.12.2021

7 ответов

11 просмотров

Natali Kolosova Автор вопроса

А на будущее?

0

04.12.2021

Natali Kolosova Автор вопроса

Ilya Chernoudov
Вот гайды мс, например. Информация об исключении в...

Во, именно это мы и делаем For non-success conditions, developers SHOULD be able to write one piece of code that handles errors consistently across different Microsoft REST API Guidelines services

0

04.12.2021

Ilya Chernoudov

Natali Kolosova
Во, именно это мы и делаем For non-success conditi...

Если вы преобразуете коды в экзепшены - это не совсем pice of code

0

04.12.2021

Natali Kolosova Автор вопроса

Ilya Chernoudov
Если вы преобразуете коды в экзепшены - это не сов...

Наоборот, экспешены в коды

0

04.12.2021