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
- 음수, 0 이라면 InvalidMenuPriceUpdateException 던짐 - Dynamic Update를 이용해 수정된 가격만 업데이트하는 쿼리를 보내기 위해 사용
- 음수, 혹은 양수면 InvalidMenuPriceUpdateException 발생 테스트 - 양수라면 업데이트 된 가격으로 업데이트 됐는지 확인
- vendorId : 업데이트를 시도하는 vendor의 id - menuId : 업데이트 할 menu의 id - updatePrice : 업데이트 할 가격
- 존재하는 메뉴만 업데이트 할 수 있다. - 자신의 가게만 업데이트 할 수 있다. - 가격은 양수만 가능하다.
- 테스트용 Menu를 매번 생성하면 중복코드가 많이 생기기 때문에 더미 객체 생성용 메서드 생성
- 존재하는 메뉴만 업데이트 할 수 있다. - 자신의 가게만 업데이트 할 수 있다. - 가격은 양수만 가능하다.
Closed
- 업데이트 할 price를 받고 업데이트 된 price를 준다.
- 업데이트 가격이 0일경우 400 - 업데이트 가격이 음수일경우 400 - 벤더가 아니면 401 - 내 메뉴를 수정하는것이 아니면 403
…uOwnerNotMatchException 로 수정
kimhyun5u
reviewed
Aug 18, 2024
|
저도 계속 하면서 메뉴 controller를 따로 둬야하나 싶었습니다! Menu의 가격을 수정하기 위해 storeId를 받을 필요가 없지만, Menu가 Store에 종속되기 때문에 일단은 Store에 뒀습니다! 저도 Menu에 있는게 더 깔끔하다고 생각이 들긴 합니다...!! |
그렇군요!! 어렵네요 ㅠㅠ 저는 일단 MenuApiController 만들긴 했씁니다 |
Dr-KoKo
approved these changes
Aug 18, 2024
kimhyun5u
approved these changes
Aug 18, 2024
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.
💡 다음 이슈를 해결했어요.
Issue Link - #104
💡 이슈를 처리하면서 추가된 코드가 있어요.
InvalidMenuPriceUpdateException: 업데이트할 가격이 0 또는 음수인 경우 던져지는 ExceptionMenuOwnerNotMatchException: 가게 주인이 아닌 사람이 수정할 경우 던져지는 ExceptionMenuPriceUpdateCommand: 업데이트를 하기 위해 서비스에 던져지는 RecordMenuPriceUpdateService: 실질적으로 업데이트를 수행하는 서비스 객체MenuPriceUpdateRequest: 사용자가 업데이트할 가격을 담아 보내는 request객체MenuPriceUpdateResponse: 요청이 모두 완료된 후 response될 객체💡 이런 고민을 했어요.
✅ 셀프 체크리스트