🚀 2단계 - 리팩터링(메뉴)

README.md

@@ -13,20 +13,20 @@ docker compose -p kitchenpos up -d
 
 - 상품을 등록할 수 있다.
 - 상품의 가격이 올바르지 않으면 등록할 수 없다.
-  - 상품의 가격은 0원 이상이어야 한다.
+    - 상품의 가격은 0원 이상이어야 한다.
 - 상품의 이름이 올바르지 않으면 등록할 수 없다.
-  - 상품의 이름에는 비속어가 포함될 수 없다.
+    - 상품의 이름에는 비속어가 포함될 수 없다.
 - 상품의 가격을 변경할 수 있다.
 - 상품의 가격이 올바르지 않으면 변경할 수 없다.
-  - 상품의 가격은 0원 이상이어야 한다.
+    - 상품의 가격은 0원 이상이어야 한다.
 - 상품의 가격이 변경될 때 메뉴의 가격이 메뉴에 속한 상품 금액의 합보다 크면 메뉴가 숨겨진다.
 - 상품의 목록을 조회할 수 있다.
 
 ### 메뉴 그룹
 
 - 메뉴 그룹을 등록할 수 있다.
 - 메뉴 그룹의 이름이 올바르지 않으면 등록할 수 없다.
-  - 메뉴 그룹의 이름은 비워 둘 수 없다.
+    - 메뉴 그룹의 이름은 비워 둘 수 없다.
 - 메뉴 그룹의 목록을 조회할 수 있다.
 
 ### 메뉴
@@ -35,14 +35,14 @@ docker compose -p kitchenpos up -d
 - 상품이 없으면 등록할 수 없다.
 - 메뉴에 속한 상품의 수량은 0 이상이어야 한다.
 - 메뉴의 가격이 올바르지 않으면 등록할 수 없다.
-  - 메뉴의 가격은 0원 이상이어야 한다.
+    - 메뉴의 가격은 0원 이상이어야 한다.
 - 메뉴에 속한 상품 금액의 합은 메뉴의 가격보다 크거나 같아야 한다.
 - 메뉴는 특정 메뉴 그룹에 속해야 한다.
 - 메뉴의 이름이 올바르지 않으면 등록할 수 없다.
-  - 메뉴의 이름에는 비속어가 포함될 수 없다.
+    - 메뉴의 이름에는 비속어가 포함될 수 없다.
 - 메뉴의 가격을 변경할 수 있다.
 - 메뉴의 가격이 올바르지 않으면 변경할 수 없다.
-  - 메뉴의 가격은 0원 이상이어야 한다.
+    - 메뉴의 가격은 0원 이상이어야 한다.
 - 메뉴에 속한 상품 금액의 합은 메뉴의 가격보다 크거나 같아야 한다.
 - 메뉴를 노출할 수 있다.
 - 메뉴의 가격이 메뉴에 속한 상품 금액의 합보다 높을 경우 메뉴를 노출할 수 없다.
@@ -53,13 +53,13 @@ docker compose -p kitchenpos up -d
 
 - 주문 테이블을 등록할 수 있다.
 - 주문 테이블의 이름이 올바르지 않으면 등록할 수 없다.
-  - 주문 테이블의 이름은 비워 둘 수 없다.
+    - 주문 테이블의 이름은 비워 둘 수 없다.
 - 빈 테이블을 해지할 수 있다.
 - 빈 테이블로 설정할 수 있다.
 - 완료되지 않은 주문이 있는 주문 테이블은 빈 테이블로 설정할 수 없다.
 - 방문한 손님 수를 변경할 수 있다.
 - 방문한 손님 수가 올바르지 않으면 변경할 수 없다.
-  - 방문한 손님 수는 0 이상이어야 한다.
+    - 방문한 손님 수는 0 이상이어야 한다.
 - 빈 테이블은 방문한 손님 수를 변경할 수 없다.
 - 주문 테이블의 목록을 조회할 수 있다.
 
@@ -73,7 +73,7 @@ docker compose -p kitchenpos up -d
 - 매장 주문은 주문 항목의 수량이 0 미만일 수 있다.
 - 매장 주문을 제외한 주문의 경우 주문 항목의 수량은 0 이상이어야 한다.
 - 배달 주소가 올바르지 않으면 배달 주문을 등록할 수 없다.
-  - 배달 주소는 비워 둘 수 없다.
+    - 배달 주소는 비워 둘 수 없다.
 - 빈 테이블에는 매장 주문을 등록할 수 없다.
 - 숨겨진 메뉴는 주문할 수 없다.
 - 주문한 메뉴의 가격은 실제 메뉴 가격과 일치해야 한다.

src/main/java/kitchenpos/menus/application/MenuGroupService.java

@@ -2,6 +2,7 @@
 
 import kitchenpos.menus.domain.MenuGroup;
 import kitchenpos.menus.domain.MenuGroupRepository;
+import kitchenpos.menus.domain.dto.MenuGroupRequest;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
@@ -18,14 +19,9 @@ public MenuGroupService(final MenuGroupRepository menuGroupRepository) {
     }
 
     @Transactional
-    public MenuGroup create(final MenuGroup request) {
+    public MenuGroup create(final MenuGroupRequest request) {
         final String name = request.getName();
-        if (Objects.isNull(name) || name.isEmpty()) {
-            throw new IllegalArgumentException();
-        }
-        final MenuGroup menuGroup = new MenuGroup();
-        menuGroup.setId(UUID.randomUUID());
-        menuGroup.setName(name);
+        final MenuGroup menuGroup = new MenuGroup(UUID.randomUUID(), name);
         return menuGroupRepository.save(menuGroup);
     }
 

src/main/java/kitchenpos/menus/application/MenuService.java

@@ -1,14 +1,20 @@
 package kitchenpos.menus.application;
 
 import kitchenpos.menus.domain.*;
+import kitchenpos.menus.domain.dto.MenuPriceRequest;
+import kitchenpos.menus.domain.dto.MenuRequest;
+import kitchenpos.menus.domain.dto.MenuRequest.MenuProductRequest;
 import kitchenpos.products.domain.Product;
 import kitchenpos.products.domain.ProductRepository;
 import kitchenpos.products.infra.PurgomalumClient;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
 import java.math.BigDecimal;
-import java.util.*;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.NoSuchElementException;
+import java.util.UUID;
 import java.util.stream.Collectors;
 
 @Service
@@ -19,10 +25,10 @@ public class MenuService {
     private final PurgomalumClient purgomalumClient;
 
     public MenuService(
-        final MenuRepository menuRepository,
-        final MenuGroupRepository menuGroupRepository,
-        final ProductRepository productRepository,
-        final PurgomalumClient purgomalumClient
+            final MenuRepository menuRepository,
+            final MenuGroupRepository menuGroupRepository,
+            final ProductRepository productRepository,
+            final PurgomalumClient purgomalumClient
     ) {
         this.menuRepository = menuRepository;
         this.menuGroupRepository = menuGroupRepository;
@@ -31,107 +37,74 @@ public MenuService(
     }
 
     @Transactional
-    public Menu create(final Menu request) {
-        final BigDecimal price = request.getPrice();
-        if (Objects.isNull(price) || price.compareTo(BigDecimal.ZERO) < 0) {
-            throw new IllegalArgumentException();
-        }
+    public Menu create(final MenuRequest request) {
         final MenuGroup menuGroup = menuGroupRepository.findById(request.getMenuGroupId())
-            .orElseThrow(NoSuchElementException::new);
-        final List<MenuProduct> menuProductRequests = request.getMenuProducts();
-        if (Objects.isNull(menuProductRequests) || menuProductRequests.isEmpty()) {
-            throw new IllegalArgumentException();
-        }
+                .orElseThrow(NoSuchElementException::new);
+        final List<MenuProduct> menuProducts = new ArrayList<>();
+        final Menu menu = new Menu(UUID.randomUUID(), request.getName(), request.getPrice(), menuGroup, request.isDisplayed(), menuProducts);
+
+        final List<MenuProductRequest> menuProductRequests = request.getMenuProducts();
         final List<Product> products = productRepository.findAllByIdIn(
-            menuProductRequests.stream()
-                .map(MenuProduct::getProductId)
-                .collect(Collectors.toList())
+                menuProductRequests.stream()
+                        .map(MenuProductRequest::getProductId)
+                        .collect(Collectors.toList())
         );
+
         if (products.size() != menuProductRequests.size()) {
             throw new IllegalArgumentException();
         }
-        final List<MenuProduct> menuProducts = new ArrayList<>();
-        BigDecimal sum = BigDecimal.ZERO;
-        for (final MenuProduct menuProductRequest : menuProductRequests) {
-            final long quantity = menuProductRequest.getQuantity();
-            if (quantity < 0) {
-                throw new IllegalArgumentException();
-            }
+        for (final MenuProductRequest menuProductRequest : menuProductRequests) {
             final Product product = productRepository.findById(menuProductRequest.getProductId())
-                .orElseThrow(NoSuchElementException::new);
-            sum = sum.add(
-                product.getPrice()
-                    .multiply(BigDecimal.valueOf(quantity))
-            );
-            final MenuProduct menuProduct = new MenuProduct();
-            menuProduct.setProduct(product);
-            menuProduct.setQuantity(quantity);
+                    .orElseThrow(NoSuchElementException::new);
+
+            final MenuProduct menuProduct = new MenuProduct(product.getId(), product.getPrice(), menuProductRequest.getQuantity());
             menuProducts.add(menuProduct);
         }
-        if (price.compareTo(sum) > 0) {
+
+        BigDecimal sum = menu.calculateMenuProductsPrice();
+
+        if (menu.getPrice().compareTo(sum) > 0) {
             throw new IllegalArgumentException();
         }
-        final String name = request.getName();
-        if (Objects.isNull(name) || purgomalumClient.containsProfanity(name)) {
+
+        if (purgomalumClient.containsProfanity(menu.getName())) {
             throw new IllegalArgumentException();
         }
-        final Menu menu = new Menu();
-        menu.setId(UUID.randomUUID());
-        menu.setName(name);
-        menu.setPrice(price);
-        menu.setMenuGroup(menuGroup);
-        menu.setDisplayed(request.isDisplayed());
-        menu.setMenuProducts(menuProducts);
         return menuRepository.save(menu);
     }
 
     @Transactional
-    public Menu changePrice(final UUID menuId, final Menu request) {
+    public Menu changePrice(final UUID menuId, final MenuPriceRequest request) {
         final BigDecimal price = request.getPrice();
-        if (Objects.isNull(price) || price.compareTo(BigDecimal.ZERO) < 0) {
-            throw new IllegalArgumentException();
-        }
         final Menu menu = menuRepository.findById(menuId)
-            .orElseThrow(NoSuchElementException::new);
-        BigDecimal sum = BigDecimal.ZERO;
-        for (final MenuProduct menuProduct : menu.getMenuProducts()) {
-            sum = sum.add(
-                menuProduct.getProduct()
-                    .getPrice()
-                    .multiply(BigDecimal.valueOf(menuProduct.getQuantity()))
-            );
-        }
+                .orElseThrow(NoSuchElementException::new);
+        menu.changePrice(price);
+
+        BigDecimal sum = menu.calculateMenuProductsPrice();
+
         if (price.compareTo(sum) > 0) {
             throw new IllegalArgumentException();
         }
-        menu.setPrice(price);
         return menu;
     }
 
     @Transactional
     public Menu display(final UUID menuId) {
         final Menu menu = menuRepository.findById(menuId)
-            .orElseThrow(NoSuchElementException::new);
-        BigDecimal sum = BigDecimal.ZERO;
-        for (final MenuProduct menuProduct : menu.getMenuProducts()) {
-            sum = sum.add(
-                menuProduct.getProduct()
-                    .getPrice()
-                    .multiply(BigDecimal.valueOf(menuProduct.getQuantity()))
-            );
-        }
+                .orElseThrow(NoSuchElementException::new);
+        BigDecimal sum = menu.calculateMenuProductsPrice();
         if (menu.getPrice().compareTo(sum) > 0) {
             throw new IllegalStateException();
         }
-        menu.setDisplayed(true);
+        menu.changeDisplay(true);
         return menu;
     }
 
     @Transactional
     public Menu hide(final UUID menuId) {
         final Menu menu = menuRepository.findById(menuId)
-            .orElseThrow(NoSuchElementException::new);
-        menu.setDisplayed(false);
+                .orElseThrow(NoSuchElementException::new);
+        menu.changeDisplay(false);
         return menu;
     }
 

src/main/java/kitchenpos/menus/application/ProductPriceChangeEventListener.java

@@ -0,0 +1,40 @@
+package kitchenpos.menus.application;
+
+import kitchenpos.menus.domain.Menu;
+import kitchenpos.menus.domain.MenuProduct;
+import kitchenpos.menus.domain.MenuRepository;
+import kitchenpos.products.application.ProductPriceChangeEvent;
+import org.springframework.context.ApplicationListener;
+import org.springframework.stereotype.Component;
+
+import java.math.BigDecimal;
+import java.util.List;
+
+@Component
+public class ProductPriceChangeEventListener implements ApplicationListener<ProductPriceChangeEvent> {
+
+    private final MenuRepository menuRepository;
+
+    public ProductPriceChangeEventListener(MenuRepository menuRepository) {
+        this.menuRepository = menuRepository;
+    }
+
+    @Override
+    public void onApplicationEvent(ProductPriceChangeEvent event) {
+        final List<Menu> menus = menuRepository.findAllByProductId(event.getProductId());
+        for (final Menu menu : menus) {
+            BigDecimal sum = BigDecimal.ZERO;
+            for (final MenuProduct menuProduct : menu.getMenuProducts()) {
+                menuProduct.changePrice(event.getPrice());
+                sum = sum.add(
+                        menuProduct
+                                .getPrice()
+                                .multiply(BigDecimal.valueOf(menuProduct.getQuantity()))
+                );
+            }
+            if (menu.getPrice().compareTo(sum) > 0) {
+                menu.changeDisplay(false);
+            }
+        }
+    }
+}

src/main/java/kitchenpos/menus/domain/JpaMenuRepository.java

@@ -8,7 +8,7 @@
 import java.util.UUID;
 
 public interface JpaMenuRepository extends MenuRepository, JpaRepository<Menu, UUID> {
-    @Query("select m from Menu m join m.menuProducts mp where mp.product.id = :productId")
+    @Query("select m from Menu m join m.menuProducts mp where mp.productId = :productId")
     @Override
     List<Menu> findAllByProductId(@Param("productId") UUID productId);
 }