[Step4] 로또(수동) #564
[Step4] 로또(수동) #564
Conversation
Signed-off-by: AI2팀-이정환 <eedys@ex-em.com>
# Conflicts: # src/main/kotlin/lotto/domain/Lotto.kt # src/main/kotlin/lotto/domain/LottoMachine.kt # src/main/kotlin/lotto/domain/LottoNum.kt # src/test/kotlin/lotto/LottoTest.kt
Signed-off-by: AI2팀-이정환 <eedys@ex-em.com>
Signed-off-by: AI2팀-이정환 <eedys@ex-em.com>
Signed-off-by: AI2팀-이정환 <eedys@ex-em.com>
| val numbers = splitManual.map { s -> s.toIntOrNull() ?: throw IllegalArgumentException() }.toList() | ||
| return Lotto(numbers) | ||
| } | ||
|
|
||
| } | ||
| } |
|
|
||
| fun create(num: String): Lotto { | ||
| val splitManual = num.split(", ") | ||
| val numbers = splitManual.map { s -> s.toIntOrNull() ?: throw IllegalArgumentException() }.toList() |
There was a problem hiding this comment.
val numbers = splitManual.map { s -> s.toInt()}
으로도 충반하답니다. 더 명확한 NumberFormatException가 발생하기 때문입니다.
| @@ -19,7 +19,7 @@ class LottoMachine(private val generator: Generator) { | |||
| .toList() | |||
| } | |||
|
|
|||
| fun issue(manuals: List<String>): List<Lotto> { | |||
| private fun issue(manuals: List<String>): List<Lotto> { | |||
There was a problem hiding this comment.
private 좋네요! 하지만 위에 있는 자동로또도 private으로 해야 할것 같아요!
그리고 issue가 2개가 생겼으니, 함수의 네이밍을 조금 더 특별히 줘도 괜찮을것 같네요!
| val lottos = mutableListOf<Lotto>() | ||
|
|
||
| for(manual in manuals) { |
There was a problem hiding this comment.
list를 만들고 add를 하는게 아닌, 컬렉션에서 제공하는 함수를 사용해보는건 어떨까요?
| private val value: MutableMap<Rank, MutableList<Lotto>> = mutableMapOf() | ||
|
|
||
| fun add(rank: Rank, lotto: Lotto) { | ||
| value[rank]?.add(lotto) ?: init(rank).add(lotto) | ||
| val list = value[rank] ?: init(rank) | ||
| list.add(lotto) | ||
| } | ||
|
|
||
| fun from(rank: Rank): List<Lotto> { | ||
| return value[rank]?.toList() ?: init(rank).toList() | ||
| val result = value[rank] ?: init(rank) | ||
| return result.toList() | ||
| } |
There was a problem hiding this comment.
해당 메서드를 외부에서 계속 호출해서 완전한 형태로 만드는것 보단, WinningMachine에서 반복하면서 List를 만든 후 Statistics에 전달하여 Statistics가 스스로 만들게 하는건 어떨까요?
| } | ||
|
|
||
| private fun toOneIfContainsInLotto(num: LottoNum): Int { | ||
| return if(lotto.contains(num)) 1 else 0 | ||
| } | ||
|
|
||
| private fun containsBonusNum(lotto: Lotto): Boolean { |
There was a problem hiding this comment.
해당 메서드를 correspondLottoResult 밑에 배치하는건 어떨가요? 코드는 위에서 아래로 읽기 떄문입니다.
There was a problem hiding this comment.
말씀하신대로 코드를 위에서 아래로 읽는다면,
correspondLottoResult
containsLottoCount
containsBonusNum
toOneIfContainsInLotto
순으로 메서드를 정렬해도 괜찮을까요??
|
getter를 최대한 사용하지 않으려해도 View 에서는 꺼내야하는 것 같은데 이게 맞을까요?? |
No description provided.