Conversation
| @Slf4j | ||
| @Transactional | ||
| @FieldDefaults(level = AccessLevel.PRIVATE, makeFinal = true) | ||
| public class CategoryServiceImpl implements CategoryService { |
There was a problem hiding this comment.
Обычно название реализующих объектов должно отражать их сущность, кем они являются. Impl не добавляет никакого контекста
| @ResponseStatus(HttpStatus.CREATED) | ||
| @PostMapping | ||
| public EventFullDto create(@PathVariable Long userId, @RequestBody @Valid NewEventDto eventDto) { | ||
| log.info("POST запрос /users/{}/events с телом {}", userId, eventDto); |
There was a problem hiding this comment.
FYI: обычно логирование http запросов можно делать на уровне фильтров
There was a problem hiding this comment.
Хорошо, получается OncePerRequestFilter и от него создать класс фильтр, а логи уровня инфо, не за бизнесс логику ручные убрать, в будущем
|
|
||
| EventFullDto getEventOfUser(Long userId, Long eventId); | ||
|
|
||
| Collection<EventShortDto> findAllByPublic(EventSearchParams params, HttpServletRequest request); |
There was a problem hiding this comment.
В сервисном слое нежелательно использовать объекты из слоя контроллера (httpRequest), они должны быть преобразованы
| eventList = eventRepository.findAllByAdmin(params.getUsers(), params.getStates(), params.getCategories(), params.getRangeStart(), params.getRangeEnd(), pageable); | ||
| } catch (Exception e) { | ||
| log.error("Ошибка при выполнении запроса к БД: ", e); | ||
| throw new RuntimeException("Ошибка при получении данных из базы данных", e); |
There was a problem hiding this comment.
RuntimeExceltion лучше не бросать, слишком общее исключение. И обработку исключений лучше делать в отдельном контроллере
2 participants
No description provided.