[Step2] 로또 자동 #797
[Step2] 로또 자동 #797
Conversation
| } | ||
| } | ||
|
|
||
| fun main() { |
There was a problem hiding this comment.
main은 controller밖의 lottery 패키지에 있어도 되겠네요~
| LotteryGameView.printPurchaseMoneyView() | ||
| val money = readln().toInt() |
| LotteryGameView.printWinnerLotteryNumber() | ||
| val numbers = readln().replace("\\s".toRegex(), "").split(",") |
There was a problem hiding this comment.
이 부분도 InputView라는것을 만들어서 처리하면 될거 같네요. 그러면 다른 fun은 OutputView를 만들어서 처리하면 될것 같아요.
| return Lotteries.makeAutoLotteries(purchasePrice / Lottery.LOTTERY_PRICE) | ||
| } | ||
|
|
||
| fun start() { |
There was a problem hiding this comment.
start 가 purchaseAutoLotteries 를 호출하니, start 밑에 purchaseAutoLotteries 가 있으면 좋겠네요.
| import lottery.domain.WinningLottery | ||
| import lottery.view.LotteryGameView | ||
|
|
||
| class LotteryGame { |
There was a problem hiding this comment.
Controller라고 정의하신 LotteryGame에 purchaseAutoLotteries 로직이 생긴것 같아요.
LotteryGame를 LotteryController로 바꾸고 로직을 가지고 있는 LotteryGame을 domain 에 만들어볼까요?
| @@ -0,0 +1,30 @@ | |||
| package lottery.domain | |||
|
|
|||
| class Lottery(numbers: Set<LotteryNumber>) { | |||
| companion object { | ||
| private val BASE_NUMBERS = (LotteryNumber.MIN_LOTTERY_NUMBER..LotteryNumber.MAX_LOTTERY_NUMBER).toSet() | ||
| const val LOTTERY_NUMBER_SIZE = 6 | ||
| const val LOTTERY_PRICE = 1000 |
There was a problem hiding this comment.
가격에 대한 정책은 게임에 더 가까울거 같아요!
만약 Lottery에서 사용하지 않는것이면 해당 객체와 어울리지 않는 상수일 수 있어요.
| package lottery.domain | ||
|
|
||
| class LotteryRank { | ||
| val lotteriesRank = LotteryPrize.values().associateWith { RANK_DEFAULT_VALUE }.toMutableMap() |
| @@ -0,0 +1,20 @@ | |||
| package lottery.domain | |||
|
|
|||
| class LotteryRank { | |||
There was a problem hiding this comment.
컨트롤러에서 Lotteries를 반복해서 해당 객체를 만드네요. Lotteries 에 메서드를 추가하여 LotteryRank를 만들게 하는건 어떨까요?
There was a problem hiding this comment.
그러면 팩터리메서드나 생성자를 통해 List<Lottery>를 받을 수 있을것이고, plusRank나 calculateProfit 메서드도 사라질 수 있겠네요.
| class WinningLottery(numbers: Set<LotteryNumber>) { | ||
| val winningNumbers: Set<LotteryNumber> |
마지막까지 잘부탁드립니다🙇🏻