블랙커피 JS 스터디 레벨1 1주차

블랙커피 JS 스터디 레벨1 1주차 시작

메이커준님과 함께하는 블랙커피 스터디(문벅스 카페메뉴 만들기) 스터디를 이번에 시작하게 되었다. 그래서 앞으로 4번에 걸쳐 블랙커피 스터디에서 배운 내용을 기록 겸 블로그에 포스팅 하려고 한다.

적절한 코드리뷰 하는 방법

1. 필요 없는 주석 또는 파일은 없는지 확인해보기

• 필요 없는 주석 또는 파일이 있다면 그에 대한 피드백

2. 네이밍이 적절한가

• 네이밍이 불규칙적인 부분은 없는지(어떤건 동사 + 형용사 또 다른건 형용사 + 동사)
• 의미를 알 수 없는 변수명은 없는지
• 함수명 또는 변수명이 그 역할을 제대로 표현하고 있는지
• 변수명 앞에 달러 표시($) 붙이는건 돔 엘리먼트를 나타내는 암묵적인 약속

3. 공백, 띄어쓰기 컨벤션, EOL

• 일정 컨벤션이 유지되고 있는가?
• 공백 또는 띄어쓰기가 제각각은 아닌가?
• 빨간 동그라미가 없는가?
• 맨 끝에 엔터(개행) 추가해주는 것은 개발자들 간에 공식적인 컨벤션이라고 볼 수 있다.
• 어떤 곳은 공백이 없고, 어떤곳은 공백이 있으면 어떻게 보면 코드가 깔끔하지 못하게 보일 수 있다.

4. 중복되는 부분, 불필요한 코드, 반복되는 코드는 없는가?

• 다른 사람이 코드리뷰 해줄 때 효과적으로 개선시킬 수 있는 코드 리뷰 부분이라고 할 수 있다.
• 불필요한 console.log는 없는가?
• 반복되는 코드는 없는가?
• 함수 또는 메서드가 중복되는 기능을 하는 곳은 없는가?
• 불필요한 변수는 없는가?

5. 칭찬, 배울만한 점

• 내가 시도해보지 못한 전략 또는 시도가 있는가?
• 나는 이 사람으로부터 어떤 걸 배울 수 있는가?