refactor: Game, CarRegistry 기능 분리

Author: chemistryx

src/main/java/io/suhan/racingcar/Main.java

@@ -13,7 +13,7 @@ public static void main(String[] args) {
         int count = InputView.getTrialsCount();
 
         Game game = Game.of(count);
-        game.registerCars(names);
+        game.getCarManager().registerCars(names);
 
         List<RoundResult> results = game.execute();
         List<Car> winners = game.getWinners();

src/main/java/io/suhan/racingcar/domain/CarRegistry.java renamed to src/main/java/io/suhan/racingcar/domain/CarManager.java

@@ -3,15 +3,15 @@
 import java.util.ArrayList;
 import java.util.List;
 
-public class CarRegistry {
+public class CarManager {
     private final List<Car> cars;
 
-    private CarRegistry() {
+    private CarManager() {
         this.cars = new ArrayList<>();
     }
 
-    public static CarRegistry of() {
-        return new CarRegistry();
+    public static CarManager of() {
+        return new CarManager();
     }
 
     public void register(Car car) {
@@ -28,6 +28,22 @@ public void moveCars() {
         }
     }
 
+    public void registerCars(List<String> names) {
+        for (String name : names) {
+            Car car = Car.of(name);
+            cars.add(car);
+        }
+    }
+
+    public List<Car> getCarsWithBestPosition() {
+        int bestPosition = getBestPosition();
+
+        return getRegisteredCars()
+                .stream()
+                .filter((car) -> car.getPosition() == bestPosition)
+                .toList();
+    }
+
     public int getBestPosition() {
         return cars
                 .stream()

src/main/java/io/suhan/racingcar/domain/Game.java

@@ -6,11 +6,11 @@
 public class Game {
     private static final int GAME_ROUNDS_MINIMUM = 1;
 
-    private final CarRegistry carRegistry;
+    private final CarManager carManager;
     private final int rounds;
 
     private Game(int rounds) {
-        this.carRegistry = CarRegistry.of();
+        this.carManager = CarManager.of();
         this.rounds = rounds;
     }
 
@@ -20,7 +20,7 @@ public static Game of() {
 
     public static Game of(int rounds) {
         if (rounds < GAME_ROUNDS_MINIMUM) {
-            throw new IllegalArgumentException("시도 회수는 " + GAME_ROUNDS_MINIMUM + " 이상이어야 합니다.");
+            throw new IllegalArgumentException("시도 횟수는 " + GAME_ROUNDS_MINIMUM + " 이상이어야 합니다.");
         }
 
         return new Game(rounds);
@@ -30,30 +30,18 @@ public List<RoundResult> execute() {
         List<RoundResult> results = new ArrayList<>();
 
         for (int i = 0; i < rounds; i++) {
-            carRegistry.moveCars();
-            results.add(RoundResult.of(carRegistry.getRegisteredCars()));
+            carManager.moveCars();
+            results.add(RoundResult.of(carManager.getRegisteredCars()));
         }
 
         return results;
     }
 
     public List<Car> getWinners() {
-        int bestPosition = carRegistry.getBestPosition();
-
-        return carRegistry.getRegisteredCars()
-                .stream()
-                .filter((car) -> car.getPosition() == bestPosition)
-                .toList();
-    }
-
-    public CarRegistry getCarRegistry() {
-        return carRegistry;
+        return carManager.getCarsWithBestPosition();
     }
 
-    public void registerCars(List<String> names) {
-        for (String name : names) {
-            Car car = Car.of(name);
-            carRegistry.register(car);
-        }
+    public CarManager getCarManager() {
+        return carManager;
     }
 }

src/test/java/io/suhan/racingcar/domain/GameTest.java

@@ -18,13 +18,9 @@ public class GameTest {
 
         Game game = Game.of();
 
-        for (String name : names) {
-            Car car = Car.of(name);
+        game.getCarManager().registerCars(names);
 
-            game.getCarRegistry().register(car);
-        }
-
-        List<String> registeredNames = game.getCarRegistry().getRegisteredCars().stream().map(Car::getName).toList();
+        List<String> registeredNames = game.getCarManager().getRegisteredCars().stream().map(Car::getName).toList();
 
         assertIterableEquals(names, registeredNames);
     }
@@ -41,9 +37,9 @@ public class GameTest {
         Car brie = Car.of("brie", stopGenerator);
         Car brown = Car.of("brown", forwardGenerator);
 
-        game.getCarRegistry().register(neo);
-        game.getCarRegistry().register(brie);
-        game.getCarRegistry().register(brown);
+        game.getCarManager().register(neo);
+        game.getCarManager().register(brie);
+        game.getCarManager().register(brown);
 
         List<String> expectedNames = List.of("neo", "brown");