[1단계 - 자동차 경주 구현] 동동(김동희) 미션 제출합니다. #32
Merged
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
carname으로 시작하던 클래스명을 car-name으로 변경한다
Model에서 CarModel로 변경한다.
View.js를 기능에 따라 CarView와 ViewController로 분리하였다
의미를 명확하게 하기위해 lap을 lap-count로 변경한다
중복되는 부분에 대하여 별도의 함수로 분리한다
JaeYeopHan
suggested changes
Feb 14, 2021
- 탭사이즈 2로 고정 - 탭 대신에 스페이스 사용하도록 설정 - 파일 마지막에 빈 라인 추가 - 패키지 매니저로 yarn 을 사용하는 것을 명시
- .prettierignore 설정 - .prettierrc.json을 빈 파일로 저장하며, package.json에 prettier의 버젼을 2.2.1로 고정하여, prettier의 모든 옵션은 2.2.1버젼의 default로 함을 명시적으로 나타낸다
여러 곳에서 반복되는 document.querySelector를 $ 함수로 대체하여 가독성을 높였다. 또한, HTML Element가 할당되는 변수명은 $로 시작하도록 변경하여 통일성을 주었다.
저는 초심자입니다 ㅎㅎ... 자바스크립트 공부한지 8개월에 접어들었고, 재엽님은 리뷰어로 선정된 이후에 구글에 검색해보다가 블로그를 우연찮게 보게 되었고, 그 이후 블로그 글을 보다보니, 그 뒤로 팬이 되었습니다 ⭐ 아키텍쳐 구조를 나름대로의 기준을 가지고 설계했다고 생각했는데, 다른 크루들 코드를 보다보니 다 가지각색인 것 같네요.. MVC 패턴, MVP 패턴, MVVM 패턴 등 구글링은 많이 해보았지만 추상적 개념이라서 코드에는 정확하게 어떻게 적용해야 하는 건지 잘 모르겠습니다. 이는 앞으로 많이 겪어보며, 또 다른 크루들의 코드를 참조해가며 깨달아가야하는 거겠죠? 갈 길이 멀지만 열심히 하겠습니다..! |
- 사용하지 않는 prettier 플러그인을 삭제한다 - cypress 플러그인을 설치한다 - ignore하는 파일을 추가한다
JaeYeopHan
approved these changes
Feb 15, 2021
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.
한재엽님 안녕하세요 I'm a big fan of yours!!
저는 동동(김동희)입니다.
신축년 항상 건강하시고 좋은 일만 생기시고, 제가 귀찮게 하더라도 너그러이 받아주시는(?) 그런 한 해 되시기를 바랍니다 😄
자동차 경주 게임 1단계 구현 미션에 대한 Pull Request를 제출합니다.
🚗 데모사이트 링크 🚗
느낀점(간략하게...)