🚀 4단계 - 자동차 경주(우승자) #6061

kjy2844 · 2025-03-20T08:55:36Z

구현 TODO 목록

자동차 이름 기능
- Car 클래스에 이름 필드 추가
- 이름 유효성 검사 (5자 초과 불가)
- Car 생성자에 이름 파라미터 추가
- CarTest에 이름 관련 테스트 추가
  - 이름이 5자를 초과하면 예외 발생
  - 유효한 이름으로 생성 가능
자동차 이름 출력
- ResultView 수정
  - 자동차 이름과 위치를 함께 출력하는 형식 변경
  - presentCars 메소드 파라미터 수정 (이름 정보 추가)
- Race 클래스 수정
  - getCarPositions 메소드를 getCarStatus로 변경
  - 자동차 이름과 위치를 함께 반환하도록 수정
자동차 이름 입력
- InputView 수정
  - 자동차 이름 입력 메소드 추가
  - 쉼표로 구분된 이름 문자열 파싱 로직 추가
- GameSettings 수정
  - 자동차 이름 목록 필드 추가
  - 생성자 수정
우승자 판정
- Race 클래스에 우승자 판정 로직 추가
  - getWinners 메소드 추가
  - 가장 멀리 간 자동차들 찾기
- ResultView에 우승자 출력 메소드 추가
- Game 클래스 수정
  - 경주 종료 후 우승자 출력 로직 추가

개발 구조

Game: 게임의 전체 생명주기 관리 및 사용자 인터랙션 조정
Race: 경주 진행 상태 관리, 자동차 이동 처리, 우승자 판정
GameSettings: 게임 설정값(자동차 이름, 라운드 수) 검증 및 관리
Car: 자동차의 기본 속성(이름, 위치) 관리
InputView: 사용자 입력 처리 및 검증
ResultView: 게임 진행 상태 및 결과 출력
CarStatus: 자동차의 현재 상태를 불변 객체로 표현
MoveStrategy: 자동차 이동 전략 정의 (RandomMoveStrategy, FixedMoveStrategy)

- [x] `Car` 클래스에 이름 필드 추가 - [x] 이름 유효성 검사 (5자 초과 불가) - [x] `Car` 생성자에 이름 파라미터 추가 - [x] `CarTest`에 이름 관련 테스트 추가 - [x] 이름이 5자를 초과하면 예외 발생 - [x] 유효한 이름으로 생성 가능

- [x] `ResultView` 수정 - [x] 자동차 이름과 위치를 함께 출력하는 형식 변경 - [x] `presentCars` 메소드 파라미터 수정 (이름 정보 추가) - [x] `Race` 클래스 수정 - [x] `getCarPositions` 메소드를 `getCarStatus`로 변경 - [x] 자동차 이름과 위치를 함께 반환하도록 수정

- [x] `InputView` 수정 - [x] 자동차 이름 입력 메소드 추가 - [x] 쉼표로 구분된 이름 문자열 파싱 로직 추가 - [x] `GameSettings` 수정 - [x] 자동차 이름 목록 필드 추가 - [x] 생성자 수정

- [x] `Race` 클래스에 우승자 판정 로직 추가 - [x] `getWinners` 메소드 추가 - [x] 가장 멀리 간 자동차들 찾기 - [x] `ResultView`에 우승자 출력 메소드 추가 - [x] `Game` 클래스 수정 - [x] 경주 종료 후 우승자 출력 로직 추가

Hyeon9mak

안녕하세요 panya 님~ 4단계도 빠르게 잘 진행해주셨습니다!
완성도가 높기 때문에 몇 가지 고민해보실만한 포인트에 코멘트 남겼습니다.
궁금하신 점 언제든지 편하게 질문주세요~

src/main/java/Car.java

Hyeon9mak · 2025-03-21T01:01:30Z

src/main/java/Car.java

+    private void validateName(String name) {
+        if (name == null || name.trim().isBlank()) {
+            throw new IllegalArgumentException("Name cannot be blank");
+        }
+        if (name.length() > 5) {
+            throw new IllegalArgumentException("Name cannot be longer than 5 characters");
        }


공백에 대한 검증을 하고자 한다면 trim() 은 생략 되어도 같은 효과가 나올거 같아요 😄

Hyeon9mak · 2025-03-21T01:46:52Z

src/main/java/GameSettings.java

+        for (String name : carNames) {
+            if (name == null || name.trim().isEmpty()) {
+                throw new IllegalArgumentException("자동차 이름은 비어있을 수 없습니다.");
+            }
+        }


마찬가지로 isBlank() 로 대체할 수 있겠네요!

Hyeon9mak · 2025-03-21T01:47:47Z

src/main/java/InputView.java

 public class InputView {
+
+    public static final String CAR_NAME_DELIMITER = ",";
    private final Scanner scanner;

    public InputView() {
        this.scanner = new Scanner(System.in);
    }

    public GameSettings getGameSettings() {
-        int carCount = promptInt("자동차 대수는 몇 대 인가요?");
+        String inputCarNames = prompt("경주할 자동차 이름을 입력하세요(이름은 쉼표(,)를 기준으로 구분).");
+        String[] carNames = inputCarNames.split(CAR_NAME_DELIMITER);
        int roundCount = promptInt("시도할 회수는 몇 회 인가요?");
-        return new GameSettings(carCount, roundCount);
+        return new GameSettings(carNames, roundCount);
+    }


InputView 는 별도로 변경되는 상태를 갖고 있지 않고, 몇 번의 자동차 게임에서도 재활용이 가능할 걸로 보이는데요,
InputView 인스턴스를 매번 생성할 필요 없이 static 하게 사용할 수 있도록 다듬어보는 건 어떨까요?

Hyeon9mak · 2025-03-21T01:47:57Z

src/main/java/MoveStrategy.java

+@FunctionalInterface
+public interface MoveStrategy {
+
+    boolean shouldMove();
+} 


함수형 인터페이스 선언 👍

Hyeon9mak · 2025-03-21T01:49:16Z

src/main/java/Race.java

+    private boolean isWinner(Car car, int maxPosition) {
+        return car.getPosition() == maxPosition;
+    }


자동차 객체로부터 값을 추출하고 비교하는 것도 좋지만, 값을 자동차 객체에게 전달하여 원하는 걸 물어보는 건 어떨까요?

객체에게 질문하라

이런 코드에는 어떤 장점이 생길까요?

Hyeon9mak · 2025-03-21T01:50:12Z

src/main/java/Race.java

+    private List<CarStatus> findWinnersWithPosition(int maxPosition) {
+        List<CarStatus> winners = new ArrayList<>();
+        for (Car car : cars) {
+            addWinnerIfQualified(winners, car, maxPosition);
+        }
+        return winners;
+    }
+
+    private void addWinnerIfQualified(List<CarStatus> winners, Car car, int maxPosition) {
+        if (isWinner(car, maxPosition)) {
+            winners.add(new CarStatus(car));
+        }
+    }


현재 List<Car> cars 컬렉션에게 컬렉션 그 이상의 요구, 비즈니스적인 요구가 많아지고 있어요.
저는 이 시점이 일급 컬렉션을 도입하기 가장 좋은 시점이라고 생각하는데요,
한번 일급 컬렉션 도입을 시도해보는 건 어떨까요? 😉
https://jojoldu.tistory.com/412

Hyeon9mak · 2025-03-21T01:50:48Z

src/main/java/RandomMoveStrategy.java

+public class RandomMoveStrategy implements MoveStrategy {
+
+    public static final int RANDOM_NUMBER_RANGE = 10;
+    private static final int MOVE_THRESHOLD = 4;


private 으로 숨겨줘도 되겠네요!
intelliJ 를 이용하신다면 상수생성 단축키를 연속 2회 입력하시면 옵션 변경이 가능합니다.

kjy2844 added 7 commits March 20, 2025 13:20

docs : 4단계 TODO 정리

8825da2

feat : 자동차 이름 입력

040e278

- [x] `InputView` 수정 - [x] 자동차 이름 입력 메소드 추가 - [x] 쉼표로 구분된 이름 문자열 파싱 로직 추가 - [x] `GameSettings` 수정 - [x] 자동차 이름 목록 필드 추가 - [x] 생성자 수정

refactor : 테스트용 생성자 제거

778c6c3

refactor : 포맷팅

63dff04

Hyeon9mak requested changes Mar 21, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚀 4단계 - 자동차 경주(우승자) #6061

🚀 4단계 - 자동차 경주(우승자) #6061

kjy2844 commented Mar 20, 2025

Hyeon9mak left a comment

Hyeon9mak Mar 21, 2025

Hyeon9mak Mar 21, 2025

Hyeon9mak Mar 21, 2025

Hyeon9mak Mar 21, 2025

Hyeon9mak Mar 21, 2025

Hyeon9mak Mar 21, 2025

Hyeon9mak Mar 21, 2025

🚀 4단계 - 자동차 경주(우승자) #6061

Are you sure you want to change the base?

🚀 4단계 - 자동차 경주(우승자) #6061

Conversation

kjy2844 commented Mar 20, 2025

구현 TODO 목록

개발 구조

Hyeon9mak left a comment

Choose a reason for hiding this comment

Hyeon9mak Mar 21, 2025

Choose a reason for hiding this comment

Hyeon9mak Mar 21, 2025

Choose a reason for hiding this comment

Hyeon9mak Mar 21, 2025

Choose a reason for hiding this comment

Hyeon9mak Mar 21, 2025

Choose a reason for hiding this comment

Hyeon9mak Mar 21, 2025

Choose a reason for hiding this comment

Hyeon9mak Mar 21, 2025

Choose a reason for hiding this comment

Hyeon9mak Mar 21, 2025

Choose a reason for hiding this comment