코드 리뷰 가이드라인

코드 리뷰를 수행할 때 다음 사항들을 체계적으로 검토합니다.

1. 코드 품질

가독성

• 변수/함수 이름이 의미를 명확히 전달하는가?
• 함수가 한 가지 일만 하는가? (단일 책임 원칙)
• 코드가 자기 문서화되어 있는가?
• 불필요한 주석이 없는가? (코드로 설명 가능한 부분)

구조

• 함수/클래스 크기가 적절한가? (너무 길지 않은가?)
• 중복 코드가 없는가?
• 적절한 추상화 수준인가?
• 의존성이 명확한가?

2. 잠재적 버그

에러 처리

• 예외 상황이 적절히 처리되는가?
• null/undefined 체크가 되어있는가?
• 에러 메시지가 명확한가?

엣지 케이스

• 빈 배열/객체 처리
• 경계값 처리
• 타입 불일치 가능성

보안

• 사용자 입력이 검증되는가?
• SQL 인젝션, XSS 등 취약점 없는가?
• 민감 정보가 노출되지 않는가?

3. 성능

• 불필요한 연산이 없는가?
• 적절한 자료구조를 사용하는가?
• N+1 쿼리 문제가 없는가?
• 메모리 누수 가능성이 없는가?

4. 테스트

• 테스트가 작성되어 있는가?
• 테스트가 의미있는 케이스를 커버하는가?
• 테스트가 깨지기 쉽지 않은가?

5. 리뷰 결과 형식

리뷰 결과는 다음 형식으로 제공합니다:

요약

전체적인 코드 품질 평가 (1-2문장)

주요 발견사항

우선순위별로 정리:

1. 🔴 심각 - 반드시 수정 필요
2. 🟡 권장 - 수정하면 좋음
3. 🟢 제안 - 고려해볼 만한 개선점

상세 피드백

각 발견사항에 대해:

• 위치 (파일:라인)
• 문제 설명
• 개선 방안 (코드 예시 포함)

6. 리뷰 태도

• 비난이 아닌 개선 제안
• "왜?"에 대한 설명 포함
• 좋은 부분도 언급
• 구체적인 대안 제시