add-bookings. Ревью. Изменить проверки времени на аннотации. Добавить проверку статуса у апи подтверждения брони. Обогатить ответ для апи по получению списка вещей у пользователя. Небольшие корректировки кода.

Author: walder86

src/main/java/ru/practicum/shareit/booking/dto/BookingDtoRequest.java

@@ -1,5 +1,7 @@
 package ru.practicum.shareit.booking.dto;
 
+import jakarta.validation.constraints.Future;
+import jakarta.validation.constraints.FutureOrPresent;
 import jakarta.validation.constraints.NotNull;
 import lombok.AllArgsConstructor;
 import lombok.Data;
@@ -11,8 +13,10 @@
 @AllArgsConstructor
 public class BookingDtoRequest {
 
+    @FutureOrPresent(message = "Дата начала бронирования не может быть раньше текущего времени")
     @NotNull(message = "Дата начала бронирования не может быть пустой")
     private LocalDateTime start;
+    @Future(message = "Дата окончания бронирования не может быть раньше текущего времени")
     @NotNull(message = "Дата окончания бронирования не может быть пустой")
     private LocalDateTime end;
     @NotNull(message = "ID вещи не может быть пустым")

src/main/java/ru/practicum/shareit/booking/service/BookingServiceImpl.java

@@ -40,7 +40,7 @@ public BookingDtoResponse getBookingById(Long userId, Long bookingId) {
         Booking booking = getBookingWithCheck(bookingId);
         checkBookingUserForGet(userId, booking.getBooker().getId(), booking.getItem().getOwner().getId());
         return BookingMapper.toBookingDtoResponse(
-                getBookingWithCheck(bookingId),
+                booking,
                 UserMapper.toUserDto(booking.getBooker()),
                 ItemMapper.toItemDto(booking.getItem()));
     }
@@ -169,33 +169,33 @@ public BookingDtoResponse updateBooking(Long userId, Long bookingId, Boolean app
         log.info("Подтверждение бронирования");
         Booking booking = getBookingWithCheck(bookingId);
         checkBookingUser(userId, booking.getItem().getOwner().getId());
+        if (!booking.getStatus().equals(Status.WAITING)) {
+            throw new ValidationException("Статус у бронирования должен быть \"В ожидании подтверждения\"");
+        }
         userService.getUserWithCheck(userId);
         if (approved) {
             booking.setStatus(Status.APPROVED);
         } else {
-            booking.setStatus(Status.APPROVED);
+            booking.setStatus(Status.REJECTED);
         }
         return BookingMapper.toBookingDtoResponse(
                 bookingRepository.save(booking),
                 UserMapper.toUserDto(booking.getBooker()),
                 ItemMapper.toItemDto(booking.getItem()));
     }
 
-    public Booking getBookingWithCheck(Long bookingId) {
+    private Booking getBookingWithCheck(Long bookingId) {
         return bookingRepository.findById(bookingId)
                 .orElseThrow(() -> new NotFoundException("Бронирование с ID = " + bookingId + " не найдено"));
     }
 
     private void checkDates(LocalDateTime start, LocalDateTime end) {
-        if (start.isBefore(LocalDateTime.now())) {
-            throw new ValidationException("Дата начала бронирования не может быть раньше текущего времени");
-        }
-        if (end.isBefore(LocalDateTime.now())) {
-            throw new ValidationException("Дата окончания бронирования не может быть раньше текущего времени");
-        }
         if (start.isEqual(end)) {
             throw new ValidationException("Дата начала и дата окончания бронирования не могут быть равны");
         }
+        if (start.isAfter(end)) {
+            throw new ValidationException("Дата начала бронирования не может быть позже даты окончания бронирования");
+        }
     }
 
     private void checkBookingUser(Long userId, Long ownerItemId) {

src/main/java/ru/practicum/shareit/item/ItemController.java

@@ -50,7 +50,7 @@ public List<ItemDto> getItemsSearchByText(@RequestParam String text) {
 
     @PostMapping("/{itemId}/comment")
     public CommentDto createCommet(
-            @RequestBody CommentDto commentDto,
+            @Valid @RequestBody CommentDto commentDto,
             @PathVariable Long itemId,
             @RequestHeader(userIdHeader) Long userId) {
         return itemService.createComment(itemId, userId, commentDto);

src/main/java/ru/practicum/shareit/item/dto/CommentDto.java

@@ -1,5 +1,6 @@
 package ru.practicum.shareit.item.dto;
 
+import jakarta.validation.constraints.NotBlank;
 import lombok.AllArgsConstructor;
 import lombok.Data;
 
@@ -9,6 +10,7 @@
 @AllArgsConstructor
 public class CommentDto {
     private Long id;
+    @NotBlank
     private String text;
     private String authorName;
     private LocalDateTime created;

src/main/java/ru/practicum/shareit/item/dto/ItemDto.java

@@ -4,11 +4,14 @@
 import jakarta.validation.constraints.NotNull;
 import lombok.AllArgsConstructor;
 import lombok.Data;
+import lombok.NoArgsConstructor;
+import ru.practicum.shareit.booking.dto.BookingDtoResponse;
 
 import java.util.List;
 
 @Data
 @AllArgsConstructor
+@NoArgsConstructor
 public class ItemDto {
     private Long id;
     @NotBlank(message = "Название не может быть пустым")
@@ -19,4 +22,14 @@ public class ItemDto {
     private Boolean available;
 
     private List<CommentDto> comments;
+
+    private List<BookingDtoResponse> bookings;
+
+    public ItemDto(Long id, String name, String description, Boolean available, List<CommentDto> comments) {
+        this.id = id;
+        this.name = name;
+        this.description = description;
+        this.available = available;
+        this.comments = comments;
+    }
 }

src/main/java/ru/practicum/shareit/item/service/ItemServiceImpl.java

@@ -24,10 +24,7 @@
 import ru.practicum.shareit.user.service.UserService;
 
 import java.time.LocalDateTime;
-import java.util.ArrayList;
-import java.util.List;
-import java.util.Objects;
-import java.util.Optional;
+import java.util.*;
 
 @Slf4j
 @Service
@@ -45,8 +42,13 @@ public List<ItemDto> getItemsByUserId(Long userId) {
         log.info("Поиск вещей пользователя с ID = {}", userId);
         User user = userService.getUserWithCheck(userId);
         List<Item> itemsByUserId = itemRepository.findAllByOwner(user);
+        Map<Long, List<BookingDtoResponse>> bookingsByIdItem = getMapItemIdAndListBooking(
+                itemsByUserId.stream()
+                        .map(Item::getId)
+                        .toList());
         return itemsByUserId.stream()
                 .map(ItemMapper::toItemDto)
+                .peek(item -> item.setBookings(bookingsByIdItem.get(item.getId())))
                 .toList();
     }
 
@@ -140,9 +142,6 @@ public CommentDto createComment(Long itemId, Long userId, CommentDto commentDto)
         User author = userService.getUserWithCheck(userId);
         Item item = getItemWithCheck(itemId);
         Comment comment = ItemMapper.toComment(commentDto, author, item);
-        comment.setItem(item);
-        comment.setAuthor(author);
-        comment.setCreated(LocalDateTime.now());
         return ItemMapper.toCommentDto(commentRepository.save(comment));
     }
 
@@ -152,4 +151,22 @@ private void checkUser(Item item, Long userIdFromRequest) {
         }
     }
 
+    private Map<Long, List<BookingDtoResponse>> getMapItemIdAndListBooking(List<Long> itemIds) {
+        Map<Long, List<BookingDtoResponse>> map = new HashMap<>();
+        List<BookingDtoResponse> bookings = bookingRepository.findAllByItemIdInOrderByStartDesc(itemIds).stream()
+                .map(booking -> BookingMapper.toBookingDtoResponse(
+                        booking,
+                        UserMapper.toUserDto(booking.getBooker()),
+                        ItemMapper.toItemDto(booking.getItem())))
+                .toList();
+        bookings.forEach(booking -> {
+            if (map.containsKey(booking.getItem().getId())) {
+                map.get(booking.getItem().getId()).add(booking);
+            } else {
+                map.put(booking.getItem().getId(), new ArrayList<>(Arrays.asList(booking)));
+            }
+        });
+        return map;
+    }
+
 }