4단계 - 자동차 경주(우승자) #6066
Open
4단계 - 자동차 경주(우승자) #6066
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 올바르지 않은 값이 들어올 경우 올바른 값이 들어올 때까지 반복
- 올바르지 않은 입력값(0, 음수)을 검증하는 과정 추가
- 전략 패턴 적용 - 테스트용 AlwaysMove 추가
- 1회가 지날 때마다 결과를 저장
- RacingGame.progressRound 메소드 분리 - RacingGameApplication bound, threshold 변수 분리 - ResultView.printEmptyLineBetweenRounds 메소드 분리
wooobo
requested changes
Mar 21, 2025
- getter 성 메소드 제거 - move 메소드 시그니쳐 변경 - equals 메소드 오버라이딩
|
각각 코멘트에 수정 내용 포함하여 요청했으니 확인 부탁드립니다 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
이번에는 기존에 개발된 내용을 고쳐야 하는 경우가 있었습니다.
이 때, 기존 테스트를 수정해야 하는 일이 빈번하게 있었는데요.
이전에는 테스트를 수정하는 것이 막연히 좋지 않은 일이라고 생각했지만
요구 사항이 변경되었기 때문에 테스트를 수정하는 것은 당연한 일이라는 생각도 드는데요.
리뷰어 분의 생각은 어떠신지 궁금합니다.
감사합니다.
p.s. 4단계 구현을 3단계 리뷰 받기 전에 해서 기존과 같은 문제점이 보일 수도 있으니 양해 부탁 드립니다 🤣