코드 리뷰 체크리스트 📋

좋은 FE 코드는 변경하기 쉬운 코드라는 기본 전제를 가지고 네가지 관점으로 코드 리뷰를 진행합니다.

자세한 예시는 Toss - Frontend Fundamentals 또는 Discussions를 참고 부탁드립니다.

가독성 (Readability)

• 복잡한 조건문에 적절한 이름을 붙여 의도가 명확한가?
• 매직 넘버가 상수로 적절히 추상화되어 있는가?
• 동시에 실행되지 않는 코드가 적절히 분리되어 있는가?
• 구현 상세가 적절한 수준으로 추상화되어 있는가?
• 삼항 연산자가 단순하고 이해하기 쉽게 사용되었는가?

예측 가능성 (Predictability)

• 함수/컴포넌트 이름이 실제 동작과 일치하며 혼란을 주지 않는가?
• 함수의 반환 타입이 일관되게 유지되는가?
• 숨겨진 부수 효과(side effect)없이 명시적으로 동작하는가?

응집도 (Cohesion)

• 함께 수정되어야 하는 코드가 같은 디렉토리에 위치하는가?
• 폼 필드들의 응집도가 요구사항에 맞게 설계되었는가?
  • 필드 단위로 분리가 필요한 경우 개별 관리되는가?
  • 폼 전체가 하나의 단위로 관리되어야 하는 경우 통합 관리되는가?

결합도 (Coupling)

• Props Drilling이 최소화되어 있는가?
• 각 컴포넌트/함수가 단일 책임을 가지도록 설계되었는가?
• 불필요한 코드 공통화로 인한 결합이 발생하지 않는가?

기타

• 테스트 코드가 작성되어 있는가?
• 변경사항이 기존 기능에 영향을 주지 않는가?