Step2 모델링 하기 #330
Open
Step2 모델링 하기 #330
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
wlroh
requested changes
Sep 13, 2023
| * 상품(`Product`)을 전체를 조회한다. | ||
| * 상품(`Product`)을 등록한다. | ||
| * 상품(`Product`)의 가격(`price`)은 0원 이상이어야 한다. | ||
| * 상품(`Product`)의 이름(`name`)은 없거나 비속어를 사용할 수 없다. |
There was a problem hiding this comment.
비속어도 용어사전으로 정의되어 있어서 백틱(`)을 활용해 용어임을 나타내 보는 것은 어떤가요?
|
|
||
|
|
||
| ### 메뉴 | ||
| * 메뉴(`Menu`)는 이름(`name`), 가격(`price`), 메뉴구성 상품(`MenuProduct`), 메뉴그룹(`MenuGroup`), 메뉴노출 상태(`displayed`)를 가진다. |
There was a problem hiding this comment.
모델링과 용어사전은 합쳐졌을 때 굉장한 시너지를 내요.
모델링의 MenuProduct 과 용어사전의 MenuProduct 를 통일시켜보는 것은 어떤가요?
There was a problem hiding this comment.
아 혹시 메뉴(Menu) 가 용어사전의 한글명(영문명)을 뜻하는 것이 맞을까요? 🤔
|
|
||
|
|
||
|
|
||
| ### 메뉴그룹 |
| @@ -185,3 +185,130 @@ docker compose -p kitchenpos up -d | |||
|
|
|||
|
|
|||
| ## 모델링 | |||
|
|
|||
| ### 상품 | |||
| * 상품(`Product`)은 이름(`name`), 가격(`price`)를 가진다. | |||
There was a problem hiding this comment.
속성과 행위로 구분해보는 것은 어떤가요?
개행으로 구분지어도 좋을 것 같고, 소제목으로 나눠도 좋을 것 같아요.
| * 등록된 메뉴(`Menu`)이어야 한다. | ||
|
|
||
|
|
||
| ### 주문테이블 |
| * 주문(`Order`) 전체를 조회한다. | ||
|
|
||
|
|
||
| #### 매장 주문 |
There was a problem hiding this comment.
매장 주문의 Flow 도 같이 표현해보는 것은 어떤가요?
예를 들면 이렇게 적어볼 수 있겠네요.
주문타입(
Ordertype)이 매장주문(EAT_IN)인 주문(Order)은 접수중(waiting) → 접수완료(accepted) ...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
늦었지만 잘 부탁드립니다!