[자동차 경주] 김도현 미션 제출합니다. #357

dohyunk58 · 2025-10-26T17:12:04Z

2주차 자동차 경주 미션 구현

작업 내용

자동차 경주 게임을 TDD, MVC, DI 패턴을 기반으로 구현했습니다.

구현 기능 목록

핵심 로직

쉼표로 구분된 이름으로 Car 객체 리스트 생성
0-9 사이 난수 생성 및 4 이상일 경우 전진 (runOneRound)
가장 멀리 이동한 자동차를 우승자로 선정(findWinners)

입/출력

InputView를 통해 자동차 이름과 시도 횟수 입력
OutputView를 통해 라운드별 실행 결과 및 최종 우승자 출력

예외 처리

자동차 이름이 5자를 초과하는 경우
자동차 이름이 null, 빈 문자열이거나 공백만으로 이루어진 경우
중복된 자동차 이름을 입력한 경우
경주 횟수에 정수가 아닌 값을 입력한 경우
경주 횟수에 자연수가 아닌 값을 입력한 경우

핵심 설계 및 고민

요구 사항을 충족하기 위해 객체 지향 설계 원칙을 적용하려 노력했습니다.

1. MVC 패턴 적용

Controller

RacingGameController: View와 Model을 중재하며 게임의 전체 흐름(입력 ➔ 실행 ➔ 결과)을 제어

Model

Car: name과 position을 가지며 position을 1 증가시키는 move() 메서드를 갖는다
RacingGame: List를 관리하며 findWinners()와 같은 핵심 로직을 수행
CarFactory: 이름 문자열을 받아 List 생성하는 Factory 객체
Validator: 이름 및 횟수 검증 로직을 static 메서드로 제공하는 유틸리티 객체

View

InputView: 사용자의 콘솔 입력을 담당
OutputView: 콘솔 출력을 담당

2. TDD 기반 구현

Model 계층의 모든 비즈니스 로직과 예외 상황에 대해 JUnit 5와 AssertJ를 사용하여 단위 테스트를 우선 작성하고 이를 통과하는 코드를 구현했습니다.

3. 의존성 주입(DI)

Application(main)이 InputView, OutputView, CarFactory 등 필요한 객체를 생성하여 RacingGameController의 생성자로 주입합니다. Controller가 View의 구체적인 구현이 아닌 인스턴스에 의존하게 되어 View와의 결합도가 낮아지며 Controller의 테스트가 용이해졌습니다.

4. 단일 책임 원칙(SRP)

각 객체가 하나의 책임만 갖도록 분리했습니다. CarFactory는 '생성', Validator는 '검증', RacingGame은 '게임 로직', Controller는 '흐름 제어'의 책임을 각각 담당합니다.

기능 목록을 입력, 예외, 메인 로직, 출력 네 가지로 나누어 추가함

Car 객체 생성 시, 객체의 name을 Validator.java에서 검증한다 - 입력값이 null인 경우 - 입력값이 빈 문자열이거나 공백으로 이루어진 경우 - 입력값이 5자를 초과하는 경우 위의 세 가지 경우를 검증하여 만약 위 경우에 해당하면 IllegalArgumentException을 반환한다.

- findMaxPosition으로 Car 리스트에서 가장 큰 position을 탐색 - findWinners로 maxPosition의 값을 가지는 Car 객체의 name을 winners에 저장

- pickNumberInRange에서 0부터 9까지 중 정수 반환 - Car 목록을 순회하며 난수를 생성하고 move에 난수를 넣어 position 변경 README의 "0부터 9 사이 무작위 정수를 구하기"는 제공된 라이브러리를 사용하므로 제거

try-catch문으로 parseInt하여 입력값이 정수가 아닌 경우 IllegalArgumentException를 던짐

paseInt를 시도하는 것은 동일하므로 실제 기능에서는 입력값이 숫자가 아니거나, 정수가 아닌 경우 모두 예외 처리함

- CarFactory에서 쉼표를 구분자로 하여 입력받은 문자열에서 이름을 분리함 - 분리된 이름을 갖는 Car 객체의 List 생성 후 반환

RacingGame: RacingGame.getCarList를 통해 Car 목록 획득 InputView: 검증 책임을 CarFactory에서 Car 목록을 생성할 때, Car 생성자에서 처리 OutputView: 횟수별 상태와 최종 우승자 출력 Application: 메인 로직 제어 기능 추가

Before: - main에서 입력받은 횟수만큼 runOneRound() 메서드가 실행되는데, 이때 난수를 가져와 Car 객체의 move 메서드를 실행한다. - move함수가 MIN_INCREASE_CONDITION(과제에서 4이상) 이상인 경우에만 INCREASE_POSITION(과제에서 1)만큼 position을 더한다. - 문제점 1. move()라는 메서드명에 맞지 않는다. 조건에 따라 움직이기 때문에 “움직인다”라는 이름의 메서드에서 기대하는 행동과는 조금 다를 수 있다. 2. runOneRound()에서 난수에 대해 Car 객체가 어떻게 움직이는지 알 수 없고 Car 클래스의 메서드를 확인해야 한다. After: - Car 클래스의 move() 메서드는 이제 position 필드 값을 INCREASE_POSITION만큼 더해 저장하는 기능만 수행한다. - RacingGame 클래스의 runOneRound() 메서드는 난수를 생성 후 조건에 부합하면 car.move()를 호출하는 방식으로, 어떻게 하나의 라운드가 흘러가는지 정확하게 이해할 수 있도록 했다. - 변경사항에 맞도록 Test를 수정했다.

기존 main에서 자동차 경주의 준비부터 실행, 결과 출력 로직을 모두 수행하던 책임을 RacingGameController로 전달함 - 의존성 주입을 사용하여 main에서 객체 생성 후 주입받아 사용함 - 의존성 주입을 위해 CarFactory, InputView, OutputView의 메서드를 정적 메서드에서 일반 메서드로 변경

CarFactory 클래스에서 createCarList()메서드 호출 시 name 리스트와 set 간의 길이 차이가 발생하는 경우 이름에 중복이 있으므로 예외 처리 - 이름 중복 입력 시 예외 발생하는 Test추가

dohyunk58

다른 분들의 코드를 보고 스스로 점검해봤습니다.

dohyunk58 · 2025-10-29T04:41:05Z

src/main/java/racingcar/controller/RacingGameController.java

+    private List<Car> setupCars() {
+        String carNameInput = inputView.readCarNames();
+        return carFactory.createCarList(carNameInput);
+    }


입력받은 문자열 자체의 검증 없이 split을 하려고 시도합니다.

입력값이 null

입력값이 빈 문자열
인 경우 예외 처리하도록 수정해야 합니다

Ref. #923

dohyunk58 · 2025-10-29T04:42:37Z

src/main/java/racingcar/model/CarFactory.java

변수 이름에 타입을 적지 않아야 합니다. nameList -> names 등

dohyunk58 · 2025-10-29T04:43:33Z

src/main/java/racingcar/model/RacingGame.java

+    public List<Car> getCarList() {
+        return carList;
+    }


getter로 참조 가능한 객체를 반환하는 경우 수정 가능성이 있으므로 깊은 복사를 통해 수정을 방지해야 합니다.

Ref. https://velog.io/@backfox/getter-%EC%93%B0%EC%A7%80-%EB%A7%90%EB%9D%BC%EA%B3%A0%EB%A7%8C-%ED%95%98%EA%B3%A0-%EA%B0%80%EB%B2%84%EB%A6%AC%EB%A9%B4-%EC%96%B4%EB%96%A1%ED%95%B4%EC%9A%94

sun007021

고생 많으셨습니다!
저도 잘 몰라서 제가 남긴 질문이나 의견이 맞지 않을 수 있으니 참고만 해주세요!

sun007021 · 2025-10-29T05:16:05Z

src/main/java/racingcar/model/Validator.java

+    public static void validateDuplicateNames(List<String> names) {
+        Set<String> uniqueNames = new HashSet<>(names);
+
+        if (uniqueNames.size() != names.size()) {


이름이 중복되는 경우에 대해서는 생각을 안해봤는데 이런 예외가 있을 수도 있겠군요!

sun007021 · 2025-10-29T05:18:02Z

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

+
+public class InputView {
+    public String readCarNames() {
+        System.out.println("경주할 자동차 이름을 입력하세요.(이름은 쉼표(,) 기준으로 구분)");


자동차 이름 입력에 대해서도 입력값 검증이 이루어지면 좋을거 같습니다!

맞습니다. @sun007021 님의 PR에서 확인한 점으로 입력값 자체가 null이거나 공백인 경우 예외처리하는 기능이 필요합니다.

sun007021 · 2025-10-29T05:21:01Z

src/main/java/racingcar/controller/RacingGameController.java

+
+    public void run() {
+        // 1. 준비
+        List<Car> cars = setupCars();


inputView를 바로 사용하지 않고 이렇게 함수로 한번 더 포장해서 사용했을때의 장점이 궁금합니다

sun007021 · 2025-10-29T05:23:26Z

src/main/java/racingcar/controller/RacingGameController.java

+        return Integer.parseInt(tryCountInput);
+    }
+
+    private void playRacingGame(RacingGame game, int tryCount) {


Controller에 전체 게임 라운드를 반복하는 로직을 직접 구현하는 것보다 RacingGame에서 동작하는 것이 역할이 더 잘 분리될거 같다고 생각하는데 어떻게 생각하시나요?

맞습니다. 저도 Controller가 반복을 수행하지 않는 것이 이상적이라고 생각합니다.
그래서 대안을 생각해보던 중 현재 코드 설계에서는 Controller에서 View를 호출하기 때문에 만약 루프가 RacingGame 클래스로 이동하면 반복마다 출력하는 경우에 View를 호출하기가 까다로워지는 문제를 발견했습니다.
구조를 수정한다고 생각해보면 게임 결과를 저장하고 출력하면 해결할 수 있을 것 같습니다. 피드백 감사합니다!

dohyunk58 added 16 commits October 24, 2025 02:03

docs: 기능 목록 추가

7de706a

기능 목록을 입력, 예외, 메인 로직, 출력 네 가지로 나누어 추가함

feat: 값이 4 이상인 경우 자동차를 전진하는 기능 추가

c8cf202

feat: 최종 우승자 결정 기능 추가

3cefad6

- findMaxPosition으로 Car 리스트에서 가장 큰 position을 탐색 - findWinners로 maxPosition의 값을 가지는 Car 객체의 name을 winners에 저장

feat: 경주 횟수가 숫자가 아니면 예외 반환 기능 구현

33b23c8

try-catch문으로 parseInt하여 입력값이 정수가 아닌 경우 IllegalArgumentException를 던짐

test: 경주 횟수가 정수가 아니면 예외 발생 테스트 추가

275f343

paseInt를 시도하는 것은 동일하므로 실제 기능에서는 입력값이 숫자가 아니거나, 정수가 아닌 경우 모두 예외 처리함

feat: 경주 횟수가 자연수가 아닌 경우 예외 발생 기능 추가

a1f8d87

feat: 입력받은 자동차 이름으로 자동차 목록 생성 기능 추가

351e494

- CarFactory에서 쉼표를 구분자로 하여 입력받은 문자열에서 이름을 분리함 - 분리된 이름을 갖는 Car 객체의 List 생성 후 반환

feat: 자동차 이름, 시도 횟수 입력 기능 추가

6789ec3

refactor: Model, View 패키지를 소문자로 변경

7d59dd3

feat: 자동차 이름 중복 검증 기능 추가

a307f98

CarFactory 클래스에서 createCarList()메서드 호출 시 name 리스트와 set 간의 길이 차이가 발생하는 경우 이름에 중복이 있으므로 예외 처리 - 이름 중복 입력 시 예외 발생하는 Test추가

docs: README 수정

3a64015

dohyunk58 commented Oct 29, 2025

View reviewed changes

sun007021 reviewed Oct 29, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[자동차 경주] 김도현 미션 제출합니다. #357

[자동차 경주] 김도현 미션 제출합니다. #357

Uh oh!

dohyunk58 commented Oct 26, 2025

Uh oh!

dohyunk58 left a comment

Uh oh!

dohyunk58 Oct 29, 2025 •

edited

Loading

Uh oh!

dohyunk58 Oct 29, 2025

Uh oh!

dohyunk58 Oct 29, 2025

Uh oh!

sun007021 left a comment

Uh oh!

sun007021 Oct 29, 2025

Uh oh!

sun007021 Oct 29, 2025

Uh oh!

dohyunk58 Oct 29, 2025

Uh oh!

sun007021 Oct 29, 2025

Uh oh!

sun007021 Oct 29, 2025

Uh oh!

dohyunk58 Oct 29, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

[자동차 경주] 김도현 미션 제출합니다. #357

Are you sure you want to change the base?

[자동차 경주] 김도현 미션 제출합니다. #357

Uh oh!

Conversation

dohyunk58 commented Oct 26, 2025

2주차 자동차 경주 미션 구현

작업 내용

구현 기능 목록

핵심 설계 및 고민

1. MVC 패턴 적용

2. TDD 기반 구현

3. 의존성 주입(DI)

4. 단일 책임 원칙(SRP)

Uh oh!

dohyunk58 left a comment

Choose a reason for hiding this comment

Uh oh!

dohyunk58 Oct 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sun007021 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

dohyunk58 Oct 29, 2025 •

edited

Loading