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

jordy-torvalds · 2023-09-17T11:24:25Z

No description provided.

wlroh

원식님 안녕하세요 :)
미션 고생하셨습니다 👍 👍
코멘트 달아두었는데요, 확인부탁드려요!

wlroh · 2023-09-17T15:51:09Z

src/main/java/kitchenpos/menus/tobe/domain/ToBeMenu.java

+    private UUID id;
+
+    @Column(name = "name", nullable = false)
+    private String name;


용어사전, 모델링에서 사용한 워딩으로 통일해보는 것은 어떤가요?
이해관계자와 소통을 할 때는 displayedName 으로 소통하다가 실제 코드와 불일치로 인해 DDD 의 핵심인 전략적 설계가 희석되는 것 같아요

wlroh · 2023-09-17T15:52:20Z

README.md

@@ -27,6 +27,7 @@ docker compose -p kitchenpos up -d
 - 메뉴 그룹을 등록할 수 있다.
 - 메뉴 그룹의 이름이 올바르지 않으면 등록할 수 없다.
  - 메뉴 그룹의 이름은 비워 둘 수 없다.
+  - 메뉴 그룹의 이름에는 비속어가 포함될 수 없다.


지속적으로 요구사항 수정 👍 👍
요구사항에 변경됨에 따라 모델링도 수정해보는 것은 어떤가요?

미션 진행하시면서 지속적으로 모델링 업데이트도 부탁드려요
지금까지 미션 진행하시면서 도메인 지식이 많이 쌓이셨을 것 같은데요~
구현하신 코드와 모델링 싱크 확인 부탁드려요

wlroh · 2023-09-17T15:53:54Z

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

-        menuGroup.setId(UUID.randomUUID());
-        menuGroup.setName(name);
-        return menuGroupRepository.save(menuGroup);
+    public MenuGroupDto create(final CreateRequest request) {


dto 적용하셨네요 👍 👍

wlroh · 2023-09-17T16:32:15Z

src/main/java/kitchenpos/common/domain/AggregateRoot.java

+import java.util.ArrayList;
+import java.util.List;
+
+public interface AggregateRoot {


애그리거트 루트를 직접 인터페이스를 통해 이벤트 발행까지 만들어주셨네요 👍 👍
Spring 에서 AbstractAggregateRoot 를 제공해주고 있는 것이 있는데, 이것도 같이 확인해보시면 좋을 것 같아요

wlroh · 2023-09-17T16:32:58Z

src/main/java/kitchenpos/common/domain/AggregateRoot.java

+    @Transient
+    ThreadLocal<List<Object>> threadLocal = ThreadLocal.withInitial(ArrayList::new);
+
+    default Object register(Object event) {


인터페이스의 default 메서드가 추가된 배경은 무엇일까요?
추상클래스 대신 인터페이스를 사용하신 이유가 있으실까요? 🤔

wlroh · 2023-09-17T16:38:39Z

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

+import static kitchenpos.common.exception.KitchenPosExceptionType.BAD_REQUEST;
+
+@Service
+public class MenuProductService {


메뉴 상품 서비스를 만드신 의도가 궁금해요 🤔

wlroh · 2023-09-17T16:40:39Z

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

+        List<CreateMenuProductRequest> menuProductRequests = request.getMenuProducts();
+        CollectionUtils.requireNonEmpty(menuProductRequests, new KitchenPosException("메뉴 구성 상품이 없으므로", BAD_REQUEST));


애플리케이션 단에서 DTO 에 대한 검증을 하고 있는데요,
빈값이 아니다는 검증을 컨트롤러에서 해보는 것은 어떤가요?
spring 에서 제공하는 @Valid 를 활용해볼 수 있을 것 같아요

wlroh · 2023-09-17T16:43:36Z

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

+        productValidator.isExistProducts(
+                menuProductRequests.stream()
+                        .map(CreateMenuProductRequest::getProductId)
+                        .collect(Collectors.toList())
        );


메뉴 상품 생성하는 곳에서 검증이 있어서, 메뉴 생성하는 곳에 위임해보는 것은 어떤가요?

wlroh · 2023-09-17T16:44:45Z

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

-        return menuRepository.save(menu);
+
+        final List<MenuProduct> menuProducts = menuProductService.create(menuProductRequests);
+        menuValidator.validatePrice(price, menuProducts);


menuValidator.validatePrice(price, menuProducts);

이부분도 메뉴에서 생성자에서 검증해볼 수 있을 것 같은데 어떻게 생각하세요?

전체적으로 도메인 로직이라 생각되는 부분이 어플리케이션 계층에서 이뤄지고 있는 거 같아요
도메인 계층으로 내려보는 것 어떤가요?
그리고 도메인 객체들의 테스트 코드도 같이 작성해보는 것은 어떤가요?

조금더 설명이 필요하시면 말씀주세요!

jordy-torvalds added 13 commits September 17, 2023 16:23

feat: to-be entity 생성

7e635b3

doc(menuGroup): 요구사항에 이름 부분 수정

d7dcb5a

refactor(menuGroup): 메뉴 그룹 리팩토링

2ec3c6b

refactor(menuGroup): 패키지 구조 변경

9468aeb

refactor(menuGroup): 기존 구현체 제거 및 tobe 접두사 제거

a740671

refactor(menu): 메뉴 테스트 리팩토링

51089c9

refactor(menu): 메뉴 tobe 접두사 제거 및 패키지 제거

66cfe66

refactor(menu): 가격 검증 최적화

693182c

refactor(menuGroup): 연관관계 끊기

2d0c25a

feat(quantity): VO 생성

1102fec

feat(quantity): 완전 구현 및 적용, 테스트 작성

aae4bd0

refactor: 이벤트 구조 개선

716390f

style: 포맷 정리 및 임포트 최전화

cdcde36

wlroh requested changes Sep 18, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

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

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

jordy-torvalds commented Sep 17, 2023

wlroh left a comment

wlroh Sep 17, 2023

wlroh Sep 17, 2023

wlroh Sep 17, 2023

wlroh Sep 17, 2023

wlroh Sep 17, 2023

wlroh Sep 17, 2023

wlroh Sep 17, 2023

wlroh Sep 17, 2023

wlroh Sep 17, 2023

wlroh Sep 17, 2023

wlroh Sep 17, 2023

		List<CreateMenuProductRequest> menuProductRequests = request.getMenuProducts();
		CollectionUtils.requireNonEmpty(menuProductRequests, new KitchenPosException("메뉴 구성 상품이 없으므로", BAD_REQUEST));

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

Are you sure you want to change the base?

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

Conversation

jordy-torvalds commented Sep 17, 2023

wlroh left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment