[자동차 경주 게임] 김민수 미션 제출합니다. #211

alstn113 · 2023-10-31T07:15:57Z

자동차 클래스 및 자동차 목록 생성
- 자동차 클래스 설계: 자동차의 이름, 전진 여부를 관리하는 메서드와 속성
- 여러 대의 자동차를 관리하는 자동차 목록 생성
- 자동차 이름은 5자 이하만 가능하며, 쉼표(,)로 구분하여 입력 받기
무작위 값으로 전진 여부 판단
- Randoms 라이브러리를 활용해 0에서 9까지의 무작위 값을 생성
- 생성된 값이 4 이상일 경우 자동차가 전진하는 조건 설정
이동 횟수 입력 및 실행
- 사용자로부터 이동을 시도할 횟수 입력 받기
- 이동 횟수에 따라 각 자동차의 전진 여부 결정
- 전진한 자동차의 상태 출력
우승자 결정
- 모든 이동이 끝난 후, 우승한 자동차(들)을 결정
- 우승자가 한 명인 경우 해당 우승자 출력
- 우승자가 여러 명일 경우 쉼표(,)로 구분하여 출력
예외 처리
- 사용자 입력에 대한 예외 처리: 잘못된 값 또는 형식 입력 시 IllegalArgumentException 발생 및 처리
- 잘못된 값이 입력되었을 때 프로그램 종료

Domain

NumberGenerator inteface, RandomNumberGenerator
RaceAttempts 시도 횟수, 일급 컬렉션, 자연수 아닌 값 예외 처리
RaceParticipants - 일급 컬렉션, 이름 구분자 처리,이름 중복 예외 처리
RaceWinners - 경기 우승자(들), 일급 컬렉션
RacingCar 레이싱 카, 이름 길이 예외 처리

View

InputView - 입력
OutputView - 출력
racingcar.view.console.util.InputUtil 일반적인 입력 예외 처리

Controller

RacingController - 레이싱 게임 진행

Exception

BaseException - 기본 예외 처리
InvalidInputException - 입력 예외 처리
ErrorMessage - 에러 메시지 상수화

…tRaceWinners 기본 코드 작성

docs/README.md

src/main/java/racingcar/Application.java

src/main/java/racingcar/controller/RaceController.java

src/main/java/racingcar/domain/RaceAttempts.java

src/main/java/racingcar/view/console/util/InputUtil.java

src/main/java/racingcar/exception/ErrorMessage.java

src/main/java/racingcar/domain/RacingCar.java

src/main/java/racingcar/Application.java

junhyeongkim2

안녕하세요! 코드 리뷰 남기고 갑니다! 감사합니다.

src/main/java/racingcar/controller/RaceController.java

src/test/java/racingcar/domain/RaceWinnersTest.java

src/test/java/racingcar/domain/RacingCarTest.java

src/main/java/racingcar/Application.java

src/main/java/racingcar/domain/RaceWinners.java

src/main/java/racingcar/exception/BaseException.java

src/main/java/racingcar/exception/ErrorMessage.java

src/main/java/racingcar/view/console/OutputView.java

src/test/java/racingcar/domain/RaceAttemptsTest.java

src/test/java/racingcar/domain/RaceParticipantsTest.java

src/test/java/racingcar/util/FakeNumberGenerator.java

src/test/java/racingcar/domain/RacingCarTest.java

cosyflower

@alstn113 님! 코드 잘 읽고 가요
Exception을 다시 분류해서 작성하신 모습을 보면서 예외를 더 깔끔하게 작성할 수 있는 방법을 알아갈 수 있었어요! 테스트 코드에서도 nested,, 저도 다음 미션 때 참고하겠습니다

2주 차 미션 고생 많으셨어요:)

docs/README.md

src/main/java/racingcar/domain/generator/NumberGenerator.java

src/main/java/racingcar/domain/RaceParticipants.java

src/main/java/racingcar/domain/RacingCar.java

src/test/java/racingcar/domain/RacingCarTest.java

src/test/java/racingcar/util/FakeNumberGenerator.java

src/test/java/racingcar/domain/RacingCarTest.java

src/test/java/racingcar/domain/RaceParticipantsTest.java

songjayhyun

2주차도 고생하셨습니다.

전체적으로 코드가 깔끔하고 많이 배워가네요 ㅎㅎ 클래스 활용과 exception 처리 부분은 다음 미션때 활용해봐야 겠습니다 !!

저도 리뷰 부탁드릴게요 ! #1170

src/main/java/racingcar/controller/RaceController.java

src/main/java/racingcar/domain/RaceParticipants.java

src/main/java/racingcar/domain/generator/NumberGenerator.java

src/main/java/racingcar/view/console/util/InputUtil.java

src/test/java/racingcar/domain/RaceParticipantsTest.java

src/main/java/racingcar/controller/RaceController.java

src/main/java/racingcar/domain/RaceParticipants.java

src/main/java/racingcar/domain/RaceWinners.java

src/test/java/racingcar/domain/RacingCarTest.java

TaeHye0n

코드 잘 봤습니다.
전체적으로 많은 고민을 하셨다는게 느껴지네요.
특히 getter를 지양한 점과 Randoms를 사용할 때 인터페이스를 구현하는 방법이 인상 깊었습니다.

궁금한점은 View의 메서드들을 Static으로 선언하면 인스턴스화 할 필요 없이 컨트롤러에서 사용할 수 있어 Static으로 저는 사용했었는데, 민수님은 어떤 방법이 더 좋다고 생각하시는지 궁금합니다.

src/main/java/racingcar/domain/generator/NumberGenerator.java

src/main/java/racingcar/view/console/OutputView.java

src/test/java/racingcar/domain/RaceAttemptsTest.java

src/main/java/racingcar/domain/RacingCar.java

src/main/java/racingcar/domain/RaceParticipants.java

alstn113 · 2023-11-02T09:45:15Z

@TaeHye0n
input이나 output을 인터페이스로 만들고 갈아끼운다면 파라미터에 넣는게 좋다고 생각합니다.

dy102

구조도 좋고, 인터페이스 사용하신 부분도 좋아요. 그리고 구조를 많이 나눈 것에 비해 코드가 읽기 어렵지 않았어요. 전반적으로 좋아보여요

docs/README.md

src/main/java/racingcar/domain/RaceParticipants.java

src/main/java/racingcar/domain/RacingCar.java

src/test/java/racingcar/util/FakeNumberGenerator.java

src/main/java/racingcar/exception/InvalidInputException.java

src/main/java/racingcar/controller/RaceController.java

JudyYeon

고생하셨습니다😆

src/main/java/racingcar/Application.java

src/main/java/racingcar/controller/RaceController.java

src/main/java/racingcar/domain/RaceParticipants.java

alstn113 added 30 commits October 28, 2023 14:08

docs: 기능 구현 목록 작성

f23d011

feat: BaseException extends IllegalArgumentException 작성

c0546bc

feat: InvalidInputException과 ErrorMessage enum 작성

2156222

feat: 기본적인 입력 유틸 정적 메소드 작성

cde2e52

feat: input view readCarNames, readRaceAttempts 작성

228ea53

feat: OutputView printResultMessage, printRaceProgress, println, prin…

2dbcf50

…tRaceWinners 기본 코드 작성

feat: NumberGenerator interface 작성

395a7e3

feat: NumberGenerator의 구현체인 RandomNumberGenerator 작성

3eed4fb

feat: RacingCar domain validate name length

3acec24

feat: RacingCar domain moveOrTop method, name, positiond getter

d72a8b3

feat: RacingCarTest FakeNumberGenerator 작성

3dc81a1

feat: RacingCarTest 생성, 이름 예외, moveOrStop method 테스트

5ab1486

refactor: 테스트의 FakeNumberGenerator를 재사용성을 위해 util로 분리

caad5bc

feat: OutputView의 printRaceProgress 작성

5a711bd

feat: ErrorMessage 자동차 상수 이용, 시도 횟수 관련 에러 메세지 작성

4bbe985

feat: RaceAttempts(시도 횟수) 일급 컬렉션으로 만들고 예외 처리

20629f2

chore: RacingCarTest display name 추가

5e48df3

feat: RaceAttemptsTest 생성, 시도 횟수 제한 예외 테스트

d8b404d

feat: RaceParticipants 일급 컬렉션 작성, (,) 구분자 처리(split)

f411282

feat: RaceParticipants 비어 있는 이름, 중복된 이름 예외 처리

daf7cf3

feat: RaceParticipants 생성 및 이름 중복, 비어 있을 경우 에러 처리 테스트

2e8a194

chore: RaceParticipantsNameExceptionTest 이름 오타 수정

2e87e36

feat: RaceWinners 일급 컬렉션 base 코드 작성

4d2631c

feat: RaceWinners maxPosition을 찾고 같은 position인 리스트 리턴

9ac9b97

feat: RaceWinnersTest 우승자가 한 명인 경우, 우승자가 여러 명인 경우 테스트

5ab380c

feat: RaceController base 코드 작성

1349c4f

feat: Application에서 RaceController 초기화

811d4d2

feat: RaceController 구조만 일단 작성

a227ef5

feat: RaceController setRaceParticipants method 작성

592b553

feat: RaceController setRaceAttempts method 작성

124fc98

alstn113 added 4 commits October 28, 2023 15:44

feat: RaceController raceInProgress method 작성

365d03c

feat: RaceController raceRaceAndGetWinners method 작성

9b30a4c

chore: 사용하지 않는 OutputView의 println 메소드 삭제

3048cea

refactor: erorr message에 race attempts 상수 적용

0a28e2e

OikimiO reviewed Nov 1, 2023

View reviewed changes

docs/README.md Show resolved Hide resolved

src/main/java/racingcar/Application.java Show resolved Hide resolved

src/main/java/racingcar/controller/RaceController.java Show resolved Hide resolved

src/main/java/racingcar/domain/RaceAttempts.java Show resolved Hide resolved

Suihub reviewed Nov 1, 2023

View reviewed changes

junhyeongkim2 reviewed Nov 1, 2023

View reviewed changes

dgh06175 reviewed Nov 1, 2023

View reviewed changes

cosyflower reviewed Nov 2, 2023

View reviewed changes

songjayhyun reviewed Nov 2, 2023

View reviewed changes

seungmin-park reviewed Nov 2, 2023

View reviewed changes

TaeHye0n reviewed Nov 2, 2023

View reviewed changes

dy102 reviewed Nov 2, 2023

View reviewed changes

JudyYeon reviewed Nov 5, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[자동차 경주 게임] 김민수 미션 제출합니다. #211

[자동차 경주 게임] 김민수 미션 제출합니다. #211

alstn113 commented Oct 31, 2023

junhyeongkim2 left a comment

cosyflower left a comment

songjayhyun left a comment

TaeHye0n left a comment

alstn113 commented Nov 2, 2023 •

edited

Loading

dy102 left a comment

JudyYeon left a comment

[자동차 경주 게임] 김민수 미션 제출합니다. #211

Are you sure you want to change the base?

[자동차 경주 게임] 김민수 미션 제출합니다. #211

Conversation

alstn113 commented Oct 31, 2023

Domain

View

Controller

Exception

junhyeongkim2 left a comment

Choose a reason for hiding this comment

cosyflower left a comment

Choose a reason for hiding this comment

songjayhyun left a comment

Choose a reason for hiding this comment

TaeHye0n left a comment

Choose a reason for hiding this comment

alstn113 commented Nov 2, 2023 • edited Loading

dy102 left a comment

Choose a reason for hiding this comment

JudyYeon left a comment

Choose a reason for hiding this comment

alstn113 commented Nov 2, 2023 •

edited

Loading