kk
Скорей всего это просто мусорный код который решили не исправлять после того как он стал не нужен
Size: a a a
2020 December 11
D
Возможно. Там такое в нескольких функциях
I
Без try catch, вроде ж, то же самое будет
в смысле в typeorm там свои try catch?
D
в смысле в typeorm там свои try catch?
В смысле что если убрать try...catch, логика работы функции вообще никак не поменяется
I
D
Так делают, когда надо выбросить другое исключение, но в том случае исключение перехватывается, и оно же выбрасывается дальше
I
В смысле что если убрать try...catch, логика работы функции вообще никак не поменяется
да и к томуже это на выборку метод.
I
Так делают, когда надо выбросить другое исключение, но в том случае исключение перехватывается, и оно же выбрасывается дальше
типа если выбрасывается 500 а мы хотим юзеру вернуть 400?!
D
типа если выбрасывается 500 а мы хотим юзеру вернуть 400?!
Ну, например, если у тебя ORM кидает исключение из-за дубля, а тебе нужно, например, выбросить другой свой Exception типа UserAlreadyExists
D
Ну да, типа того. Правда запихивать вообще всё в try смысла никакого. Можно только save(..) в try сунуть - гораздо красивее выглядеть будет
D
Хотя в твоём случае запрос в БД не только в save идёт, так что не важно
I
Ну да, типа того. Правда запихивать вообще всё в try смысла никакого. Можно только save(..) в try сунуть - гораздо красивее выглядеть будет
но может быть несколько save
D
Да. А зачем тебе return true?
D
У тебя ж false не возвращается никогда
I
Да. А зачем тебе return true?
:) действительно :)
I
согласен
D
Но вообще http-исключения я бы из сервисного слоя не выбрасывал
D
Мне так кажется (моё мнение), что http-исключения должны выбрасывать контроллеры
D
А в сервисах исключения не должны быть связаны с http вообще никак. Ведь ты можешь из командной строки в теории сервис дёрнуть. И в чём прикол, если он кинет BadRequestException?)