Step3 - 3단계 : 자동차 경주 #1662
Merged
Step3 - 3단계 : 자동차 경주 #1662
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 ~ 9 사이의 random 값을 구한 후 random 값이 4이상일 경우 전진한다
경기에 참가하는 자동차의 대수는 사용자가 정한다 여러대의 자동차가 이동 요구사항을 만족하면서 이동한다
자동차가 몇 번의 이동을 할지는 사용자가 정한다
1. 사용자의 입력을 받아서 Cars 객체를 생성한다. 2. 생성한 Cars 객체로 Racing 게임을 한다.
young891221
suggested changes
Dec 6, 2020
|
이번에도 디테일한 피드백 감사합니다 영재님!! |
young891221
approved these changes
Dec 7, 2020
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.
안녕하세요, 신지호 입니다.
이번 과제에서는 최대한 메소드 분리를 잘 지키도록 노력했습니다.
테스트 코드의 작성을 위해
MovingCondition인터페이스를 구현하는ImFixedCondition클래스를 만들었고, 실제 게임 동작시에는 Random값을 주는RandomCondition클래스를 사용하도록 했습니다.매직넘버의 사용을 피하였습니다.
Cars클래스는 일급 컬렉션을 만족하도록 마지막에 리팩토링 했습니다.( 마지막 커밋 이전까진 Cars 클래스에서 MovingCondition을 가지도록 했었는데 -> 이후에 MovingCondition은 Car 객체가 가지도록 변경했습니다)
이번 과제에서는 이정도의 내용을 토대로 봐주시면 좋을 것 같습니다.
감사합니다!!