Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Step2 - 모델링 하기 #496

Open
wants to merge 3 commits into
base: gugbab2
Choose a base branch
from
Open

Step2 - 모델링 하기 #496

wants to merge 3 commits into from

Conversation

gugbab2
Copy link

@gugbab2 gugbab2 commented Feb 17, 2025

안녕하세요. 리뷰어님!

pr 이 많이 늦었네요.. 그래도 미션 빼먹지 않고 진행하려 합니다.

주문 컨텍스트 별로 구분하지 않고, 일단은 하나의 컨텍스트로 모델링하였습니다.
주문 유형별 컨텍스트로 구분하는 것에 대해서 코멘트 부탁드립니다.

Copy link

@liquidjoo liquidjoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

안녕하세요 경호님 :)
미션 잘해주셨어요 👍
몇 가지 코멘트 남겼습니다~
확인하고 다시 요청해 주세요 🙇

- `Product Name`이 올바르지 않으면 등록할 수 없다.
- `Product Name`에는 `Profanity`가 포함될 수 없다.
- `Product Price`을 변경할 수 있다.
- `Product Price`이 올바르지 않으면 변경할 수 없다.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

올바르지 않은 경우가 어떤 경우일 까요?

- `Product Price`을 변경할 수 있다.
- `Product Price`이 올바르지 않으면 변경할 수 없다.
- `Product Price`은 0원 이상이어야 한다.
- `Product Price`이 변경될 때 `Menu Price`이 `Menu Product Total Price`보다 크면 `Menu Display Off` 된다.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

상품과 메뉴 상품은 같은 도메인일까요?

- 1 개 이상의 등록된 `Product`으로 `Menu`를 등록할 수 있다.
- `Product`이 없으면 등록할 수 없다.
- `Menu Product Quantity`은 0 이상이어야 한다.
- `Menu Price`이 올바르지 않으면 등록할 수 없다.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

어떤 경우가 올바르지 않은 경우인지 자세히 표현하면 좋을 것 같아요~

Comment on lines +191 to +192
- `Empty Order Table`을 해지할 수 있다.
- `Empty Order Table`로 설정할 수 있다.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Empty Order Table 을 해지하면 어떠한 상태로 될 수 있을까요?

- `Order`의 `Order Status`가 `COMPLETED`가 아닌 경우 `Order Table`은 `Empty Order Table`로 설정할 수 없다.
- `Guest Quantity`를 변경할 수 있다.
- `Guest Quantity`가 올바르지 않으면 변경할 수 없다.
- `Guest Quantity`는 0 이상이어야 한다.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Guest Quantity 가 0 이면 Empty Order Table 으로 볼 수 있지 않을까요?

Comment on lines +201 to +203
- 1개 이상의 등록된 `Menu`로 `Delivery Order`을 등록할 수 있다.
- 1개 이상의 등록된 `Menu`로 `Takeout Order`을 등록할 수 있다.
- 1개 이상의 등록된 `Menu`로 `EatIn Order`을 등록할 수 있다.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

주문의 영역을 3가지로 나누어 볼 수 있을 것 같아요!
Delivery Order, Takeout Order, EatIn Order 로 영역을 나누었을 때 각 영역별로 필요한 모델링을 만들어보면 좋을 것 같아요!

Comment on lines +213 to +225
- `Order`의 `Order Status`를 `ACCEPTED`로 변경한다.
- `Order`의 `Order Status`가 `WATING`인 경우만 `ACCEPTED`로 변경할 수 있다.
- `Delivery Order`의 `Order Status`가 `ACCEPTED` 가 되면 `Rider Client`를 호출한다.
- `Order`의 `Order Status`를 `SERVED`로 변경한다.
- `Order`의 `Order Status`가 `ACCEPTED`인 경우만 `SERVED`로 변경할 수 있다.
- `Order`의 `Order Status`를 `DELIVERING`로 변경한다.
- `Delivery Order`만 `Order Status`를 `DELIVERING`로 변경할 수 있다.
- `Order`의 `Order Status`가 `SERVED`인 경우만 `DELIVERING`로 변경할 수 있다.
- `Order`의 `Order Status`를 `DELIVERED`로 변경한다.
- `Order`의 `Order Status`가 `DELIVERING`인 경우만 `DELIVERED`로 변경할 수 있다.
- `Order`의 `Order Status`를 `COMPLETED`로 변경한다.
- `Delivery Order`의 경우 `Order Status`가 `DELIVERED`인 경우만 `COMPLETED`로 변경할 수 있다.
- `Takeout Order` 또는 `EatIn Order`의 경우 `Order Status`가 `SERVED`인 경우만 `COMPLETED`로 변경할 수 있다.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

영역별로 나누게 된다면 정리가 더 쉬워질 것 같습니다!

Comment on lines +226 to +227
- `Order Table`의 모든 `Order Status`가 `COMPLETED`가 되면 `Empty Order Table`로 설정한다.
- `Order Table`의 모든 `Order Status`가 `COMPLETED`가 아니라면 `Empty Order Table`로 설정하지 않는다.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Table 을 사용하는 경우는 어떤 영역에서 필요할까요?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants