[201800288 채다영] 2주차 미션을 제출합니다. #9
Open
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
startGame 함수 생성(자동차 이름과 시도 횟수 입력받고 예외발생시키기)
startGame 함수에 입력할 값을 안내해주는 문장추가
중간과정을 출력하는 printProgress 추가
최종결과를 출력하는 printResult 함수 추가
daemin-kim
reviewed
Feb 28, 2024
| public static void main(String[] args) { | ||
| // TODO 구현 진행 | ||
| } | ||
| public static void startGame() { |
There was a problem hiding this comment.
startGame 메소드에서 입력과 예외 처리를 같이 진행하고 있네요! 이는 입력 받기, 게임 실행, 결과 출력 등 책임을 나누어 별도의 메소드나 클래스로 관리할 수 있습니다. 코드를 구현하신 다음 책임을 하나씩 분리해보는 작업을 진행해보시면 어떨까요?
| throw new IllegalArgumentException("[ERROR] 시도 횟수는 숫자여야 한다."); | ||
| } | ||
| } | ||
| public void printProgress(Car car) { |
There was a problem hiding this comment.
printProgress와 printResult는 선언 방식이 각각 public 과 public static으로 다르네요! 특별한 이유가 있는지 궁금합니다.
| try { | ||
| attempt = Integer.parseInt(attemptInput); | ||
| } catch (NumberFormatException e) { | ||
| throw new IllegalArgumentException("[ERROR] 시도 횟수는 숫자여야 한다."); |
There was a problem hiding this comment.
잘못된 입력에 대한 예외를 처리해 주셨지만, 사용자에게 재입력을 요청하고 있지 않네요! 이 로직을 추가하여 구현해 주시면 좋을 것 같습니다.
src/main/java/racingcar/Car.java
Outdated
| @@ -10,4 +12,16 @@ public Car(String name) { | |||
| } | |||
|
|
|||
| // 추가 기능 구현 | |||
| public void moveForward() { | |||
| int move = Randoms.pickNumberInRange(0,9); | |||
| if(move >= 4) { | |||
There was a problem hiding this comment.
moveForward에 사용된 숫자 4는 매직 넘버로 선언하여 관리한다면, 가독성과 유지보수성에 도움을 줍니다. 다음은 예시이니 참고 부탁드립니다!
Suggested change
| if(move >= 4) { | |
| private static final int MOVE_THRESHOLD = 4 |
| } | ||
| System.out.println(sb); | ||
| } | ||
| public static void printResult(List<Car> cars) { |
There was a problem hiding this comment.
printResult에는 결과 “출력” 로직만 포함되는 것이 좋습니다. 현재는 우승자를 결정하는 로직까지 포함되어 있음으로 이를 별도의 메소드로 분리해주신다면 더 좋은 코드가 될 것 같습니다!
printProgress에 static 추가
Input 클래스 추가
Input 클래스가 아닌 메서드 추가
InputCarName 메서드 생성 잘못된 입력을 받았을 때 재귀호출로 다시 입력받기
InputAttempt 메서드 생성 잘못된 입력을 받았을 때 재귀호출로 다시 입력받기
indWinners 메서드 생성 우승자 찾기
findWinners 메서드 생성 우승자 찾기
printResult 메서드 수정 우승자 출력
|
피드백 주신 부분들 반영해서 고쳤보았습니다. |
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.
No description provided.