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

Author: walder86

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

@@ -13,7 +13,7 @@
 import ru.practicum.shareit.exception.NotAccessException;
 import ru.practicum.shareit.exception.NotFoundException;
 import ru.practicum.shareit.exception.ValidationException;
-import ru.practicum.shareit.item.dto.ItemDto;
+import ru.practicum.shareit.item.dto.ItemByIdDto;
 import ru.practicum.shareit.item.mapper.ItemMapper;
 import ru.practicum.shareit.item.model.Item;
 import ru.practicum.shareit.item.service.ItemService;
@@ -100,7 +100,7 @@ public List<BookingDtoResponse> getBookingByUser(Long bookerId, State state) {
     public List<BookingDtoResponse> getBookingByItemsUser(Long userOwnerItemId, State state) {
         log.info("Запрос всех забронированных вещей пользователя с ID = " + userOwnerItemId);
         userService.getUserWithCheck(userOwnerItemId);
-        List<Long> itemIds = itemService.getItemsByUserId(userOwnerItemId).stream().map(ItemDto::getId).toList();
+        List<Long> itemIds = itemService.getItemsByUserId(userOwnerItemId).stream().map(ItemByIdDto::getId).toList();
         return switch (state) {
             case ALL -> bookingRepository.findAllByItemIdInOrderByStartDesc(itemIds).stream()
                     .map(booking -> BookingMapper.toBookingDtoResponse(

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

@@ -19,7 +19,7 @@ public class ItemController {
     private final String userIdHeader = "X-Sharer-User-Id";
 
     @GetMapping
-    public List<ItemDto> getItemsByUserId(@RequestHeader(userIdHeader) Long userId) {
+    public List<ItemByIdDto> getItemsByUserId(@RequestHeader(userIdHeader) Long userId) {
         return itemService.getItemsByUserId(userId);
     }
 

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

@@ -5,7 +5,6 @@
 import lombok.AllArgsConstructor;
 import lombok.Data;
 import lombok.NoArgsConstructor;
-import ru.practicum.shareit.booking.dto.BookingDtoResponse;
 
 import java.util.List;
 
@@ -23,13 +22,4 @@ public class ItemDto {
 
     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/ItemService.java

@@ -8,7 +8,7 @@
 import java.util.List;
 
 public interface ItemService {
-    List<ItemDto> getItemsByUserId(Long userId);
+    List<ItemByIdDto> getItemsByUserId(Long userId);
 
     Item getItemWithCheck(Long itemId);
 

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

@@ -38,7 +38,7 @@ public class ItemServiceImpl implements ItemService {
 
 
     @Override
-    public List<ItemDto> getItemsByUserId(Long userId) {
+    public List<ItemByIdDto> getItemsByUserId(Long userId) {
         log.info("Поиск вещей пользователя с ID = {}", userId);
         User user = userService.getUserWithCheck(userId);
         List<Item> itemsByUserId = itemRepository.findAllByOwner(user);
@@ -47,8 +47,18 @@ public List<ItemDto> getItemsByUserId(Long userId) {
                         .map(Item::getId)
                         .toList());
         return itemsByUserId.stream()
-                .map(ItemMapper::toItemDto)
-                .peek(item -> item.setBookings(bookingsByIdItem.get(item.getId())))
+                .map(item -> {
+                    List<BookingDtoResponse> bookings = bookingsByIdItem.getOrDefault(item.getId(), new ArrayList<>());
+                    BookingDtoResponse lastBooking = bookings.stream()
+                            .filter(booking -> booking.getStatus().equals(Status.APPROVED))
+                            .max(Comparator.comparing(BookingDtoResponse::getEnd))
+                            .orElse(null);
+                    BookingDtoResponse nextBooking = bookings.stream()
+                            .filter(booking -> booking.getStatus().equals(Status.APPROVED))
+                            .min(Comparator.comparing(BookingDtoResponse::getStart))
+                            .orElse(null);
+                    return ItemMapper.toItemByIdDto(item, nextBooking, lastBooking);
+                })
                 .toList();
     }