feat: реализовал фичу подписки и написал postman тесты#4
Conversation
| .stream().filter(request -> request.getStatus().equals(CONFIRMED)) | ||
| .count())).collect(Collectors.toList()); | ||
| List<EventShortDto> eventsToReturn; | ||
| if (sort.equals(VIEWS)) { |
There was a problem hiding this comment.
Лучше сделать наоборот - сравнение VIEVS с sort, так не понятно nullable это поле или нет. Может быть нуллпоинтер
There was a problem hiding this comment.
По идее, оно не должно прийти null, ведь у меня стоит значение по умолчанию в контроллере VIEWS
There was a problem hiding this comment.
Да, это понятно, но лучше всегда сравнивать константу со значением, чем наоборот
| request.setRequester(user); | ||
| State status = event.getRequestModeration() ? PENDING : CONFIRMED; | ||
| request.setStatus(event.getParticipantLimit() == 0 ? CONFIRMED : status); | ||
| if (request.getStatus().equals(CONFIRMED)) { |
|
|
||
| @Override | ||
| public void unfollow(Long userId, Long followerId) { | ||
| log.debug("unfollow({}, {})", userId, followerId); |
There was a problem hiding this comment.
Такие комментарии лучше не оставлять - ниже хороший пример, если пишешь на русском, то и остальные так пиши. Здесь просто выглядит вразнобой
There was a problem hiding this comment.
Мне удалить логи во всем проекте на уровне debug?
There was a problem hiding this comment.
Нет, оставь, просто текст можно немного иначе писать - подробнее. Здесь выглядит, будто лог временный для отладки
| User user = userSearch(userId); | ||
| User follower = userSearch(followerId); | ||
| if (!subscriptionRepository.existsByUserAndFollower(user, follower)) { | ||
| throw new DataViolationException("Вы не подписаны на пользователя"); |
There was a problem hiding this comment.
Хорошо, что валидация на уровне сервиса
PolnySkvorets
left a comment
There was a problem hiding this comment.
Привет! Оставил несколько комментариев, повторяется в нескольких местах. Поправь это
No description provided.