[둘리] 2단계 로또 제출합니다. #28
Conversation
src/main/kotlin/domain/Count.kt
Outdated
|
|
||
| fun toInt() = count | ||
|
|
||
| operator fun times(number: Int): Int = number * count |
There was a problem hiding this comment.
각각의 원시값 포장 클래스에 operator overloading 활용해주시는 시도 좋습니다 👍
다만, operator overloading 의 리턴 타입은 감싸고 있는 원시값이 아닌 해당 클래스로 정의해보는 것은 어떨까요?
Count * Count 를 하였는데 Count 가 아닌 원시값이 리턴된다면, 리턴결과를 쉽게 예측하기 어려울 것 같아요. (int 인지? float 인지?.. etc)
There was a problem hiding this comment.
피드백 주신대로 원시값이 아닌 해당 클래스(Count 또는 Amount)로 리턴하도록 시도해보았는데요!
기본 생성자에서 유효값 검증하는 과정이 있다보니, 연산 결과가 검증에서 걸리는 경우 문제가 생겼습니다.ㅠㅠ
ex) 구매금액을 로또 가격(1000원)으로 나누는 경우, 1000이상~100000이하가 아니라서 예외가 throw됩니다.
오버로딩을 하지 않고 함수 이름을 timesToNumber와 같이 리턴타입을 함수에 명시해주는건 어떨까요?
아니면 클래스가 갖고 있는 변수(count, amount)의 private를 풀고 controller에서 직접 꺼내서 계산하도록 하는게 좋을까요? 이 방법으로 우선 수정했는데 더 깔끔한 방법이 있을지 고민되네요..
| fun toInt() = count | ||
|
|
||
| operator fun times(number: Int): Int = number * count | ||
|
|
There was a problem hiding this comment.
timesToNumber 와 같이 스스로 클래스가 처리할 수 있는 접근 방식도 좋은 것 같아요!
반면 controller 의 경우 view 와 model 을 중계하는 로직 외에는 최대한 안넣는 방향으로 해보는건 어떨까요? controller 가 비즈니스 로직에 대해 알고 있는 영역이 많아질수록 model 과의 경계가 희미해질 수 있을 것 같아요.
There was a problem hiding this comment.
추가로,
require((amount / LOTTO_PRICE).toInt() >= count) { ERROR_NUMBER_RANGE.format(count) }
이 검증이 count 내부에서 꼭 진행되어야하는 것인지 고민해봐도 좋을 것 같아요.
count 에 넘어오는 실제 value 에 대한 검증과 count 와 연관된 검증을 구분해볼 수 있지 않을까요? :)
| package domain | ||
|
|
||
| class LottoCalculator(private val price: Int) { | ||
| fun getAutoCount(amount: Amount, manualCount: Count): Count { |
안녕하세요, 리뷰어님! 수동 기능을 추가하여 코드를 완성했습니다.
피드백 부탁드립니다!