[자동차 경주] 박현규 미션 제출합니다.#2
Open
ParkHyeonkyu wants to merge 6 commits intoJava-JavaScript-Language-Stuty:mainfrom
Open
[자동차 경주] 박현규 미션 제출합니다.#2ParkHyeonkyu wants to merge 6 commits intoJava-JavaScript-Language-Stuty:mainfrom
ParkHyeonkyu wants to merge 6 commits intoJava-JavaScript-Language-Stuty:mainfrom
Conversation
구현할 기능 목록 정리
Car 클래스 구현
Car 클래스를 배열로 받는 RacingGame 클래스 구현
자동차 이름, 시도 횟수 관련 예외처리 구현
복잡한 구조를 피하기 위해 gameStart메서드의 기능을 playRound로 분리하고, 코드가 ApplicationTest.js를 통과할 수 있도록 수정했습니다.
게임의 시도를 명확하게 하기 위해 setTimeout을 활용한 delay함수를 추가함
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
신경써서 구현한 부분
1주차 동안 따로 공부했었던 class를 적극적으로 활용하고자 했습니다.
indent depth가 깊어지는 것을 최대한 막고자 함수도 많이 만들었습니다!
여러 예외처리나 비동기 작업을 위해 다양한 메서드들을 끌고와서 활용해보고자 했습니다..ㅎ(Set, setTimeout 등)
피드백이 필요한 부분
test를 pass하기 위해서 모든 에러들을 [ERROR]를 출력하도록 했는데 이렇게 하는게 맞나요?
아니면 그냥 별도로 메시지를 입력해도 되는 건가요?
일단 메시지를 주석 처리해놓았습니다
내가 고려한 예외
1-1. 5자를 초과하는 경우
1-2. 이름이 공백인 경우
1-3. 중복되는 이름이 존재하는 경우