2단계 - 요구사항 정리#861
Open
YounghoonKwon wants to merge 1 commit intonext-step:younghoonkwonfrom
Open
Conversation
hyunssooo
requested changes
Feb 18, 2025
hyunssooo
left a comment
hyunssooo
left a comment
There was a problem hiding this comment.
안녕하세요. 영훈님 :)
몇 가지 코멘트 남겼습니다.
함께 고민해 보면 좋을 것 같아요!
Comment on lines
+19
to
+21
| - 메뉴의 이름에 **비속어가 포함되면 예외가 발생한다**. | ||
| - [ ] 사용자는 **메뉴의 가격을 변경할 수 있다**. | ||
| - 변경하려는 가격이 **메뉴를 구성하는 제품 가격 합보다 크면 예외가 발생한다**. |
There was a problem hiding this comment.
요구 사항은 개발자가 작성하는 것이 아닙니다. 그리고 모든 직군이 읽을 것 같은데요. 예외가 발생한다. 보다는 메뉴의 이름에는 비속어가 포함될 수 없다. 와 같은 문장으로 다듬어 보는 것은 어떨까요?
Comment on lines
+31
to
+32
| - 주문은 **배달, 포장, 매장 식사 타입을 가진다**. | ||
| - 주문 항목은 **최소 1개 이상의 메뉴를 포함해야 한다**. |
There was a problem hiding this comment.
final long quantity = orderLineItemRequest.getQuantity();
if (type != OrderType.EAT_IN) {
if (quantity < 0) {
throw new IllegalArgumentException();
}
}이 부분도 요구 사항에 정리되면 어떨까요?
Comment on lines
+36
to
+40
| - [ ] 사용자는 **주문을 수락할 수 있다**. | ||
| - 주문이 **대기 상태(WAITING)** 인 경우만 수락할 수 있다. | ||
| - 배달 주문의 경우 **배달 요청이 전달된다**. | ||
| - [ ] 사용자는 **주문을 제공(서빙)할 수 있다**. | ||
| - 주문이 **승인 상태(ACCEPTED)** 인 경우만 제공할 수 있다. |
There was a problem hiding this comment.
수락과 승인 이라는 단어가 같은 뜻으로 사용되는 것 같은데 이 부분은 통일해도 괜찮지 않을까요?
Comment on lines
+72
to
+74
| - 가격은 **0 이상이어야 한다**. | ||
| - [ ] 사용자는 **제품의 가격을 변경할 수 있다**. | ||
| - 가격은 **0 이상이어야 한다**. |
Comment on lines
+79
to
+92
| --- | ||
|
|
||
| ### **5. 외부 API 연동** | ||
| - [ ] 시스템은 **Purgomalum API**를 사용하여 **비속어 검열을 수행한다**. | ||
| - 메뉴 또는 제품 이름에 비속어가 포함된 경우 **예외를 발생시킨다**. | ||
| - [ ] 시스템은 **배달 서비스(Kitchenriders API)** 와 연동하여 **배달 요청을 전송한다**. | ||
| - 배달 주문이 수락될 경우 **배달 요청이 자동으로 전달된다**. | ||
|
|
||
| --- | ||
|
|
||
| ### **6. 데이터 관리** | ||
| - [ ] **애플리케이션이 실행되면 기본 데이터가 삽입된다** (`V2__Insert_default_data.sql`). | ||
| - [ ] **애플리케이션이 처음 실행될 때 테이블이 초기화된다** (`V1__Initialize_project_tables.sql`). | ||
|
|
There was a problem hiding this comment.
요구 사항에 api 연동, 데이터 관리에 대한 내용이 필요할까요?
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
감사합니다