[Step4] 4단계 - 로또(수동) PR 요청드립니다. #552
Conversation
- LottoRank의 기본값 설정
- companion object 함수에서 일반함수로
- 함수 실행할때마다 생성되는 객체를 생성자로
- 지불 금액에 대한 검증 코드 추가
integer 타입만 다루고 있어 이름 변경
모든 원시값 포장을 잘못이해하여 생서한 클래스들 제거
+ 수익률 오류 수정 + Payment 클래스의 프로퍼티명 수정
payment(지불금액)이 로또를 구매할 때 변경돼야하기 때문에 lottoShop에서 payment를 가지는 구조로 변경
|
남재님 안녕하세요! 😃 추가 수정된 내용도 있으니 함께 리뷰 부탁드립니다! |
| import io.kotest.core.spec.style.StringSpec | ||
| import io.kotest.matchers.shouldBe | ||
|
|
||
| class PaymentTest : StringSpec({ |
There was a problem hiding this comment.
리팩터링 과정에서 테스트가 꺠졌네요!
테스트 코드도 전체적으로 돌려봐주세요!
| val inputLuckyNumbers = inputView.inputLuckyNumbers() | ||
| val inputBonusNumber = inputView.inputBonusNumber() | ||
|
|
||
| val lottoResultService = LottoResultService(LuckyNumbers(inputLuckyNumbers, inputBonusNumber)) | ||
| val lottoResultService = LottoResultService(Lotto(inputLuckyNumbers.integerNumberList.map { LottoNumber(it) }), LottoNumber(inputBonusNumber)) |
There was a problem hiding this comment.
죄송합니다 🙏 이전 미션 코멘트에서
커뮤니케이션 미스가 있었던거 같아요 ㅠㅠ!
luckyNumbers는 lotto 클래스와 동일한 구조인거 같아요!
당첨번호의 luckyNumbers는 lotto 번호라고 보아도 되지않을가요?
LuckyNumbers 클래스가 아닌, LuckyNumbers객체의 List인 luckyNumbers 컬렉션을 의미하는 코멘트였습니다 ㅠㅠ
이전처럼 class LuckyNumbers( private val lotto: List<Lotto>, private val bonusNumber: Int)가 rank를 생성에 대한 책임은 이곳에 있어야하는게 맞는거같네요 ㅠ,ㅠ
| @@ -1,12 +1,26 @@ | |||
| package lotto.domain | |||
|
|
|||
| class LottoResultService( | |||
| private val luckyNumbers: LuckyNumbers, | |||
| private val luckyLotto: Lotto, | |||
| private val bonusNumber: LottoNumber | |||
There was a problem hiding this comment.
LottoResultService 라는 이름에 맞게
luckyLotto, luckyNumbers가 아닌
List<LottoRank>가 생성자에 있어야하지 않을까요?
| import lotto.util.RandomNumberGenerator | ||
| import lotto.view.InputView | ||
| import lotto.view.ResultView | ||
|
|
||
| class Application { | ||
| private val inputView = InputView() | ||
| private val resultView = ResultView() | ||
| private val lottoShop = LottoShop(LottoGenerator(RandomNumberGenerator())) | ||
| private val lottoGenerator = LottoGenerator(RandomNumberGenerator()) | ||
|
|
||
| fun run() { |
There was a problem hiding this comment.
run함수가 비대해진거 같아요!
로또 구매, 로또확인, 결과출력 과 같이 함수로 분리해보면 어떨까요?
| package lotto.common | ||
|
|
||
| class IntegerNumberList( | ||
| val integerNumberList: List<Int> |
There was a problem hiding this comment.
수동 로또 구매를 위한 컬렉션이라면,
차라리 LottoTicket과 같은 네이밍은 어떨까요?
- Ticket에도 숫자가 6개 인지도 필요하긴한거같아요
| @@ -5,11 +5,10 @@ import lotto.util.NumberUtil | |||
| class LottoStatistics( | |||
There was a problem hiding this comment.
LottoStatistics 클래스는 List의 일급컬렉션이라과 봐도될거같아요!
개인적으로 LottoRanks 같은 이름이 좀더 가독성에 좋을거같단 생각이 들어요!
| val manualLottoList = lottoShop.buyManualLotto(manualNumberList) | ||
| val autoLottoList = lottoShop.buyAutoLotto() | ||
| resultView.printLottoList(manualLottoList, autoLottoList) |
There was a problem hiding this comment.
개인적으로 수동 로또를 구매하고, 자동로또를 구매하는 로직은
구매에 대한 책임이 있는 LottoShop가 수행하여도 좋을거같아요!
남재님 안녕하세요! 😃
4단계 리뷰 요청이 늦었습니다흑ㅠ
3단계 리뷰에서 주신 코멘트에 대한 내용 반영했습니다.
이번에도 잘 부탁 드리겠습니닷~! 🙇