add-controllers. Ревью. Метод по получению ID для новой сущности сделать внутренним. Вынести получение пользователя и вещи с проверкой на существование в отдельный метод.

Author: walder86

src/main/java/ru/practicum/shareit/item/mapper/ItemMapper.java

@@ -17,9 +17,8 @@ public static ItemDto toItemDto(Item item) {
         );
     }
 
-    public static Item toItem(Long itemId, ItemDto itemDto, User owner, ItemRequest request) {
+    public static Item toItem(ItemDto itemDto, User owner, ItemRequest request) {
         return new Item(
-                itemId,
                 itemDto.getName(),
                 itemDto.getDescription(),
                 itemDto.getAvailable(),

src/main/java/ru/practicum/shareit/item/model/Item.java

@@ -15,4 +15,12 @@ public class Item {
     private Boolean available;
     private User owner;
     private ItemRequest request;
+
+    public Item(String name, String description, Boolean available, User owner, ItemRequest request) {
+        this.name = name;
+        this.description = description;
+        this.available = available;
+        this.owner = owner;
+        this.request = request;
+    }
 }

src/main/java/ru/practicum/shareit/item/repository/ItemRepository.java

@@ -7,8 +7,6 @@
 
 public interface ItemRepository {
 
-    List<Item> getItems();
-
     List<Item> getItemsByUserId(Long userId);
 
     Item createItem(Item item);
@@ -19,5 +17,4 @@ public interface ItemRepository {
 
     List<Item> searchItemsByText(String text);
 
-    Long getNewItemId();
 }

src/main/java/ru/practicum/shareit/item/repository/ItemRepositoryImpl.java

@@ -8,12 +8,7 @@
 @Repository
 public class ItemRepositoryImpl implements ItemRepository {
 
-    private Map<Long, Item> items = new HashMap<>();
-
-    @Override
-    public List<Item> getItems() {
-        return new ArrayList<>(items.values());
-    }
+    private final Map<Long, Item> items = new HashMap<>();
 
     @Override
     public List<Item> getItemsByUserId(Long userId) {
@@ -24,6 +19,7 @@ public List<Item> getItemsByUserId(Long userId) {
 
     @Override
     public Item createItem(Item item) {
+        item.setId(getNewItemId());
         items.put(item.getId(), item);
         return items.get(item.getId());
     }
@@ -51,10 +47,8 @@ private Boolean containsIgnoreCase(String text, String containsText) {
         return text.toLowerCase().contains(containsText.toLowerCase());
     }
 
-    @Override
-    public Long getNewItemId() {
-        return getItems().stream()
-                .map(Item::getId)
+    private Long getNewItemId() {
+        return items.keySet().stream()
                 .max(Long::compareTo)
                 .orElse(0L) + 1;
     }

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

@@ -10,7 +10,7 @@
 import ru.practicum.shareit.item.model.Item;
 import ru.practicum.shareit.item.repository.ItemRepository;
 import ru.practicum.shareit.user.model.User;
-import ru.practicum.shareit.user.repository.UserRepository;
+import ru.practicum.shareit.user.service.UserService;
 
 import java.util.ArrayList;
 import java.util.List;
@@ -22,52 +22,54 @@
 public class ItemServiceImpl implements ItemService {
 
     private final ItemRepository itemRepository;
-    private final UserRepository userRepository;
+    private final UserService userService;
 
 
     @Override
     public List<ItemDto> getItemsByUserId(Long userId) {
         log.info("Поиск вещей пользователя с ID = {}", userId);
-        userRepository.getUserById(userId)
-                .orElseThrow(() -> new NotFoundException("Пользователь с ID = " + userId + " не найден"));
+        userService.getUserWithCheck(userId);
         List<Item> itemsByUserId = itemRepository.getItemsByUserId(userId);
         return itemsByUserId.stream()
                 .map(ItemMapper::toItemDto)
                 .toList();
     }
 
+    public Item getItemWithCheck(Long itemId) {
+        return itemRepository.getItemById(itemId)
+                .orElseThrow(() -> new NotFoundException("Вещь с ID = " + itemId + " не найдена"));
+    }
+
     @Override
     public ItemDto createItem(ItemDto itemDto, Long userId) {
         log.info("Создание вещи у пользователя с ID = {}", userId);
-        User user = userRepository.getUserById(userId)
-                .orElseThrow(() -> new NotFoundException("Пользователь с ID = " + userId + " не найден"));
+        User user = userService.getUserWithCheck(userId);
         return ItemMapper.toItemDto(
                 itemRepository.createItem(
-                        ItemMapper.toItem(
-                                itemRepository.getNewItemId(), itemDto, user, null)));
+                        ItemMapper.toItem(itemDto, user, null)));
     }
 
     @Override
     public ItemDto updateItem(ItemDto itemDto, Long itemId, Long userId) {
         log.info("Обновление вещи у пользователя с ID = {}", userId);
-        Item findItem = itemRepository.getItemById(itemId)
-                .orElseThrow(() -> new NotFoundException("Вещь с ID = " + itemId + " не найдена"));
+
+        Item findItem = getItemWithCheck(itemId);
         checkUser(findItem, userId);
-        userRepository.getUserById(userId)
-                .orElseThrow(() -> new NotFoundException("Пользователь с ID = " + userId + " не найден"));
+        userService.getUserWithCheck(userId);
+
         findItem.setName(itemDto.getName() == null || itemDto.getName().isBlank() ?
                 findItem.getName() : itemDto.getName());
         findItem.setDescription(itemDto.getDescription() == null || itemDto.getDescription().isBlank() ?
                 findItem.getDescription() : itemDto.getDescription());
         findItem.setAvailable(itemDto.getAvailable() != null ? itemDto.getAvailable() : findItem.getAvailable());
+
         return ItemMapper.toItemDto(itemRepository.updateItem(findItem));
     }
 
     @Override
     public ItemDto getItemById(Long itemId) {
         log.info("Запрос вещи с ID = {}", itemId);
-        Item item = itemRepository.getItemById(itemId)
-                .orElseThrow(() -> new NotFoundException("Вещь с ID = " + itemId + " не найдена"));
+        Item item = getItemWithCheck(itemId);
         return ItemMapper.toItemDto(item);
     }
 

src/main/java/ru/practicum/shareit/user/mapper/UserMapper.java

@@ -12,9 +12,8 @@ public static UserDto toUserDto(User user) {
         );
     }
 
-    public static User toUser(UserDto user, Long userId) {
+    public static User toUser(UserDto user) {
         return new User(
-                userId,
                 user.getName(),
                 user.getEmail()
         );

src/main/java/ru/practicum/shareit/user/model/User.java

@@ -9,4 +9,9 @@ public class User {
     private Long id;
     private String name;
     private String email;
+
+    public User(String name, String email) {
+        this.name = name;
+        this.email = email;
+    }
 }

src/main/java/ru/practicum/shareit/user/repository/UserRepository.java

@@ -17,7 +17,5 @@ public interface UserRepository {
 
     void deleteUserById(Long userId);
 
-    Long getNewUserId();
-
     void checkEmail(String email);
 }

src/main/java/ru/practicum/shareit/user/repository/UserRepositoryImpl.java

@@ -9,7 +9,7 @@
 @Repository
 public class UserRepositoryImpl implements UserRepository {
 
-    private Map<Long, User> users = new HashMap<>();
+    private final Map<Long, User> users = new HashMap<>();
 
     @Override
     public List<User> getUsers() {
@@ -23,6 +23,7 @@ public Optional<User> getUserById(Long userId) {
 
     @Override
     public User createUser(User user) {
+        user.setId(getNewUserId());
         users.put(user.getId(), user);
         return users.get(user.getId());
     }
@@ -38,14 +39,6 @@ public void deleteUserById(Long userId) {
         users.remove(userId);
     }
 
-    @Override
-    public Long getNewUserId() {
-        return getUsers().stream()
-                .map(User::getId)
-                .max(Long::compareTo)
-                .orElse(0L) + 1;
-    }
-
     @Override
     public void checkEmail(String email) {
         if (getUsers().stream()
@@ -54,4 +47,10 @@ public void checkEmail(String email) {
             throw new AlreadyExistException("Пользователь с почтой " + email + " уже существует");
         }
     }
+
+    private Long getNewUserId() {
+        return users.keySet().stream()
+                .max(Long::compareTo)
+                .orElse(0L) + 1;
+    }
 }

src/main/java/ru/practicum/shareit/user/service/UserService.java

@@ -1,11 +1,14 @@
 package ru.practicum.shareit.user.service;
 
 import ru.practicum.shareit.user.dto.UserDto;
+import ru.practicum.shareit.user.model.User;
 
 public interface UserService {
 
     UserDto getUserById(Long userId);
 
+    User getUserWithCheck(Long userId);
+
     UserDto createUser(UserDto user);
 
     UserDto updateUser(UserDto user, Long userId);