블랙커피JS 레벨1 | 문벅스 카페메뉴 만들기 2주차, 코드리뷰

블랙커피 JS 레벨 1 | 문벅스 카페메뉴 만들기

(2021.10.05 - 2021.11.02)

출처 : https://github.com/blackcoffee-study/moonbucks-menu/raw/main/src/images/moonbucks.png

-

스터디 리더 : 메이커준님 

-

2주차, Github를 통해 코드리뷰 진행

 

 

---

 

 

Github에서 PR로 코드리뷰를 하는 시간을 가졌다. 다른 사람에게 내 코드를 보여주고 리뷰를 받는 건 처음이라 긴장되기도 했지만, 한편으로 사람마다 쓰는 코드가 다르기 때문에 다른 개발자분들은 어떻게 코드를 쓰고 기능을 구현했나 궁금하기도 했다. 코드리뷰를 하기 전에 한 번도 하지 않은 사람을 위해 코드리뷰가 무엇인지, 어떻게 해야 좋을 지에 대한 메이커준님의 설명이 있었다.

 

 

---

 

코드리뷰란, 상호 간의 피드백을 주고 받는 것이다.

 

-

 

코드리뷰의 장점은,

• 코드가 팀의 소유가 된다.
• 더 큰 버그가 생기기 전에 버그의 결함율을 낮춰 그 가능성을 줄이고 이를 개선할 수 있다.
• 팀 단위에서 통일된 스타일을 유지할 수 있다.
• 다른 사람도 코드를 읽어야 한다는 점을 유의하기 때문에, 코드의 가독성을 올릴 수 있다.
• 코드를 다양한 방식으로 생각할 수 있다.
• 코드를 왜 이렇게 썼는 지 대답하고 이 과정에서 설득도 하며 소프트 스킬이 성장할 수 있다.

-

 

코드리뷰를 할 때 살펴볼 것들

• 필요없는 주석 또는 파일이 있는가
• 공백, 띄어쓰기, 컨벤션(코드 작성 후 밑에 있는 한 칸의 공백)이 잘 되어있는가
• 네이밍이 적절한가 
  • 불규칙적인 부분은 없는지, 의미를 알 수 없는 변수명은 없는지
  • 함수・변수명이 그 역할을 제대로 표현하고 있는지
  • 단수, 복수를 나누는 네이밍인지
  • 일시적인 것이라도 신경을 써서 했는지
• 중복되는 부분, 불필요한 코드, 반복되는 코드는 없는가
  • 줄일 수 있는 여지가 있다면 제안해본다. 예를 들어 DOM을 조작하면서 가장 많이 쓰이는 `document.querySelector`의 경우, 함수로 만들어서 `const $ = selector => document.querySelector(selector)` 방향으로 쓸 수도 있다.
• 리뷰하는 코드에서 배울만한 점을 칭찬
  • 내가 시도하지 못한 전략이 있는지
  • 이 사람으로부터 어떤 점을 배울 수 있는지

 

 

---

 

이 날 코드리뷰를 하고 나서 적었던 문장

 

"혼자서 인강만 듣다가 최근에 스터디를 시작해서 오늘 처음 코드리뷰라는 걸 했다. 진짜 재미있고 유익한 시간이었다! 같은 기능을 구현할 때 사람마다 다르게 코드를 써서 그걸 보는 재미도 있고, 내가 생각치 못한 부분까지 신경써서 쓴 코드를 보고 배울 점도 많았어."