[산타] 짱구 미션 제출합니다. #16
Open
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
1jeongg
approved these changes
Mar 6, 2024
| get() = "평일 할인" | ||
|
|
||
| override fun isSatisfiedBy(orderMenu: OrderMenu): Boolean { | ||
| val days = listOf(3, 4, 5, 6, 7, 10, 11, 12, 13, 14, 17, 18, 19, 20, 21, 24, 25, 26, 27, 28, 31) |
There was a problem hiding this comment.
달이 바뀐다면 매번 다시 입력해줘야하니까 귀찮지 않을까요? 나머지를 통해 평일인지 구별하는게 좋을 것 같아요!
| import kotlin.reflect.cast | ||
|
|
||
|
|
||
| object Container { |
There was a problem hiding this comment.
오.... 사실 뭔지 모르겠지만 멋있습니다 👍👍
| fun getVisitDay(): Int { | ||
| val visitDay = (Console.readLine() | ||
| ?.toIntOrNull() | ||
| ?: throw IllegalArgumentException("숫자만 입력해 주세요.")) |
There was a problem hiding this comment.
이 class의 Exception들은 어디서 catch하나요??
| if (orders.keys.all { it.menuType == MenuType.DRINK }) throw IllegalArgumentException("$ERROR_PREFIX 음료만 주문 시, 주문할 수 없습니다.") | ||
|
|
||
|
|
||
| require(orders.values.count() < 20) { |
There was a problem hiding this comment.
문제조건에 메뉴는 한 번에 최대 20개까지만 주문할 수 있습니다. 이렇게 되어있어서 20개까지 주문 가능할 것 같아요!
|
|
||
| fun printReceipt(ordersDto: OrderMenu) { | ||
| val giveawayMenu = if (ordersDto.giveMenu != Menu.NOTHING) { | ||
| ordersDto.giveMenu.mainMenuName + " 1개" |
There was a problem hiding this comment.
giveMenu에서 증정 메뉴의 개수도 같이 관리해주면 좋을 것 같아요!
| println("\n<증정 메뉴>") | ||
| println(giveawayMenu) | ||
|
|
||
| println("\n<혜택 내역>") |
There was a problem hiding this comment.
금액이 0원인 경우에 -는 안 붙어지는 건가요? (돌려보고 싶었는데,, reflection 에러가 나네요 ㅠㅠ)
그리고 혜택 내역이 없을 경우 "없음"이 출력되어야 할 것 같아요!
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.
구현하면서 고민되었던 점
이번 미션에서는 by lazy를 많이 애용했습니다! 이유는 방문 날짜와 주문 메뉴만 안다면 할인 전 총주문 금액, 증정 메뉴 등을 알 수 있기 때문입니다. lateinit var를 사용하지 않은 이유는 미니언 선배님께서 var로 선언할 경우 언제 값이 바뀔지 몰라 긴장하게 된다고 말씀해 주셔서 입니다.
그런데 by lazy가 과연 좋은 방법인지 아직까지 잘 모르겠네요..ㅎㅎ 로직이 복잡해 진다면 다른 방법을 사용해야 할 것 같습니다. 다른 분들 의견도 궁금하네요!
또, OrderMenu 객체와 EventDiscountPolicyFactory가 서로 강하게 커플링 되어 있다고 생각이 드는데 느슨하게 할 수 있는 방법이 있을까요?
자랑스러운 점
by lazy 사용이 좋은지는 아직 잘 모르겠으나 사용해 봤을 때, 코드가 깔끔해 진 것 같습니다!
죄송한 점
테스트 코드를 못 짰습니다..!! 정말로 죄송합니다..ㅠㅠ