[코드 리뷰용 PR] - 재구현 스터디

docs/README.md

@@ -0,0 +1,24 @@
+# 시스템의 핵심 기능
+
+자동차들이 여러 try 동안 전진 조건에 따라 이동하고, 최종 우승자를 얻어낸다.
+
+이 중에서도 더 핵심은 자동차들이 전진 조건에 따라 이동한다.
+
+# 구현할 기능 목록
+
+## 핵심 비즈니스
+
+* 자동차들이 이동한다.
+    * 자동차의 전진 조건을 검사한다.
+    * 자동차가 이동한다.
+* 자동차들의 위치에 따라 최종 자동차 게임의 우승자를 알아낸다.
+
+## 입출력
+* 자동차 이름을 입력받는다.
+* 시도할 횟수를 입력받는다.
+
+* 자돋차의 위치를 출력한다. 형식은 `pobi : --`
+* 최종 우승자를 안내하는 문구를 출력한다.
+* 그 외 안내 메시지, 프폼프트를 출력한다.
+
+

src/main/kotlin/racingcar/Application.kt

@@ -1,5 +1,10 @@
 package racingcar
 
+import racingcar.controller.GameController
+import racingcar.model.RandomNumberMoveStrategy
+
 fun main() {
-    // TODO: 프로그램 구현
+    val gameController = GameController(RandomNumberMoveStrategy())
+    gameController.start()
+
 }

src/main/kotlin/racingcar/controller/GameController.kt

@@ -0,0 +1,31 @@
+package racingcar.controller
+
+import racingcar.model.Cars
+import racingcar.model.MoveStrategy
+import racingcar.view.InputView
+import racingcar.view.OutputView
+
+class GameController(private val moveStrategy: MoveStrategy) {
+
+    private val inputView = InputView()
+    private val outputView = OutputView()
+
+    fun start() {
+        val cars = (inputView.readCars()).toCars()
+        val tryCount = inputView.readTryCount()
+
+        outputView.showResultHeader()
+        moveCars(tryCount, cars)
+        outputView.showTotalWinner(cars.getWinners())
+    }
+
+    private fun moveCars(tryCount: Int, cars: Cars) {
+        repeat(tryCount) {
+            cars.cars.forEach { car ->
+                moveStrategy.moveForward(car)
+            }
+            outputView.showTryResult(cars)
+        }
+    }
+
+}

src/main/kotlin/racingcar/controller/InputConverter.kt

@@ -0,0 +1,13 @@
+package racingcar.controller
+
+import racingcar.model.Cars
+
+fun String.toCars(): Cars {
+    require(this.isNotBlank()) { INPUT_CARS_BLANK }
+
+    val list = this.split(INPUT_CARS_DELIMITERS).map { it.trim() }
+    return Cars.withNames(*list.toTypedArray())
+}
+
+const val INPUT_CARS_BLANK = "경주할 자동차 이름을 입력하지 않았습니다."
+const val INPUT_CARS_DELIMITERS = ','

src/main/kotlin/racingcar/model/Car.kt

@@ -0,0 +1,24 @@
+package racingcar.model
+
+data class Car(
+    val name: String,
+    private var _position: Int = 0
+) {
+    init {
+        require(name.isNotBlank()) { BLANK_NAME }
+        require(name.length <= NAME_MAX_LENGTH) { TOO_LONG_NAME }
+    }
+
+    val position: Int
+        get() = _position
+
+    fun moveForward() {
+        _position++
+    }
+
+    companion object {
+        private const val NAME_MAX_LENGTH = 5
+        const val TOO_LONG_NAME = "자동차의 이름은 ${NAME_MAX_LENGTH}자 이하만 가능합니다."
+        const val BLANK_NAME = "자동차의 이름은 공백일 수 없습니다."
+    }
+}

src/main/kotlin/racingcar/model/Cars.kt

@@ -0,0 +1,21 @@
+package racingcar.model
+
+data class Cars(val cars: List<Car>) {
+    init {
+        require(cars.size == cars.distinct().size) { DUPLICATED_CARS }
+    }
+
+    fun getWinners(): List<Car> {
+        val maxPosition = cars.maxOf { it.position }
+        return cars.filter { it.position == maxPosition }
+    }
+
+    companion object {
+        fun withNames(vararg carNames: String): Cars {
+            val carsList = carNames.map { Car(it) }
+            return Cars(carsList)
+        }
+
+        const val DUPLICATED_CARS = "자동차에 중복된 이름이 있습니다."
+    }
+}

src/main/kotlin/racingcar/model/MoveStrategy.kt

@@ -0,0 +1,5 @@
+package racingcar.model
+
+interface MoveStrategy {
+    fun moveForward(car: Car)
+}

src/main/kotlin/racingcar/model/RandomNumberMoveStrategy.kt

@@ -0,0 +1,17 @@
+package racingcar.model
+
+import camp.nextstep.edu.missionutils.Randoms.pickNumberInRange
+
+class RandomNumberMoveStrategy : MoveStrategy {
+    override fun moveForward(car: Car) {
+        if (pickNumberInRange(RANDOM_START_RANGE, RANDOM_END_RANGE) >= RANDOM_NUMBER_CONDITION) {
+            car.moveForward()
+        }
+    }
+
+    companion object {
+        const val RANDOM_START_RANGE = 0
+        const val RANDOM_END_RANGE = 9
+        const val RANDOM_NUMBER_CONDITION = 4
+    }
+}

src/main/kotlin/racingcar/view/InputView.kt

@@ -0,0 +1,23 @@
+package racingcar.view
+
+import camp.nextstep.edu.missionutils.Console.readLine
+
+class InputView {
+    fun readCars(): String {
+        println(INPUT_CARS_GUIDE_MESSAGE)
+        return readLine()
+    }
+
+    fun readTryCount(): Int {
+        println(INPUT_TRY_COUNT_GUIDE)
+        val tryCount = readLine().toIntOrNull() ?: -1
+        require(tryCount >= 0)
+
+        return tryCount
+    }
+
+    companion object {
+        const val INPUT_CARS_GUIDE_MESSAGE = "경주할 자동차 이름을 입력하세요.(이름은 쉼표(,) 기준으로 구분)"
+        const val INPUT_TRY_COUNT_GUIDE = "시도할 횟수는 몇 회인가요?"
+    }
+}

src/main/kotlin/racingcar/view/OutputView.kt

@@ -0,0 +1,29 @@
+package racingcar.view
+
+import racingcar.model.Car
+import racingcar.model.Cars
+
+class OutputView {
+
+    fun showResultHeader() = println(TRY_RESULT_HEADER)
+
+    fun showTryResult(cars: Cars) {
+        cars.cars.forEach {
+            println("${it.name} : " + "-".repeat(it.position))
+        }
+        println()
+    }
+
+    fun showTotalWinner(winners: List<Car>) {
+        print(TOTAL_WINNER_GUIDE)
+        repeat(winners.size - 1) {
+            print("${winners[it].name}, ")
+        }
+        println(winners.last().name)
+    }
+
+    companion object {
+        const val TRY_RESULT_HEADER = "실행 결과"
+        const val TOTAL_WINNER_GUIDE = "최종 우승자 : "
+    }
+}

src/test/kotlin/racingcar/controller/InputConverterTest.kt

@@ -0,0 +1,56 @@
+package racingcar.controller
+
+import org.assertj.core.api.Assertions.assertThat
+import org.junit.jupiter.api.assertThrows
+import org.junit.jupiter.params.ParameterizedTest
+import org.junit.jupiter.params.provider.Arguments
+import org.junit.jupiter.params.provider.MethodSource
+import org.junit.jupiter.params.provider.ValueSource
+import racingcar.model.Car
+import racingcar.model.Cars
+import java.util.stream.Stream
+
+class InputConverterTest {
+
+    @ParameterizedTest
+    @ValueSource(strings = ["", " ", "\n"])
+    fun `경주할 자동차 문자열이 공백이면 예외를 던진다`(inputCars: String) {
+        val exception = assertThrows<IllegalArgumentException> { inputCars.toCars() }
+        assertThat(exception.message).isEqualTo(INPUT_CARS_BLANK)
+    }
+
+    @ParameterizedTest
+    @MethodSource("provideNormalInputCars")
+    fun `입력받은 자동차들 문자열을 Cars 로 변환한다`(inputCars: String, expectedCars: Cars) {
+        val result = inputCars.toCars()
+        assertThat(result).isEqualTo(expectedCars)
+    }
+
+    companion object {
+        @JvmStatic
+        fun provideNormalInputCars(): Stream<Arguments> = Stream.of(
+            Arguments.of(
+                "pobi ,woni,jun",
+                Cars(
+                    listOf(
+                        Car("pobi", 0),
+                        Car("woni", 0),
+                        Car("jun", 0),
+                    )
+                )
+            ),
+            Arguments.of(
+                "pobi,woni,jun,simji",
+                Cars(
+                    listOf(
+                        Car("pobi", 0),
+                        Car("woni", 0),
+                        Car("jun", 0),
+                        Car("simji", 0),
+                    )
+                )
+            ),
+        )
+    }
+
+}

src/test/kotlin/racingcar/model/CarTest.kt

@@ -0,0 +1,49 @@
+package racingcar.model
+
+import org.assertj.core.api.Assertions.assertThat
+import org.junit.jupiter.api.assertDoesNotThrow
+import org.junit.jupiter.api.assertThrows
+import org.junit.jupiter.params.ParameterizedTest
+import org.junit.jupiter.params.provider.CsvSource
+import org.junit.jupiter.params.provider.ValueSource
+import racingcar.model.Car.Companion.BLANK_NAME
+import racingcar.model.Car.Companion.TOO_LONG_NAME
+
+class CarTest {
+
+    @ParameterizedTest
+    @ValueSource(strings = ["123456", "abcdef", "심지 자동차", "πøˆoi9"])
+    fun `자동차의 이름이 5자 초과이면 예외를 던진다`(carName: String) {
+        assertThrowsWithMessage(carName, TOO_LONG_NAME)
+    }
+
+    @ParameterizedTest
+    @ValueSource(strings = ["", "  ", "\n"])
+    fun `자동차의 이름이 공백이나 빈 값이면 예외를 던진다`(carName: String) {
+        assertThrowsWithMessage(carName, BLANK_NAME)
+    }
+
+    @ParameterizedTest
+    @ValueSource(strings = ["a", " a ", "pobi", "woni", "jun", "simji"])
+    fun `정상적인 자동차의 이름은 통과한다`(carName: String) {
+        assertDoesNotThrow { Car(carName) }
+    }
+
+    @ParameterizedTest
+    @CsvSource("0, 1", "1, 2", "10, 11", "100, 101")
+    fun `자동차가 전진한다`(firstPosition: Int, expectedPosition: Int) {
+        val car = Car("pobi", firstPosition)
+        car.moveForward()
+
+        assertThat(car.position == expectedPosition)
+    }
+
+
+    private fun assertThrowsWithMessage(carName: String, message: String) {
+        val exception = assertThrows<IllegalArgumentException> {
+            Car(carName)
+        }
+        assertThat(exception.message).isEqualTo(message)
+    }
+
+}