우아한테크코스 레벨1 자바스크립트로 구현하는 자동차 경주 게임
|
|
| 에디(강상원) | 해온(백솔비) |
- Github 기반으로 온라인 코드 리뷰
- 코딩 컨벤션을 준수하며 개발
- 단위 테스트를 작성하며 개발
- 함수(또는 메서드)를 분리하는 리팩터링
- 터미널에서
node src/index.js를 입력하면 게임이 시작된다.
step1
초간단 자동차 경주 게임을 구현한다.
- 주어진 횟수 동안 n 대의 자동차는 전진 또는 멈출 수 있다.
- 자동차에 이름을 부여할 수 있다. 전진하는 자동차를 출력할 때 자동차 이름을 같이 출력한다.
- 자동차 이름은 쉼표(,)를 기준으로 구분하며 이름은 5자 이하만 가능하다.
- 사용자는 몇 번의 이동을 할 것인지를 입력할 수 있어야 한다.
- 전진하는 조건은 0에서 9 사이에서 무작위 값을 구한 후 무작위 값이 4 이상일 경우이다.
- 자동차 경주 게임을 완료한 후 누가 우승했는지를 알려준다. 우승자는 한 명 이상일 수 있다.
- 우승자가 여러 명일 경우 쉼표(,)를 이용하여 구분한다.
- 사용자가 잘못된 입력 값을 작성한 경우 에러 메시지를 보여주고, 다시 입력할 수 있게 한다.
step2
리팩터링을 진행한다.
- 도메인 로직은 domain/ 하위로, UI 관련 로직은 view/ 하위에서 관리한다.
- domain/ 하위의 모듈은 view/ 하위의 모듈을 의존하지 않아야 한다.
- 도메인 로직 내에서 테스트하기 어려운 부분을 분리하고, 테스트 가능한 부분에 대해서만 단위 테스트를 작성한다.
- 테스트 코드에서 jest.fn()을 사용하지 않는다.
가독성과 재사용성을 유의하며 기능을 구현한다.
- 코드 스타일 가이드에 따라 컨벤션을 준수하며 개발한다.
- 변수 선언시 var를 사용하지 않는다. let, const를 사용한다.
- 전역 변수를 만들지 않는다.
- 축약하지 않는다.
- 하드 코딩된 값 대신에 의미 있는 상수를 활용한다.
- 동등 연산자는 === 로만 사용한다.
- 함수(또는 메서드)의 길이가 10라인을 넘어가지 않도록 구현한다.
- 함수(또는 메서드)가 한 가지 일만 하도록 만든다.
- 함수(또는 메서드)의 들여쓰기 depth는 2단계까지만 허용한다.
- 예를 들어 while문 안에 if문이 있으면 depth는 2단계 이다.
- 힌트) 함수(또는 메서드) 분리는 들여쓰기 depth를 줄이는 좋은 방법이다.
- else 예약어를 쓰지 않는다.
- 도메인 로직과 UI 로직을 분리한다.
- 모든 도메인 로직에 단위 테스트를 구현한다. (UI 로직은 제외)