리팩토링3

- Car, InputService 예외 검증 책임 분리
- getWinners 메서드 stream 사용
- Cars 오타 수정
- InputService 사용하지 않는 코드 삭제
- 변경사항에 맞게 테스트 코드 수정 및 추가

Author: kdongd

src/main/java/carrace/domain/car/Car.java

@@ -14,7 +14,10 @@ public Car(String carName) {
     }
 
     private void validateName(String carName) {
-        if (carName == null || carName.length() > MAX_NAME_LENGTH) {
+        if (carName == null || carName.isBlank()) {
+            throw new IllegalArgumentException("자동 이름은 비어있을 수 없습니다.");
+        }
+        if (carName.length() > MAX_NAME_LENGTH) {
             throw new IllegalArgumentException("자동차 이름은 5글자를 초과할 수 없습니다.");
         }
     }

src/main/java/carrace/domain/car/Cars.java

@@ -8,6 +8,7 @@
 import java.util.List;
 import java.util.Set;
 
+
 public class Cars {
 
     private final List<Car> cars;
@@ -29,7 +30,7 @@ public static Cars from(String input) {
 
     private void validate(List<Car> cars) {
         if (cars.isEmpty()) {
-            throw new IllegalArgumentException("자동차는 최소 1대 이상이어여 합니다.");
+            throw new IllegalArgumentException("자동차는 최소 1대 이상이어야 합니다.");
         }
         validateDuplicateNames(cars);
     }
@@ -48,20 +49,18 @@ public void moveAll(MoveCondition condition) {
     }
 
     public List<Car> getWinners() {
-        int max = 0;
-        for (Car car : cars) {
-            max = Math.max(max, car.getPosition());
-        }
+        int maxPosition = cars.stream()
+                .mapToInt(Car::getPosition)
+                .max()
+                .orElse(0);
 
-        List<Car> winners = new ArrayList<>();
-        for (Car car : cars) {
-            if (car.getPosition() == max) {
-                winners.add(car);
-            }
-        }
-        return winners;
+        return cars.stream()
+                .filter(car -> car.getPosition() == maxPosition)
+                .toList();
     }
 
+
+
     public List<Car> asList() {
         return List.copyOf(cars);
     }

src/main/java/carrace/service/InputService.java

@@ -16,44 +16,28 @@ public String readCarNames() {
         return input;
     }
 
-    public int readCarCount() {
-        int value = parseInt(
-                inputView.readCarCount(),
-                "자동차 대수는 숫자여야 합니다."
-        );
-
-        if (value <= 0) {
-            throw new IllegalArgumentException("자동차는 1대 이상이어야 합니다.");
-        }
-        return value;
-    }
-
     public int readRounds() {
-        int value = parseInt(
-                inputView.readMoveCount(),
-                "시도 횟수는 숫자여야 합니다."
-        );
+        int value = parseInt(inputView.readMoveCount());
 
         if (value <= 0) {
             throw new IllegalArgumentException("시도 횟수는 1 이상이어야 합니다.");
         }
         return value;
     }
 
-    private int parseInt(String input, String errorMessage) {
-        try {
-            return Integer.parseInt(input);
-        } catch (NumberFormatException e) {
-            throw new IllegalArgumentException(errorMessage);
-        }
+    private int parseInt(String input) {
+        return Integer.parseInt(input);
     }
 
     private void validateCarNames(String input) {
+        if (input == null || input.isBlank()) {
+            throw new IllegalArgumentException("자동차 이름을 입력하세요");
+        }
+
         String[] names = input.split(",");
         for (String name : names) {
-            String trimmed = name.trim();
-            if (trimmed.isEmpty() || trimmed.length() > 5) {
-                throw new IllegalArgumentException("자동차 이름은 5자를 초과할 수 없습니다.");
+            if (name.trim().isBlank()) {
+                throw new IllegalArgumentException("자동차 이름은 공백일 수 없습니다.");
             }
         }
     }

src/main/java/carrace/view/CarRaceInputView.java

@@ -11,10 +11,6 @@ public String readCarNames() {
         System.out.println("경주할 자동차 이름을 입력하세요(이름은 쉼표(,)를 기준으로 구분).");
         return scanner.nextLine();
     }
-    public String readCarCount() {
-        System.out.println("자동차 대수는 몇 대인가요?");
-        return scanner.nextLine();
-    }
 
     public String readMoveCount() {
         System.out.println("시도할 횟수는 몇 회인가요?");

src/test/java/carrace/domain/car/CarTest.java

@@ -3,7 +3,6 @@
 import carrace.fixture.MoveConditions;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
-
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.AssertionsForClassTypes.assertThatThrownBy;
 
@@ -55,4 +54,18 @@ void name_too_long_throws() {
                 .isInstanceOf(IllegalArgumentException.class);
     }
 
+    @Test
+    @DisplayName("이름이 null 이면 예외")
+    void name_null_throws() {
+        assertThatThrownBy(() -> new Car(null))
+                .isInstanceOf(IllegalArgumentException.class);
+    }
+
+    @Test
+    @DisplayName("이름이 공백이면 예외")
+    void name_blank_throws() {
+        assertThatThrownBy(() -> new Car("  "))
+                .isInstanceOf(IllegalArgumentException.class);
+    }
+
 }

src/test/java/carrace/service/InputServiceTest.java

@@ -8,36 +8,13 @@
 
 class InputServiceTest {
 
-    @DisplayName("자동차 수가 1이면 정상 입력 (경계값)")
-    @Test
-    void carCountOneIsValid() {
-        InputService service = new InputService(new StubInputView("1", "5"));
-        assertEquals(1, service.readCarCount());
-    }
-
-    @DisplayName("자동차 수가 0이면 예외 발생 (경계값)")
-    @Test
-    void carCountZeroThrows() {
-        InputService service = new InputService(new StubInputView("0", "5"));
-        assertThrows(IllegalArgumentException.class, service::readCarCount);
-    }
-
     @DisplayName("시도 횟수가 음수면 예외 발생 (경계값)")
     @Test
     void roundsNegativeThrows() {
         InputService service = new InputService(new StubInputView("3", "-1"));
         assertThrows(IllegalArgumentException.class, service::readRounds);
     }
 
-    @Test
-    @DisplayName("자동차 이름이 5글자 초과이면 예외 발생")
-    void carNameTooLongThrows() {
-        StubInputView inputView = new StubInputView("TOOLONG","3");
-        InputService service = new InputService(inputView);
-
-        assertThrows(IllegalArgumentException.class, service::readCarNames);
-    }
-
     @Test
     @DisplayName("자동차 이름 공백시 예외 발생")
     void carNameBlankThrows() {