🚀 2단계 - 모델링 하기 #478

sumiini · 2025-02-11T13:57:48Z

안녕하세요 리뷰어님!
모델링하여 리뷰 요청드립니다 🙂

kkt219a

안녕하세요 수민님! 2단계 잘 진행해주셨습니다.😊
몇가지 코멘트를 남겼는데 확인 부탁드려요!
추가로, 1단계 코멘트가 반영되지 않은 것 같아 이 부분도 함꼐 확인 부탁드립니다.
궁금하신 점 있으시면 편하게 댓글이나 DM으로 연락주세요.☺️

kkt219a · 2025-02-11T17:31:32Z

README.md

+
+### `메뉴 등록하기(create menu)` 정책
+- 하나 이상의 `상품(product)`이 포함된 경우에만 `메뉴(menu)`를 등록할 수 있다.
+- `메뉴 가격(menu price)`이 `메뉴 상품(menu product)` 가격 합보다 크거나 같아야 등록이 가능하다.


메뉴 등록 시 가격 확인에 대한 조건이 맞는지 확인 부탁드려요!

kkt219a · 2025-02-11T17:33:44Z

README.md

+---
+
+## `주문 테이블 (order table)`
+- `주문 테이블(order table)`은 `주문 테이블 상태(order table status)`, `방문 손님 수(number of guests)`를 가진다.


주문 테이블 상태(order table status)라는 용어가 새롭게 생긴 것 같아요!
필요한 부분들을 용어사전에 정의해보는 게 어떨까요?

kkt219a · 2025-02-11T17:35:46Z

README.md

+### `주문 테이블 등록하기(create order table)` 정책
+- `주문 테이블 이름(order table name)`이 비어 있지 않은 경우 `주문 테이블(order table)`을 등록할 수 있다.


속성을 정의하실 때 주문 테이블 이름이 빠진 것 같아요! 채워보면 좋을 것 같습니다 ㅎㅎ

kkt219a · 2025-02-11T17:38:16Z

README.md

+- `주문 테이블(order table)`을 사용 중 상태로 변경할 수 있다.
+
+### `주문 테이블 치우기(clear order table)` 정책
+- 완료되지 않은 `주문(order)`이 있는 경우 `주문 테이블 치우기(clear order table)`로 설정할 수 없다.


주문 테이블 치우기(clear order table)는 기능이다라고 용어사전에 명시해주셨는데,
설정한다라는 문맥이 어떤 상태를 나타내는 것 같아요 :) 어떻게 풀어볼 수 있을까요?

kkt219a · 2025-02-11T17:39:42Z

README.md

+- `주문 목록(find order list)`을 조회할 수 있다.
+
+### `주문 등록(create order)`하기 정책
+- 하나 이상의 `메뉴(menu)`가 포함된 경우에만 `주문(order)`을 등록할 수 있다.


메뉴보단 정의하신 주문항목이 더 적합할 것 같은데 어떻게 생각하실까요??
그리고 더 다양한 정책들이 있는 것 같은데 확인 부탁드릴게요!

kkt219a · 2025-02-11T17:45:07Z

README.md

+### 주문 상태 (order status) 변경 정책
+- `대기(waiting)` 상태의 주문만 `주문 수락(accepted)`할 수 있다.
+- `주문 수락(accepted)`된 주문만 `주문 제공(served)`할 수 있다.
+- `주문 제공(served)`된 주문만 `배달 중(delivering)`으로 변경할 수 있다.
+- `배달 중(delivering)`인 주문만 `배달 완료(delivered)`할 수 있다.
+- `배달 완료(delivered)`된 주문만 `주문 완료(completed)`할 수 있다.
+- `포장 주문(take out order)` 및 `매장 주문(table order)`의 경우 `주문 제공(served)`된 주문만 `주문 완료(completed)`할 수 있다.


행위별로 풀어보는 건 어떨까요?
served 상태이면 주문 유형과 상관없이 배달중으로 변경될 수 있을 것 같아요!

kkt219a · 2025-02-11T17:50:50Z

README.md

+- `배달 주문(delivery order)`은 `배달 주소(delivery address)`를 가진다.
+- `배달 대행사 호출하기(call delivery agency)`를 통해 배달을 요청할 수 있다.
+- `배달 주소(delivery address)`는 비워 둘 수 없다.


주문 등록하기의 정책에 필요한 내용들과 속성들이 혼합되어 있어서 파악하기 어려운 것 같아요.
모델링 전체적으로 속성과 행위를 기반으로 분리해보면 어떨까요?
예시와 같이 풀어볼 수 있을 것 같아요.

배달주문 속성 - 주문 유형(order type), 주문 상태(order status), 주문 항목(order item), 배달 주소(delivery address)를 가진다. -`배달 주소(delivery address)`는 비워 둘 수 없다. ... 행위 - `배달 대행사 호출하기(call delivery agency)`를 통해 배달을 요청할 수 있다. ...

kkt219a · 2025-02-11T17:52:05Z

README.md

+
+
+```mermaid
+sequenceDiagram


다이어그램을 통해 모델링 해보셨군요👍🏻👍🏻
한 눈에 파악하기 좋은 것 같아요!

hongsumin and others added 6 commits February 8, 2025 17:15

docs (kitchenpos) : 용어 사전 정리

9c59995

docs (kitchenpos) : 용어 사전 정리 수정 (페어 기반)

efe6862

docs (kitchenpos) : 용어 통일 (페어)

638b1f0

docs (kitchenpos) : 피드백 기반 용어 사전 수정

aca1874

docs (kitchenpos) : 모델링, 시컨스 다이어그램 추가

827a45b

Merge branch 'sumiini' into step2

73441c4

kkt219a requested changes Feb 11, 2025

View reviewed changes

docs (kitchenpos) : 리뷰 반영

87c895c

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 2단계 - 모델링 하기 #478

🚀 2단계 - 모델링 하기 #478

sumiini commented Feb 11, 2025

kkt219a left a comment

kkt219a Feb 11, 2025

kkt219a Feb 11, 2025

kkt219a Feb 11, 2025

kkt219a Feb 11, 2025

kkt219a Feb 11, 2025

kkt219a Feb 11, 2025

kkt219a Feb 11, 2025

kkt219a Feb 11, 2025

		### `주문 테이블 등록하기(create order table)` 정책
		- `주문 테이블 이름(order table name)`이 비어 있지 않은 경우 `주문 테이블(order table)`을 등록할 수 있다.



		```mermaid
		sequenceDiagram

🚀 2단계 - 모델링 하기 #478

Are you sure you want to change the base?

🚀 2단계 - 모델링 하기 #478

Conversation

sumiini commented Feb 11, 2025

kkt219a 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