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.

Sign up for GitHub

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

Jump to bottom

크리스마스 프로모션 리뷰 #1

Closed
wants to merge 85 commits into from
Closed

크리스마스 프로모션 리뷰 #1

wants to merge 85 commits into from

Conversation

alstn113
Copy link
Owner

@alstn113 alstn113 commented Nov 16, 2023 •
edited
Loading

✅ 제가 리뷰 받고 싶은 부분은 다음과 같습니다.

  1. 할인과 증정을 strategy pattern을 이용해서 작성했습니다. 모든 Event는 EventResult를 반환하는데 여기에는 할인 금액과, 증정이 들어 있습니다. 다른 분들은 어떻게 이벤트를 처리했는지 궁금합니다.
image
  1. EventBenefits를 통해 이벤트 혜택 관련 정보를 생성했습니다. 여기 너무 많은 정보가 담겨있는 것 같아서 다른 분들의 의견이 궁금합니다.
image

나머지 부분은 docs에서 기능 구현 목록을 확인해주시면 감사하겠습니다.

@alstn113
docs: 기능 구현 목록 초안 작성
65079b1
@alstn113
docs: 기능 구현 목록 체크 박스 추가
9e11051
@alstn113
feat: Category Enum과 Menu Class 작성
3cb5008
@alstn113
feat: Category 메뉴 이름으로 메뉴 찾는 메소드 작성 및 테스트
ccebffa
@alstn113
docs: 메뉴 카테고리 enum 구현 체크
370b0be
@alstn113
feat: Order class 작성 및 테스트
1fe8452
@alstn113
refactor: Order클래스를 OrderItem 클래스로 변경
67efbdf
@alstn113
feat: Order 일급 컬렉션 작성 및 테스트
521b3e5
@alstn113
feat: Order, OrderItem 메소드 구현 및 테스트
bb7cc13
@alstn113
docs: 기능 구현 목록 수정
c2b4bb9
@alstn113
feat: Visit Record 작성 및 예외 처리 테스트
92f1879
@alstn113
docs: 기능 구현 목록에 Customer 클래스 관련 항목 추가
66b2564
@alstn113
feat: AppException 작성
01ed9d9
@alstn113
feat: ErrorMessage, InvalidInputException 작성
f5ddd72
@alstn113
docs: 기능 구현 목록 체크
f01c9c5
@alstn113
feat: OutputView base interface
8c2034c
@alstn113
feat: InputView base interface
7876e30
@alstn113
feat: ConsoleInputView, ConsoleOutputView base 코드 작성
d93624b
@alstn113
feat: EventPlannerController base 코드 작성
2c635e1
@alstn113
feat: controller 내 프로세스 base 코드 작성
cdde6b1
@alstn113
refactor: error message enum prefix 부분 제거
66a4395
@alstn113
feat: readVisitDate 작성
a3475e5
@alstn113
feat: readOrder base code 작성
49ffedb
@alstn113
feat: OrderItem 수량, 존재 예외 처리 및 테스트
f9b2afd
@alstn113
feat: readOrder 예외 처리 작성
1aa4d9c
@alstn113
feat: ConsoleInputView 구현
8f384d7
@alstn113
refactor: output order details 수정
48902a7
@alstn113
feat: 중복된 메뉴가 있을 경우 예외 처리 및 테스트
9c7db54
@alstn113
refactor: displayTotalPriceBeforeDiscount 파라미터 int로 변경
5c65e62
@alstn113
feat: DiscountStrategy, WeekdayDiscountStrategy 작성
2806990
@alstn113
refactor: totalGiftPrice 순서 변경
ad3b728
@alstn113
refactor: parseToOrderItems로 메서드 명 변경
ef711d7
@alstn113
refactor: EventBenefits를 통해서 outputViewf를 분리
4eadebb
@alstn113
refactor: EventBenefits 메소드 분리
03a2340
@alstn113
refactor: validateMenu -> findMenuByMenuName으로 수정
23ce39a
@alstn113
refactor: event isApplicable 조건 추가
3880f16
@alstn113
refactor: parse 메소드 명 변경
93fca4e
@alstn113
refactor: EventGroup 일급 컬렉션 생성
03eb187
@alstn113
refactor: EventBenefits 생성자 코드 개선
8648021
@alstn113
refactor: import 순서 정렬
c26fd83
@alstn113
refactor: VisitDateTest 리팩토링
d3e1534
@alstn113
refactor: BadgeTest 개선
3960c26
@alstn113
refactor: CategoryTest 개선
7b30e39
@alstn113
refactor: MenuTest 개선
2423568
@alstn113
refactor: OrderItemTest 개선
61065b3
@alstn113
refactor: OrderTest 개선
ad5ea55
@alstn113
refactor: EventStrategyTest 테스트 개선
ec03aa7
@alstn113
refactor: EventResultTest 개선
0751349
@alstn113
refactor: WeekdayDiscountStrategyTest 개선
ee72c45
@alstn113
refactor: SpecialDiscountStrategyTest 개선
0c23351
@alstn113
refactor: GiftEventStrategyTest 개선
da9d3cd
@alstn113
refactor: ChristmasDdayDiscountStrategyTest 개선
88ce18b
@alstn113
refactor: InputUtilTest 개선
51592f3
@alstn113
refactor: OrderItem의 equals, hashCode Override후 테스트 코드 수정
285c23b
@alstn113
refactor: 기능 구현 목록 다시 작성
ad51b25
@alstn113
refactor: EventGroup private final 붙이기
5cfe9cf
pricelees
pricelees reviewed Nov 16, 2023
View reviewed changes
Copy link

@pricelees pricelees left a comment

Choose a reason for hiding this comment

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

상속을 활용해서 할인 가능 여부를 깔끔하게 처리하신 부분이 인상깊었습니다!

4주차 고생많으셨고, 앞으로의 여정도 응원하겠습니다!!

src/main/java/christmas/controller/EventPlannerController.java Show resolved Hide resolved
src/main/java/christmas/domain/Badge.java Show resolved Hide resolved
src/main/java/christmas/domain/VisitDate.java Show resolved Hide resolved
src/main/java/christmas/domain/event/ChristmasDdayDiscountStrategy.java Show resolved Hide resolved
src/main/java/christmas/domain/event/EventResult.java Show resolved Hide resolved
src/main/java/christmas/view/console/ConsoleInputView.java Show resolved Hide resolved
src/main/java/christmas/view/console/ConsoleOutputView.java Show resolved Hide resolved
src/main/java/christmas/view/util/InputUtil.java Show resolved Hide resolved
src/main/java/christmas/domain/EventBenefits.java Show resolved Hide resolved
src/main/java/christmas/domain/EventBenefits.java Show resolved Hide resolved
dgh06175
dgh06175 reviewed Nov 17, 2023
View reviewed changes
Copy link

@dgh06175 dgh06175 left a comment

Choose a reason for hiding this comment

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

코드가 정말 꼼꼼하고 의문이 생기다가도 읽다보면 이해가 되는 코드였습니다.
최종합격 응원하겠습니다. 프리코스 기간동안 코드 보면서 많이 배워갑니다 ㅎㅎ

src/main/java/christmas/Application.java Show resolved Hide resolved
src/main/java/christmas/view/console/ConsoleOutputView.java Show resolved Hide resolved
src/main/java/christmas/domain/VisitDate.java Show resolved Hide resolved
src/main/java/christmas/domain/EventBenefits.java Show resolved Hide resolved
src/main/java/christmas/domain/event/EventResult.java Show resolved Hide resolved
src/main/java/christmas/domain/event/GiftEventStrategy.java Show resolved Hide resolved
src/main/java/christmas/domain/event/WeekdayDiscountStrategy.java Show resolved Hide resolved
src/main/java/christmas/domain/order/Category.java Show resolved Hide resolved
src/main/java/christmas/view/util/InputUtil.java Show resolved Hide resolved
src/test/java/christmas/domain/order/OrderTest.java Show resolved Hide resolved
@alstn113 alstn113 deleted the branch alstn113 February 11, 2024 07:53
@alstn113 alstn113 closed this Feb 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dgh06175 dgh06175 dgh06175 left review comments

@pricelees pricelees pricelees left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@alstn113 @dgh06175 @pricelees