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

lehdqlsl · 2022-10-16T04:21:09Z

강의를 듣고나니 step1 미션을 완전 잘못했었네요!

잘 부탁드립니다.

sah3122

요구사항 구현 잘해주셧습니다 👍
몇가지 고민거리를 남겨두었는데 확인 부탁드려요 !

sah3122 · 2022-10-17T14:19:19Z

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

+        this.menuProducts = createMenuProducts;
+    }
+
+    private void checkExpensive(Price price, MenuProducts menuProducts) {


MenuProduct에 지정된 가격이 현시점에 사용가능한 가격인지 유효성 검사가 필요하지 않을까요 ?

예를 든다면 현시점에 MenuProduct는 Request 데이터에 대한 유효성 검사가 존재하지 않습니다.
Product의 가격이 변경되었을 경우 전달받는 값을 그대로 사용하기엔 무리가 있을것 같아요.
이러한 정합성을 보장하기 위해서 어떤 노력이 필요할지 고민해보세요 😄

sah3122 · 2022-10-17T14:21:03Z

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

+        this.displayed = false;
+    }
+
+    public void display() {


마찬가지로 display 시점에 Product의 가격이 변경되었을 경우 데이터의 정합성을 보장하지 못하는 구조입니다.
domain service또는 api client 를 통해 정합성을 보장해보는건 어떨까요 ?

sah3122 · 2022-10-17T14:21:26Z

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

+    }
+}
+
+class MenuProducts {


일급 컬렉션은 파일을 분리해보는것이 어떨까요 ?

lehdqlsl added 14 commits October 13, 2022 22:44

create(Menu): Menu Entity 생성

f133737

test(Menu): Menu 테스트 케이스 작성.

6408326

refactor(Menu): 가격 예외처리 추가

c4d5eb8

add(DisplayName): 값객체 선언 및 테스트코트 작성.

230639b

refactor(Menu): DisplayName 적용

693780d

create(Price): Price 값객체 생성 및 테스트코드 작성

1110f6d

refactor(Menu): Price 값 객체 적용

c3d97ff

refactor(MenuTest): Price 값 객체 적용

1163d0e

refactor(DisplayName): 클래스명 변경

4ffcb62

add(MenuProduct): MenuProduct 도메인 생성 및 적용.

f472be6

fix(MenuProducts): 기존 일급컬렉션 패키지 이동

5247ef0

refactor(MenuProducts): 일급컬렉션 리팩토링

ed3ac72

refactor(Menu): displayed 추가 및 가격 수정 메소드 추가.

eae5bdc

test(Menu): 테스트코드 작성.

757cf97

sah3122 requested changes Oct 17, 2022

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

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

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

lehdqlsl commented Oct 16, 2022

sah3122 left a comment

sah3122 Oct 17, 2022

sah3122 Oct 17, 2022

sah3122 Oct 17, 2022

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

Are you sure you want to change the base?

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

Conversation

lehdqlsl commented Oct 16, 2022

sah3122 left a comment

Choose a reason for hiding this comment

sah3122 Oct 17, 2022

Choose a reason for hiding this comment

sah3122 Oct 17, 2022

Choose a reason for hiding this comment

sah3122 Oct 17, 2022

Choose a reason for hiding this comment