-
Notifications
You must be signed in to change notification settings - Fork 177
2단계 - 리팩터링(메뉴) #169
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
Open
chr0m3
wants to merge
35
commits into
next-step:chr0m3
Choose a base branch
from
chr0m3:step2
base: chr0m3
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
2단계 - 리팩터링(메뉴) #169
Changes from 1 commit
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
bde7ea3
Price에 multiply() 추가
chr0m3 86b5760
MenuGroup entity 추가
chr0m3 4022276
MenuGroupEntity 추가
chr0m3 8382a9b
MenuGroupEntityConverter 추가
chr0m3 a1644b4
MenuGroupRepository 추가
chr0m3 b84cfe6
JpaMenuGroupRepository, JpaMenuGroupDao 추가
chr0m3 eb121a6
Price.multiply() 테스트 추가
chr0m3 4e44a8f
Price에 add() 추가
chr0m3 2c52931
Price가 Comparable을 구현하도록 수정
chr0m3 0146904
MenuProductQuantity VO 추가
chr0m3 a968bc6
MenuProduct VO 추가
chr0m3 7ff9973
MenuGroup VO 추가
chr0m3 3621adb
Menu entity 추가
chr0m3 5aab011
MenuDisplayPolicy 추가
chr0m3 58bd527
MenuRepository 추가
chr0m3 0701e17
Menu의 isVisible을 displayed로 변경
chr0m3 bf9bfb1
MenuEntity 추가
chr0m3 44adb96
MenuGroup VO 삭제
chr0m3 2d16e74
MenuProductEntity 추가
chr0m3 218f580
MenuProductEntityConverter 추가
chr0m3 c346395
누락된 final 추가
chr0m3 26aa5e7
MenuEntityConverter 추가
chr0m3 ac684f6
JpaMenuRepository 추가
chr0m3 748ee71
JpaMenuRepository 리팩터링 - 중복 코드 제거
chr0m3 96fe49a
메뉴 노출 정책을 위반하는 가격 변경이 불가능하도록 수정
chr0m3 f795b5c
새로운 도메인 모델을 사용하도록 Menu 컨텍스트 수정
chr0m3 2c7035e
불필요해진 과거 코드 제거
chr0m3 a662282
CreateMenuGroupCommand DTO 추가
chr0m3 a753755
ChangeMenuPriceCommand DTO 추가
chr0m3 86dc9e6
CreateMenuCommand DTO 추가
chr0m3 81afbbc
메뉴 노출 정책을 위반하는 메뉴 생성이 불가능하도록 수정
chr0m3 bd08ccb
메뉴 생성시 정확한 상품 가격을 반영하도록 개선
chr0m3 71f200a
DB 마이그레이션
chr0m3 139c6f8
도메인 모델 수정
chr0m3 1c460a0
Converter 인터페이스 제거
chr0m3 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -19,28 +19,28 @@ public class Menu { | |
|
|
||
| private final List<MenuProduct> menuProducts; | ||
|
|
||
| private Boolean isVisible; | ||
| private Boolean displayed; | ||
|
|
||
| private Price price; | ||
|
|
||
| public Menu( | ||
| final UUID id, | ||
| final Name name, | ||
| final Boolean isVisible, | ||
| final Boolean displayed, | ||
| final Price price, | ||
| final MenuGroup menuGroup, | ||
| final List<MenuProduct> menuProducts | ||
| ) { | ||
| this.id = id; | ||
| this.name = name; | ||
| this.isVisible = isVisible; | ||
| this.displayed = displayed; | ||
| this.price = price; | ||
| this.menuGroup = menuGroup; | ||
| this.menuProducts = menuProducts; | ||
| } | ||
|
|
||
| public boolean isVisible() { | ||
| return this.isVisible; | ||
| public boolean displayed() { | ||
| return this.displayed; | ||
| } | ||
|
|
||
| public Price price() { | ||
|
|
@@ -55,11 +55,11 @@ public void display() { | |
| if (!MenuDisplayPolicy.isDisplayable(this)) { | ||
| throw new IllegalStateException("메뉴 노출 정책에 따라 이 메뉴를 노출할 수 없습니다"); | ||
| } | ||
| this.isVisible = true; | ||
| this.displayed = true; | ||
| } | ||
|
|
||
| public void hide() { | ||
| this.isVisible = false; | ||
| this.displayed = false; | ||
| } | ||
|
|
||
| public void setPrice(final Price price) { | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. setter 네이밍을 사용하기 보다는 의미있는 네이밍을 고민해보세요 😄 |
||
|
|
||
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.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
굳이
isVisible로 바꿔야만 하는 이유가 없어서 불필요한 데이터베이스 변경을 만들지 않기 위해displayed를 그대로 사용하기로 했습니다.