209 похожих чатов

Вчера тут сказали что не стоит в сервисах кидать эксепшены,

а просто возвращать в целом в сервисе два поля. сделал как сказал. это будет считаться нормальным кодом?

9 ответов

29 просмотров

Не будет. Нормальный код всегда объявляешь только ты.

Нет, в сервисе возвращаешь статус коды которые привязаны к конкретному транспорту, что неправильно

384
а как делать?

Вариантов много

Антон Горбачёв
Нет, в сервисе возвращаешь статус коды которые при...

Типа для котроллера отдельный метод сервиса, который возвращает специфический эксепшн? Например getUser и getUserHttp (который является обернутым getUser с прописанными ошибками)

Sw
Типа для котроллера отдельный метод сервиса, котор...

Как написал выше, вариантов много, самый простой и тупой, это кинуть из сервиса ошибку что UserNotFound например, а в catch блоке контроллера уже проверять ошибку на тип и если тип UserNotFound, то уже кидать специфичную для транспорта ошибку с нужным статус кодом и тд. Вариант чуть лучше, возвращать из сервисов Either/Option и матчить результат в контроллере, возвращая специфичную для транспорта ошибку Плюс ещё пачка вариантов сверху есть

Антон Горбачёв
Как написал выше, вариантов много, самый простой и...

А в чем разница кидать error UserNotFound или исключение HttpException это же по факту одно и тоже, тебе в любом случае надо ставить тру-кэтч чтоб обработать это по своему... тут скорее всего HttpException звучит как то инфраструктурно и не подходит по неймингу для сервиса и вот если бы в Nestjs была иерархия наследования Error->Exception->HttpException Тогда все эти вопросы отпали бы

DB
А в чем разница кидать error UserNotFound или искл...

Это не одно и тоже и дело не в нейминге. Иерархия наследования есть

Похожие вопросы

Обсуждают сегодня

@Benzenoid can you tell me the easiest, and safest way to bu.y HEX now?
Živa Žena
20
This is a question from my wife who make a fortune with memes 😂😂 About the Migration and Tokens: 1. How will the old tokens be migrated to the new $LGCYX network? What is th...
🍿 °anton°
2
What is the Dex situation? Agora team started with the Pnetwork for their dex which helped them both with integration. It’s completed but as you can see from the Pnetwork ann...
Ben
1
Anyone knows where there are some instructions or discort about failed bridge transactions ?
Jochem
21
@lozuk how do I get my phex copies of my ehex from a atomic wallet, to move to my rabby?
Justfrontin 👀
11
Hello, Is iExec also part of the "inception program" or another one ? Would it be a name to qualified the nature of the relationship between iExec and Nvidia? And does Secret ...
Ñïķøłäś
8
Ready for some fun AND a chance to win TKO Tokens? Join us for exciting minigames in our Telegram group! 🕒 Don’t miss out—games start on today 25 October 2024, at 8 PM! Ge...
Milkyway | Tokocrypto
255
any reference of this implementation?
BitBuddha
29
Also, why can’t the community have a vote/ say when it comes to initiatives like buybacks. Isn’t the point of crypto decentralisation? Don’t we deserve input as long term supp...
👨🏽‍🦰
13
Hi guys, any problem with Pulsebrige? Trying to transfer from wETH to ETH. First it tells me to connect my metamask "through mobile app" not desktop. Then I did and confirmed ...
Snowflakecrypto
13
Карта сайта