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

ChoiEungi · 2022-10-08T14:36:52Z

안녕하세요, 2단계 미션 제출드립니다. 이번 미션은 고민할 부분이 많았던 것 같네요! 저는 직접적으로 도메인 코드를 수정해가며 코드를 작성했어요 :)

메뉴서비스에서 사용되는 값 검증을 최대한 도메인에서 진행해보려 노력했던 것 같습니다! 그럼에도 메뉴서비스의 검증 로직을 도메인 레이어로 온전히 옮기지 못했습니다. 이는 맥락적으로 부자연스럽다고 느껴져서 옮기지 못했던 것 같습니다.

또한, product와 menu context의 결합을 끊기 위해서 event를 통해 분리했습니다. 이벤트 로직은 fake 객체로 검증하기 어려워서 @SpringBootTest를 이용해 통합테스트 검증을 진행했습니다.

혹시 더 좋은 제언이 있으시면 코멘트 남겨주시면 감사하겠습니다.

sah3122

메뉴 리팩터링 잘해주셨네요 💯
몇가지 고민거리를 남겨두었는데, 확인 부탁드려요 🙏

sah3122 · 2022-10-09T03:41:18Z

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

-        if (Objects.isNull(price) || price.compareTo(BigDecimal.ZERO) < 0) {
-            throw new IllegalArgumentException();
-        }
+    public Menu create(final MenuRequest request) {


미션 내용과는 조금 벗어낫지만 제가 Service Layer에 DTO를 활용하는 방법을 소개 드릴게요.
현재는 domain 하위에 dto로 정의해주셨지만 DTO를 Layer 간 구분 없이 사용할수도 있지만 Layer간 구분을 한다면
MenuRequest 객체는 UI Layer 계층에 속하는 DTO 라고 볼수 있어요. (DTO를 Layer간 구분 없이 사용한다면 어느순간 불필요한 데이터가 침범할수 있어요.)
현재 Service Layer에 상위 Layer가 침법한것을 볼수 있죠.

그로 인해 저희 팀에서는 Layer간 DTO 분리를 통해 계층을 엄격하게 나누고 있습니다.
UI Layer에서는 Request / Response 와 같은 DTO 네이밍을 사용하고, Service Layer 로 전달하기 위해선 Command 와 같은 네이밍을 사용하곤 합니다.

참고만 부탁드려요 !

sah3122 · 2022-10-09T03:48:12Z

src/main/java/kitchenpos/products/application/ProductService.java

-                menu.setDisplayed(false);
-            }
-        }
+        productPriceChangeEventPublisher.publishEvent(price, productId);


Application Event 활용 👍
개인적인 생각으로 Service Layer 에서 Event 를 발행하게 된다면 도메인 로직이 어플리케이션 레이어로 분산되는것 같아요.
JPA 기준 도메인 이벤트를 발행하는 방법에 대해서 소개 드릴게요 😄

https://www.baeldung.com/spring-data-ddd

이벤트 관련 테스트에 대해서 질문 남겨주셨는데, 저는 도메인 이벤트를 주로 사용하다 보니 정확한 답변을 남기기 어려울것 같습니다.
도메인 이벤트 발행으로 변경한다면, 도메인 메소드가 실행되고, 이벤트가 정상적으로 적재되었는지 확인할 수 있을거에요 😄

sah3122 · 2022-10-09T03:51:00Z

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

+    }
+
+    @Override
+    public void onApplicationEvent(ProductPriceChangeEvent event) {


이벤트를 활용한 느슨한 결합 좋네요 👍

sah3122 · 2022-10-09T03:54:46Z

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

+        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(


메뉴 컨텍스트에서 상품 조회를 하게된다면 컨텍스트간 침범 및 결합이 발생하였다고 볼수 있어요.
이러한 결합을 줄여볼수 있을까요 ?

sah3122 · 2022-10-09T04:03:45Z

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

    }

-    public void setPrice(final BigDecimal price) {
-        this.price = price;
+    public void changePrice(final BigDecimal price) {


변경하려는 메뉴의 가격이 상품 금액의 합보다 적은것을 항상 만족할수 있을까요 ?

ChoiEungi added 10 commits October 3, 2022 17:15

refactor: MenuGroup setter제거 및 객체를 생성자로 생성

eaedced

refactor: VO 분리 및 MenuGroupRequest 생성

8edd801

refactor: setter 제거, 객체 생성을 생성자 사용 및 MenuRequest 생성

3ef3acf

refactor: Menu name, price VO 분리

bb67269

refactor: menu_product VO 생성

91a2de3

refactor: menuDisplay event로 분리

a5c3f49

refactor: price of menuproducts 구현

47d3a4d

refactor: Quantity VO 분리

d1e16a2

feat: menu context test 작성

346af00

feat: product test 작성

2091229

sah3122 requested changes Oct 9, 2022

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

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

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

ChoiEungi commented Oct 8, 2022

sah3122 left a comment

sah3122 Oct 9, 2022

sah3122 Oct 9, 2022

sah3122 Oct 9, 2022

sah3122 Oct 9, 2022

sah3122 Oct 9, 2022

sah3122 Oct 9, 2022

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

Are you sure you want to change the base?

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

Conversation

ChoiEungi commented Oct 8, 2022

sah3122 left a comment

Choose a reason for hiding this comment

sah3122 Oct 9, 2022

Choose a reason for hiding this comment

sah3122 Oct 9, 2022

Choose a reason for hiding this comment

sah3122 Oct 9, 2022

Choose a reason for hiding this comment

sah3122 Oct 9, 2022

Choose a reason for hiding this comment

sah3122 Oct 9, 2022

Choose a reason for hiding this comment

sah3122 Oct 9, 2022

Choose a reason for hiding this comment