[둘리] 코틀린 자동차 경주 제출합니다. #50
Conversation
Constants 클래스 생성
| private val outputView = OutputView() | ||
| private val inputView = InputView() |
There was a problem hiding this comment.
멤버 변수를 활용해주셨네요. 👍
멤버 변수는 클래스 내부에서 언제든지 접근해서 사용할 수 있는 장점이 있지만, 함수 호출 시 멤버 변수의 값에 따라 달라지는 단점이 있습니다.
가급적이면 멤버 변수를 사용하지 않고 코드를 개선해보면 어떨까요?
ref. https://dev.to/adammc331/the-imposters-guide-to-dependency-injection-1eak
| val cars = executeInputCarNames() | ||
| val tryNumber = executeInputTryNumber() | ||
| outputView.outputResults() | ||
| for (i in 1..tryNumber) { |
There was a problem hiding this comment.
| for (i in 1..tryNumber) { | |
| repeat(tryNumber) { |
repeat을 활용해보면 어떨까요?
| outputView.outputResults() | ||
| for (i in 1..tryNumber) { | ||
| tryMove(cars) | ||
| println() |
There was a problem hiding this comment.
println은 출력과 관련된 함수입니다.
OutputView에서 관리해보면 어떨까요?
| } | ||
|
|
||
| private fun getInputTryNumber(number: String): Int { | ||
| return try { |
There was a problem hiding this comment.
코틀린에서는 runCatching이라는 것을 지원합니다.
runCatching에 대해서 학습해보시고 적용해보면 어떨까요? (이하 관련 내용 동일)
|
|
||
| class RandomGenerator { | ||
| fun getRandomNumber(): Int { | ||
| return (0..9).random() |
There was a problem hiding this comment.
0과 9는 각각 어떤 것을 의미할까요?
숫자 0과 9와 같은 것은 "매직리터럴"이라고 합니다.
"매직리터럴"에 대해서 학습해보시고, 어떻게 개선해보면 좋을지 고민해보면 어떨까요? (이하 관련 내용 동일)
src/main/kotlin/model/Cars.kt
Outdated
| } | ||
|
|
||
| private fun mappingCars(input: String): List<Car> = input.split(",").mapIndexed { _, name -> Car(name.trim()) } | ||
| fun getCarInfo(position: Int): Pair<String, Int> = cars[position].getInfo() |
There was a problem hiding this comment.
Pair을 활용해주셨네요. 👍
Pair는 두 개의 값을 편하게 wrapping할 수 있다는 장점이 있지만, first와 second가 어떤 것을 의미하는지 알기가 어렵습니다.
Pair 대신 data class를 활용해보면 어떨까요?
src/main/kotlin/util/Constants.kt
Outdated
| class Constants { | ||
|
|
||
| companion object { |
There was a problem hiding this comment.
| class Constants { | |
| companion object { | |
| object Constants { |
클래스 내 companion object만 존재한다면, object를 활용해보면 어떨까요?
src/main/kotlin/model/Cars.kt
Outdated
| fun findWinners(): List<String> { | ||
| val equalCars = cars.groupBy({ it.getInfo().second }, { it.getInfo().first }) | ||
| return equalCars[equalCars.keys.max()]?.toList() ?: listOf() | ||
| } |
There was a problem hiding this comment.
우승자를 찾는 함수를 잘 구현해주셨네요. 👍
하지만 Cars 클래스는 자동차의 리스트를 관리하는 책임을 가지고 있는데, 우승자를 찾는 함수로 인하여 우승자를 관리하는 책임까지 가지게 됩니다.
우승자를 찾는 책임을 갖는 클래스를 만들어보면 어떨까요?
src/main/kotlin/model/Car.kt
Outdated
| fun move(random: Int) { | ||
| if (random >= 4) position++ | ||
| } |
src/test/kotlin/CarTest.kt
Outdated
| @Test | ||
| fun moveTest() { | ||
| val car = Car("dool", 0) | ||
| car.move(3) | ||
| assertThat(car.getInfo().second).isEqualTo(0) | ||
| car.move(4) | ||
| assertThat(car.getInfo().second).isEqualTo(1) | ||
| } |
There was a problem hiding this comment.
하나의 테스트 함수에 두 가지 케이스를 테스트하고 있네요.
만약 요구사항이 3 이상일 경우 전진하는 것으로 변경된다면, 테스트 코드가 반쪽은 통과하고 반쪽은 통과하지 못하게 돼요.
4 이상일 경우와 4 미만인 경우 케이스를 분리해서 테스트 코드를 작성해보면 어떨까요?
…용으로 수정, nullable 수정
|
피드백 감사합니다!! 수정하는 중 궁금증이 생겼습니다.
|
| class CarsHelper { | ||
| companion object { |
| @@ -0,0 +1,3 @@ | |||
| package model | |||
|
|
|||
| data class CarInfo(val name: String, val position: Int) | |||
There was a problem hiding this comment.
Car 클래스를 보시면, 캡슐화를 하고 있는 프로퍼티가 동일합니다.
CarInfo보다는 Car 클래스를 활용하도록 코드를 개선해보면 어떨까요?
A. 코맨트에 작성했듯이, Car 클래스를 활용해보면 좋을 것 같아요! ref. https://kotlinlang.org/docs/coding-conventions.html#names-for-backing-properties |
A. exception은 논리 오류를 뜻하며, 논리 오류는 개발자에게 버그를 알리는 방법입니다. ref. https://elizarov.medium.com/kotlin-and-exceptions-8062f589d07 |
안녕하세요, 리뷰어님! 바쁘신 와중에 제 코드를 리뷰해주셔서 감사합니다. :)