Step5 - 자동차 경주(리팩토링) 및 피드백 반영

src/main/java/racingcar/RacingGame.java

@@ -28,7 +28,8 @@ public static void main(String[] args) {
         Winner winner = Winner.of(Winner.resultPointOfFirst(cars));
         System.out.println(winner.firstResultPoint());
 
-        winner.winnerSelection(resultView, cars);
+        List<Car> winners = winner.winnerSelection(cars);
+        resultView.getWinner(winners);
     }
 
     public static void printAll(List<Car> cars) {

src/main/java/racingcar/domain/Name.java

@@ -2,17 +2,23 @@
 
 public class Name {
     private static final int LIMIT_CAR_NAME_LENGTH = 5;
+    private static final String ERR_MESSAGE = "이름은 5글자가 최대입니다.";
+
     private String carName;
 
     private Name(String carName) {
         this.carName = carName;
     }
 
-    public static Name validateByName(String carName) {
+    public static Name createName(String carName) {
         if(carName.length() >= LIMIT_CAR_NAME_LENGTH) {
-            throw new IllegalArgumentException("이름은 5글자가 최대입니다.");
+            throw new IllegalArgumentException(ERR_MESSAGE);
         }
+
         return new Name(carName);
     }
 
+    public String getCarName() {
+        return carName;
+    }
 }

src/main/java/racingcar/domain/Winner.java

@@ -30,11 +30,11 @@ private int getFirstResultPoint() {
         return firstResultPoint;
     }
 
-    public void winnerSelection(ResultView resultView, List<Car> cars) {
+    public List<Car> winnerSelection(List<Car> cars) {
         List<Car> winners = cars.stream()
                 .filter(car -> car.getMoveSpace() == firstResultPoint())
                 .collect(Collectors.toList());
 
-        resultView.getWinner(winners);
+        return winners;
     }
 }

src/main/java/racingcar/utils/RandomNumber.java

@@ -5,6 +5,8 @@
 public class RandomNumber {
     private static final int MAX_VALUE = 10;
 
+    private final Random random = new Random();
+
     private int randomNumber;
 
     public int producedRandomNumber() {
@@ -13,7 +15,6 @@ public int producedRandomNumber() {
     }
 
     private int createRandomNum() {
-        Random random = new Random();
         return random.nextInt(MAX_VALUE);
     }
 }

src/main/java/racingcar/view/InputView.java

@@ -25,9 +25,11 @@ public String[] splitCarName(String text) {
     private String[] createNameOfCarsArray(String text) {
         String[] cars = text.split(",");
         for (int i = 0; i < cars.length; i++) {
-            Name.validateByName(cars[i]);
+            Name name = Name.createName(cars[i]);
+            cars[i] = name.getCarName();
         }
-        return text.split(",");
+
+        return cars;
     }
 
     public int getNumberOfTry() {

src/test/java/racingcar/CarTest.java

@@ -14,14 +14,12 @@
 
 class CarTest {
     Car car;
-    Random random;
 
     @BeforeEach
     void setUpCar() {
         String[] carNames = {"pobi"};
         List<Car> cars = Car.createCars(carNames);
         car = cars.get(0);
-        random = new Random();
     }
 
     @Test
@@ -31,8 +29,7 @@ void randomValueTest() {
         int result = 0;
 
         for(int i = 0; i < tryNumber; i++){
-            int randomNumber = random.nextInt(10);
-            result = car.move(randomNumber);
+            result = car.move(5);
         }
 
         assertThat(car.getMoveSpace()).isEqualTo(result);