[step5] 자동차 경주(리팩토링)

src/main/kotlin/racingcar/CarApplication.kt

@@ -4,7 +4,6 @@ import racingcar.controller.RacingController
 
 private val racingController: RacingController = RacingController()
 fun main() {
-    val joinRacingCar = racingController.join()
-    val playRacingCar = racingController.play(joinRacingCar)
-    racingController.winner(playRacingCar)
+    val playRacingCar = racingController.play()
+    racingController.win(playRacingCar)
 }

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

@@ -2,23 +2,24 @@ package racingcar.controller
 
 import racingcar.domain.Cars
 import racingcar.domain.WinnerCar
-import racingcar.inputCarName
-import racingcar.inputTryCount
-import racingcar.view.printRacing
-import racingcar.view.printWinner
+import racingcar.dto.CarsDto
+import racingcar.service.RacingService
+import racingcar.view.OutputView
+import racingcar.view.OutputView.printWinners
+import racingcar.view.inputCarName
 
-class RacingController {
-    fun join(): Cars = Cars.of(inputCarName().split(","))
-
-    fun play(racingCars: Cars): Cars {
-        repeat(inputTryCount()) {
-            racingCars.move()
-            printRacing(racingCars)
-        }
-        return racingCars
+class RacingController(
+    private val racingService: RacingService = RacingService()
+) {
+    fun play(): CarsDto {
+        val snapshots = racingService.play(join())
+        OutputView.printRacing(snapshots)
+        return snapshots.last()
     }
 
-    fun winner(playRacingCar: Cars) {
-        printWinner(WinnerCar.from(playRacingCar))
+    private fun join(): Cars = Cars.of(inputCarName())
+
+    fun win(playRacingCar: CarsDto) {
+        printWinners(WinnerCar.from(playRacingCar))
     }
 }

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

@@ -1,5 +1,7 @@
 package racingcar.domain
 
+import racingcar.dto.CarDto
+
 class Car(val name: String) {
     var position: Int = DEFAULT_POSITION
         private set
@@ -33,16 +35,17 @@ class Cars(cars: List<Car>) {
         }
     }
 
-    fun maxOfPositions(): Int {
-        return cars.maxOf { car -> car.position }
-    }
-
     companion object {
-        fun of(names: List<String>): Cars {
+        private const val RACING_CAR_DELIMITER = ","
+
+        fun of(inputName: String): Cars {
+            val names = inputName.split(RACING_CAR_DELIMITER)
             val cars = names.map {
                 Car.from(it)
             }
             return Cars(cars)
         }
     }
+
+    fun toCarDtoList(): List<CarDto> = cars.map { CarDto.from(it) }
 }

src/main/kotlin/racingcar/domain/WinnerCar.kt

@@ -1,8 +1,10 @@
 package racingcar.domain
 
+import racingcar.dto.CarDto
+import racingcar.dto.CarsDto
+
 object WinnerCar {
-    fun from(racingCars: Cars): List<Car> {
-        return racingCars.cars
-            .filter { it.position == racingCars.maxOfPositions() }
+    fun from(racingCars: CarsDto): List<CarDto> {
+        return racingCars.contentsFilter { it -> it.position == racingCars.maxOfPositions() }
     }
 }

src/main/kotlin/racingcar/dto/CarDto.kt

@@ -0,0 +1,34 @@
+package racingcar.dto
+
+import racingcar.domain.Car
+
+data class CarDto(val name: String, val position: Int) {
+    companion object {
+        fun from(car: Car): CarDto {
+            return CarDto(
+                name = car.name,
+                position = car.position
+            )
+        }
+    }
+}
+
+data class CarsDto(val carsDto: List<CarDto>) {
+    fun racingTrace() = buildString {
+        carsDto.forEach {
+            appendLine("${it.name} : ${RACING_COURSE.repeat(maxOf(0, it.position))}")
+        }
+    }
+
+    fun contentsFilter(predicate: (CarDto) -> Boolean): List<CarDto> {
+        return carsDto.filter(predicate)
+    }
+
+    fun maxOfPositions(): Int {
+        return carsDto.maxOf { car -> car.position }
+    }
+
+    companion object {
+        private const val RACING_COURSE = "-"
+    }
+}

src/main/kotlin/racingcar/service/RacingService.kt

@@ -0,0 +1,16 @@
+package racingcar.service
+
+import racingcar.domain.Cars
+import racingcar.dto.CarsDto
+import racingcar.view.inputTryCount
+
+class RacingService {
+    fun play(racingCars: Cars): List<CarsDto> {
+        val snapshots = mutableListOf<CarsDto>()
+        repeat(inputTryCount()) {
+            racingCars.move()
+            snapshots.add(CarsDto(racingCars.toCarDtoList()))
+        }
+        return snapshots
+    }
+}

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

@@ -1,4 +1,4 @@
-package racingcar
+package racingcar.view
 
 fun inputTryCount(): Int {
     println("시도할 횟수는 몇 회인가요?")

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

@@ -1,18 +1,24 @@
 package racingcar.view
 
-import racingcar.domain.Car
-import racingcar.domain.Cars
+import racingcar.dto.CarDto
+import racingcar.dto.CarsDto
 
-fun printRacing(racingCars: Cars) {
-    racingCars.cars.forEach {
-        println("${it.name} : ${"-".repeat(maxOf(0, it.position))}")
+object OutputView {
+
+    private const val WINNER_JOIN_SEPARATOR = ","
+    fun printRacing(snapShots: List<CarsDto>) {
+        val result = buildString {
+            snapShots.forEach {
+                appendLine(it.racingTrace())
+            }
+        }
+        println(result)
     }
-    println()
-}
 
-fun printWinner(winnerCars: List<Car>): String = buildString {
-    val winnersCarNames = winnerCars
-        .joinToString(",") { it.name }
+    fun printWinners(winnerCars: List<CarDto>): String = buildString {
+        val winnersCarNames = winnerCars
+            .joinToString(WINNER_JOIN_SEPARATOR) { it.name }
 
-    println("${winnersCarNames}가 최종 우승했습니다.")
+        println("${winnersCarNames}가 최종 우승했습니다.")
+    }
 }

src/test/kotlin/racingcar/CarsTest.kt

@@ -0,0 +1,15 @@
+package racingcar
+
+import io.kotest.matchers.shouldBe
+import org.junit.jupiter.params.ParameterizedTest
+import org.junit.jupiter.params.provider.ValueSource
+import racingcar.domain.Cars
+
+class CarsTest {
+    @ValueSource(strings = ["test,123,ㅅㄷㅈ", "민균,한솔,재성"])
+    @ParameterizedTest
+    fun `각 자동차 이름을 구분자(,)로 구분한다`(param: String) {
+        val cars = Cars.of(param)
+        cars.cars shouldBe param.split(",")
+    }
+}