코드 리뷰 가이드라인
코드 리뷰를 수행할 때 다음 사항들을 체계적으로 검토합니다.
1. 코드 품질
가독성
- • 변수/함수 이름이 의미를 명확히 전달하는가?
- • 함수가 한 가지 일만 하는가? (단일 책임 원칙)
- • 코드가 자기 문서화되어 있는가?
- • 불필요한 주석이 없는가? (코드로 설명 가능한 부분)
구조
- • 함수/클래스 크기가 적절한가? (너무 길지 않은가?)
- • 중복 코드가 없는가?
- • 적절한 추상화 수준인가?
- • 의존성이 명확한가?
2. 잠재적 버그
에러 처리
- • 예외 상황이 적절히 처리되는가?
- • null/undefined 체크가 되어있는가?
- • 에러 메시지가 명확한가?
엣지 케이스
- • 빈 배열/객체 처리
- • 경계값 처리
- • 타입 불일치 가능성
보안
- • 사용자 입력이 검증되는가?
- • SQL 인젝션, XSS 등 취약점 없는가?
- • 민감 정보가 노출되지 않는가?
3. 성능
- • 불필요한 연산이 없는가?
- • 적절한 자료구조를 사용하는가?
- • N+1 쿼리 문제가 없는가?
- • 메모리 누수 가능성이 없는가?
4. 테스트
- • 테스트가 작성되어 있는가?
- • 테스트가 의미있는 케이스를 커버하는가?
- • 테스트가 깨지기 쉽지 않은가?
5. 리뷰 결과 형식
리뷰 결과는 다음 형식으로 제공합니다:
요약
전체적인 코드 품질 평가 (1-2문장)
주요 발견사항
우선순위별로 정리:
- •🔴 심각 - 반드시 수정 필요
- •🟡 권장 - 수정하면 좋음
- •🟢 제안 - 고려해볼 만한 개선점
상세 피드백
각 발견사항에 대해:
- •위치 (파일:라인)
- •문제 설명
- •개선 방안 (코드 예시 포함)
6. 리뷰 태도
- •비난이 아닌 개선 제안
- •"왜?"에 대한 설명 포함
- •좋은 부분도 언급
- •구체적인 대안 제시