feat: добавить main-service #5
Conversation
…запросов на мероприятия, подборки мероприятий и работы с указанными классами
| public List<EventShortDto> getEventsByUserId(@PathVariable Integer userId, | ||
| @RequestParam(required = false, defaultValue = "0") @PositiveOrZero Integer from, | ||
| @RequestParam(required = false, defaultValue = "10") @Positive Integer size) { | ||
| log.info("GET /users/{}/events PARAMS from {}, size {}", userId, from, size); |
| eventRepository.save(event); | ||
| } | ||
|
|
||
| public void getEventsStat(List<Event> events) { |
There was a problem hiding this comment.
у тебя в классах хрянятся Set из событий, а методы принимают листы. Лучше тоже поменять параметры на сет
There was a problem hiding this comment.
Давид, добрый день!
Не скорректировала одно ваше замечание, хотелы бы обсудить его.
"у тебя в классах хрянятся Set из событий, а методы принимают листы. Лучше тоже поменять параметры на сет".
Данные методы у меня в классе Statistic и вызываются в двух методах (getEventsByUsers, getEventById) класса EventServiceImp.
Также они не имеют пересечений с классами, где у меня сет.
В методе getEventsByUsers осуществляется поиск с использованием Критерия АПИ, в результате чего возвращается List result = typedQuery.getResultList(), который я сразу же передаю в методы класса Statistic (postStat - для сохранения статистики. Из него данные уходят сразу в клиент; getEventsStat - для получения данных из сервиса статистики).
В методе getEventById я достаю значение из ИвентРепозитория и также сразу передаю в методы класса Statistic (postStat, getEventStat).
В связи с этим, изменение Листа на Сет в данном случае вижу как усложнение кода.
| Location location; | ||
|
|
||
| @NotNull(message = "Information about payment of the event cannot be empty") | ||
| Boolean paid; |
There was a problem hiding this comment.
paid может иметь лишь два состояния, поэтому лучше изменить тип на примитив. Если значение не будет передано, в таком случае оно будет проинициализировано не null, а false
| @JsonFormat(pattern = DATATIMEPATTERN) | ||
| LocalDateTime publishedOn; | ||
|
|
||
| Boolean requestModeration; |
There was a problem hiding this comment.
requestModeration может иметь лишь два состояния, поэтому лучше изменить тип на примитив. Если значение не будет передано, в таком случае оно будет проинициализировано не null, а false
| @NotBlank(message = "Email cannot be empty") | ||
| @Size(min = 6, max = 254, message = "User email length must be between 6 and 254") | ||
| String email; |
No description provided.