🚀 5단계 - 자동차 경주(리팩터링)

README.md

@@ -1,8 +1,6 @@
 
-# 요구사항 정리
 
-## Step 4.  자동차 경주(우승자)
-- [x] 게임 진행 시 자동차의 이름을 받는다.
-- [x] RacingCar 객체에 이름을 부여하는 기능을 만든다
-- [x] 우승자를 선정하는 책임을 가진 객체를 생성한다
-- [x] 게임 완료 후 우승자를 출력한다.
+##🚀 5단계 - 자동차 경주(리팩터링)
+- [x] 핵심 비지니스 로직을 가지는 객체를 domain 패키지, UI 관련한 객체를 view 패키지에 구현한다.
+- [x] 모듈 의존성에 대해 고려하며 설계한다. 
+- [x] 이전 리뷰 피드백을 반영하여 구현한다.

src/main/kotlin/misson/car/PlayGame.kt

@@ -1,5 +1,7 @@
 package misson.car
 
+import misson.car.domain.RacingCar
+import misson.car.domain.RacingCars
 import view.InputView
 import view.ResultView.printRoundResult
 import view.ResultView.printWinner
@@ -20,9 +22,9 @@ class PlayGame(private val racingCars: RacingCars, private val attempts: Int) {
     fun play() {
         repeat(attempts) {
             racingCars.moveAll { (0..9).random() }
-            printRoundResult(racingCars.getPositionsWithName())
+            printRoundResult(racingCars)
         }
 
-        printWinner(racingCars.findWinners().representWinners())
+        printWinner(racingCars.findWinners())
     }
 }

src/main/kotlin/misson/car/domain/CarPositionFormatter.kt

@@ -0,0 +1,13 @@
+package misson.car.domain
+
+class CarPositionFormatter {
+    companion object {
+        fun formatCarPosition(car: RacingCar): String {
+            return "자동차 ${car.name} : ${formatPosition(car.position)}"
+        }
+
+        fun formatPosition(position: Int): String {
+            return "-".repeat(position)
+        }
+    }
+}

src/main/kotlin/misson/car/domain/RacingCar.kt

@@ -0,0 +1,19 @@
+package misson.car.domain
+
+// 코틀린 문법에서 position을 private으로 하지 않아도 내부적으로는 getter로 접근
+class RacingCar(val name: String = "붕붕이", var position: Int = 0) {
+    init {
+        validateName(name)
+    }
+
+    private fun validateName(name: String) {
+        require(name.length <= 5) { "이름은 5자 이하만 가능합니다." }
+        require(name.isNotBlank()) { "이름은 공백일 수 없습니다." }
+    }
+
+    fun move(randomNumberGenerator: () -> Int) {
+        if (randomNumberGenerator() >= 4) {
+            position++
+        }
+    }
+}

src/main/kotlin/misson/car/domain/RacingCars.kt

@@ -0,0 +1,14 @@
+package misson.car.domain
+
+class RacingCars(val cars: List<RacingCar>) {
+    fun moveAll(randomNumberGenerator: () -> Int) {
+        cars.forEach { it.move(randomNumberGenerator) }
+    }
+
+    // 우승자를 찾는 책임 이동
+    fun findWinners(): Winners {
+        val maxPosition = cars.maxOf { it.position }
+        val winners = cars.filter { it.position == maxPosition }
+        return Winners(winners)
+    }
+}

src/main/kotlin/misson/car/domain/Winners.kt

@@ -0,0 +1,8 @@
+package misson.car.domain
+
+@JvmInline
+value class Winners(private val winners: List<RacingCar>) {
+    fun getNames(): List<String> {
+        return winners.map { it.name }
+    }
+}

src/main/kotlin/view/ResultView.kt

@@ -1,12 +1,19 @@
 package view
 
+import misson.car.domain.CarPositionFormatter
+import misson.car.domain.RacingCars
+import misson.car.domain.Winners
+
 object ResultView {
-    fun printRoundResult(positions: List<String>) {
-        positions.forEach { println(it) }
+    fun printRoundResult(racingCars: RacingCars) {
+        racingCars.cars
+            .map { CarPositionFormatter.formatCarPosition(it) }
+            .forEach { println(it) }
         println()
     }
 
-    fun printWinner(winners: String) {
-        println("최종 우승자는 $winners 입니다.")
+    fun printWinner(winners: Winners) {
+        val winnerNames = winners.getNames().joinToString(", ")
+        println("최종 우승자는 $winnerNames 입니다.")
     }
 }

src/test/kotlin/misson/RacingCarTest.kt

@@ -1,23 +1,21 @@
 package misson
 
-import misson.car.RacingCar
+import misson.car.domain.RacingCar
 import org.assertj.core.api.Assertions.assertThat
 import org.junit.jupiter.api.Test
 import org.junit.jupiter.api.assertThrows
 
 class RacingCarTest {
     @Test
     fun `이름이 5자를 초과하면 예외가 발생한다`() {
-        assertThrows<IllegalArgumentException> {
-            RacingCar("1234567")
-        }
+        assertThat(assertThrows<IllegalArgumentException> { RacingCar("1234567") }.message)
+            .isEqualTo("이름은 5자 이하만 가능합니다.")
     }
 
     @Test
     fun `이름이 빈 문자열이면 예외가 발생한다`() {
-        assertThrows<IllegalArgumentException> {
-            RacingCar("")
-        }
+        assertThat(assertThrows<IllegalArgumentException> { RacingCar("") }.message)
+            .isEqualTo("이름은 공백일 수 없습니다.")
     }
 
     @Test

src/test/kotlin/misson/RacingCarsTest.kt

@@ -1,27 +1,31 @@
 package misson
 
-import misson.car.RacingCar
-import misson.car.RacingCars
+import misson.car.domain.RacingCar
+import misson.car.domain.RacingCars
 import org.assertj.core.api.Assertions.assertThat
 import org.junit.jupiter.api.Test
+import org.junit.jupiter.params.ParameterizedTest
+import org.junit.jupiter.params.provider.CsvSource
 
 class RacingCarsTest {
     @Test
     fun `자동차 그룹을 한번에 이동시킨다`() {
         val racingCars = RacingCars(List(3) { RacingCar() })
         racingCars.moveAll { 4 }
 
-        val positions = racingCars.getPositions()
-        assertThat(positions).containsExactly("-", "-", "-")
+        racingCars.cars.forEach {
+            assertThat(it.position).isEqualTo(1)
+        }
     }
 
     @Test
     fun `자동차 그룹을 값이 4이하 이면 이동하지 않는다`() {
         val racingCars = RacingCars(List(3) { RacingCar() })
         racingCars.moveAll { 3 }
 
-        val positions = racingCars.getPositions()
-        assertThat(positions).containsExactly("", "", "")
+        racingCars.cars.forEach {
+            assertThat(it.position).isEqualTo(1)
+        }
     }
 
     @Test
@@ -36,7 +40,7 @@ class RacingCarsTest {
             )
 
         val winners = racingCars.findWinners()
-        assertThat(winners.representWinners()).isEqualTo("car1")
+        assertThat(winners.getNames()).containsExactly("car1")
     }
 
     @Test
@@ -51,6 +55,25 @@ class RacingCarsTest {
             )
 
         val winners = racingCars.findWinners()
-        assertThat(winners.representWinners()).isEqualTo("car1, car2")
+        assertThat(winners.getNames()).containsExactly("car1", "car2")
+    }
+
+    @ParameterizedTest
+    @CsvSource(
+        "4, 1",
+        "5, 1",
+        "3, 0",
+        "2, 0",
+    )
+    fun `Param 값에 따라 자동차 그룹이 이동한다`(
+        randomValue: Int,
+        expectedPosition: Int,
+    ) {
+        val racingCars = RacingCars(List(3) { RacingCar() })
+        racingCars.moveAll { randomValue }
+
+        racingCars.cars.forEach { car ->
+            assertThat(car.position).isEqualTo(expectedPosition)
+        }
     }
 }

src/test/kotlin/misson/car/CarPositionFormatterTest.kt

@@ -0,0 +1,34 @@
+package misson.car
+
+import misson.car.domain.CarPositionFormatter
+import misson.car.domain.RacingCar
+import org.assertj.core.api.Assertions.assertThat
+import org.junit.jupiter.api.Test
+import org.junit.jupiter.params.ParameterizedTest
+import org.junit.jupiter.params.provider.CsvSource
+
+class CarPositionFormatterTest {
+    @Test
+    fun `자동차 위치마다 -를 , 앞에 자동차 이름을 붙인다`() {
+        val car = RacingCar("car1", 3)
+        val result = CarPositionFormatter.formatCarPosition(car)
+
+        assertThat(result).isEqualTo("자동차 car1 : ---")
+    }
+
+    @ParameterizedTest
+    @CsvSource(
+        "0, ''",
+        "1, '-'",
+        "3, '---'",
+        "5, '-----'",
+    )
+    fun `position마다 -를 붙인다`(
+        position: Int,
+        expected: String,
+    ) {
+        val result = CarPositionFormatter.formatPosition(position)
+
+        assertThat(result).isEqualTo(expected)
+    }
+}

src/test/kotlin/misson/car/WinnerTest.kt

@@ -1,5 +1,7 @@
 package misson.car
 
+import misson.car.domain.RacingCar
+import misson.car.domain.Winners
 import org.assertj.core.api.Assertions.assertThat
 import org.junit.jupiter.api.Test
 
@@ -14,6 +16,6 @@ class WinnerTest {
                 ),
             )
 
-        assertThat(winners.representWinners()).isEqualTo("car1, car2")
+        assertThat(winners.getNames()).containsExactly("car1", "car2")
     }
 }