🏡K
Кастомные ошибки и фильтр для их отображения если отличается формат от кастомных ошибок ядра, если формат совпадает то только кастомная ошибка и енам с типами ошибок
Формат в ядре
{code}
Size: a a a
2020 December 09
🏡K
Каждая либа несёт свои ошибки и енам
kk
Разумно. То есть Если ошибка сводится к коду 400 - возвращать не Error а HttpException?
🏡K
хттп еррор вообще не нужно возвращать, кастомные возвращай и превращай их во что хочешь через фильтр ошибок
D
Делаете ли вы разные типы ошибок для обработки результата работы сервиса в контроллере, или высылаете готовый ответ из сервиса?
Перехватываю эксепшн из сервиса в контроллере и выбрасываю HttpException с нужным статус-кодом
kk
Перехватываю эксепшн из сервиса в контроллере и выбрасываю HttpException с нужным статус-кодом
Вот вот.
🏡K
лучше фильтр для модуля делать который это конвертнет
kk
хттп еррор вообще не нужно возвращать, кастомные возвращай и превращай их во что хочешь через фильтр ошибок
Фильтр ошибок? Глобальный pipe?
🏡K
чтобы в контроллере мапингом не заниматся
🏡K
kk
Ок, посмотрю на этот способ
kk
D
Я примерно вот так делаю:
Ну и также есть ExceptionFilter, который умеет переводить ошибки, а если исключение не HttpException, то формирует общую ошибку типа "что-то пошло не так"
@UseGuards(JwtAuthOptionalGuard)
@UsePipes(new JsonValidationPipe())
@HttpCode(201)
@Post('/nicknames')
async nicknamesOffer(@Body() dto: NicknameOfferDto, @Req() request): Promise<HttpResponse<null, null>> {
try {
await this.commonService.saveNickname(dto, Status.draft, request.user?request.user.id:null);
} catch (AlreadyExistsException) {
throw new HttpException(ResponseFactory.error({ nickname: 'nicknameAlreadyExists' }), HttpStatus.BAD_REQUEST)
}
return ResponseFactory.success(null);
}
Ну и также есть ExceptionFilter, который умеет переводить ошибки, а если исключение не HttpException, то формирует общую ошибку типа "что-то пошло не так"
🏡K
сделал бы кастомную ошибку для сервиса
new UserError(UserErrorEnum.AlreadyExists,'nicknameAlreadyExists')
new UserError(UserErrorEnum.AlreadyExists,'nicknameAlreadyExists')
🏡K
и фильтр для модуля юзер которй конвертнет и в контроллере ничего бы вообще не было
🏡K
@UseGuards(JwtAuthOptionalGuard)
@UsePipes(new JsonValidationPipe())
@HttpCode(201)
@Post('/nicknames')
nicknamesOffer(@Body() dto: NicknameOfferDto, @Req() request): Observable<NicknameOffer> {
return from(this.commonService.saveNickname(dto, Status.draft, request.user?request.user.id:null));
}
D
Имеешь в виду, чтобы в контроллере не ловить? Тоже вариант. Но я считаю, что в контроллере ловить ошибки сервисов - хорошая практика. По крайней мере, в том смысле, что я сразу вижу, какие внештатные ситуации обрабатываются
D
Я имею в виду не совсем внештатные типа 500-й, а естественные ошибки, как, например, в моём случае, когда юзер пытается предложить кличку собаки, которая у меня уже есть
D
Ну и, кстати, если я в фильтре буду ловить все подобные исключения, то он ведь сильно разрастётся?