refactor: Car 객체 테스트 가능하도록 수정

Author: minji-go

README.md

@@ -46,4 +46,18 @@
   --
   -
   --
-    ```
+    ```
+  
+---
+## 1차 코멘트
+- [x] View가 상태를 가지고 있는게 맞을까요?
+- [ ] Scanner에 static을 붙여 메모리를 아껴보면 어떨까요 ㅎㅎ
+- [x] 객체지향 설계: Car라는 도메인이 있고 그 Car가 Position을 가지고 있고, move하는 행위도 스스로 하는거 아닐까요?
+- [x] 상수를 통해 해당 코드의 의미를 나타내보시면 어떨까요?
+
+## 2차 코멘트
+- [x] String.repeat() 사용
+- [ ] [일급컬렉션 적용](https://jojoldu.tistory.com/412) `private final List<Car> cars = new ArrayList<>();`
+- [ ] [테스트 하기 좋은 코드로 인터페이스를 통해 전략패턴](https://tecoble.techcourse.co.kr/post/2020-05-17-appropriate_method_for_test_by_interface/)
+  - canMove에서 랜덤값을 처리하기 떄문에 실제 테스트를 하기가 어려운 문제가 있습니다. 이유는 position이 1이 될지, 0이 될지 알수가 없기 떄문이죠.
+- [ ] position의 유효성은 따로 없을까요? 예를들어 0 이상의 값이여야만 한다. 같은 거요!

src/main/java/Car.java

@@ -2,16 +2,28 @@
 
 public class Car {
     private static final Random random = new Random();
-    private static final int moveThreshold = 4;
+    private static final int MOVE_THRESHOLD = 4;
     private int position;
-    
+
+
+//    public int move() {
+//        if (canMove()) position++;
+//        return position;
+//    }
+
+//    private boolean canMove() {
+//        return random.nextInt(10) >= MOVE_THRESHOLD;
+//    }
+
     public int move() {
-        if (canMove()) position++;
-        return position;
+        int randomValue = random.nextInt(10);
+        return move(randomValue);
     }
 
-    private boolean canMove() {
-        return random.nextInt(10) >= moveThreshold;
+    public int move(int random) {
+        if(random >= MOVE_THRESHOLD) position++;
+        return position;
     }
 
+
 }

src/main/java/CarRace.java

@@ -1,31 +1,28 @@
-import java.util.ArrayList;
 import java.util.List;
+import java.util.stream.Collectors;
+import java.util.stream.IntStream;
 
 public class CarRace {
-    private final List<Car> cars = new ArrayList<>();
     private final int runCount;
+    private final List<Car> cars;
 
     public CarRace(int carCount, int runCount) {
-        for (int i = 0; i < carCount; i++) {
-            cars.add(new Car());
-        }
         this.runCount = runCount;
+        this.cars = IntStream.range(0, carCount)
+                .mapToObj(i -> new Car())
+                .collect(Collectors.toList());
     }
 
     public List<List<Integer>> run() {
-        List<List<Integer>> result = new ArrayList<>(runCount);
-        for (int i = 0; i < runCount; i++) {
-            result.add(runOnce());
-        }
-        return result;
+        return IntStream.range(0, runCount)
+                .mapToObj(i -> runOnce())
+                .collect(Collectors.toList());
     }
 
     private List<Integer> runOnce() {
-        List<Integer> result = new ArrayList<>(cars.size());
-        for (Car car : cars) {
-            result.add(car.move());
-        }
-        return result;
+        return cars.stream()
+                .map(Car::move)
+                .collect(Collectors.toList());
     }
 
 }

src/main/java/InputView.java

@@ -6,6 +6,7 @@ public class InputView {
     public int getCarCount() {
         return validatePositiveInt(scanInt("자동차 대수는 몇 대 인가요?"));
     }
+
     public int getRunCount() {
         return validatePositiveInt(scanInt("시도할 회수는 몇 회 인가요?"));
     }

src/main/java/ResultView.java

@@ -1,24 +1,22 @@
 import java.util.List;
 
 public class ResultView {
+    private static final String PATTERN = "-";
 
     public void print(List<List<Integer>> result) {
         System.out.println();
         System.out.println("실행 결과");
 
-        for (List<Integer> cars : result) {
-            for(int car : cars){
-                printDash(car);
+        for (List<Integer> runResult : result) {
+            for (int carPosition : runResult) {
+                printPattern(carPosition);
             }
             System.out.println();
         }
         System.out.println();
     }
 
-    private void printDash(int count) {
-        while(count-->0){
-            System.out.print("-");
-        }
-        System.out.println();
+    private void printPattern(int count) {
+        System.out.println(PATTERN.repeat(count));
     }
 }

src/test/java/CarRaceTest.java

@@ -33,7 +33,7 @@ void printGuideMessage(int carNumber, int tryCount) {
     }
 
     @ParameterizedTest
-    @ValueSource(strings = {"-3\n","0\n"})
+    @ValueSource(strings = {"-3\n", "0\n"})
     @DisplayName("자동차 대수는 양수여야한다.")
     void throwIfCarCountPositive(String carCount) {
         System.setIn(new ByteArrayInputStream(carCount.getBytes()));
@@ -45,7 +45,7 @@ void throwIfCarCountPositive(String carCount) {
     }
 
     @ParameterizedTest
-    @ValueSource(strings = {"0\n","-1\n"})
+    @ValueSource(strings = {"0\n", "-1\n"})
     @DisplayName("시도할 회수는 양수여야한다.")
     void throwIfRunCountPositive(String runCount) {
         System.setIn(new ByteArrayInputStream(runCount.getBytes()));