C# 코드 리뷰 스킬
OOP 원칙, SOLID 원칙, GoF 디자인 패턴, 최신 C# 기능, 성능 관점에서 C# 코드를 체계적으로 리뷰합니다.
중요: 모든 리뷰 결과는 반드시 한국어로 작성합니다. 코드 식별자, 기술 용어, 패턴 이름 등은 원문 그대로 유지하되, 설명·문제점·개선안 등 서술 부분은 한국어를 사용합니다.
인자
$ARGUMENTS[0]: 대상 파일 또는 디렉토리 경로 (선택, 미지정 시 최근 수정된.cs 파일을 탐색)
실행 단계
1단계: 리뷰 대상 식별
사용자가 파일을 지정하지 않은 경우:
- 최근 수정된
.cs파일 확인 - 또는 사용자에게 리뷰할 파일/디렉토리 지정 요청
2단계: 코드 분석
대상 코드를 읽고 아래 관점에서 분석합니다.
리뷰 체크리스트
OOP 4대 원칙
| 원칙 | 리뷰 항목 |
|---|---|
| 캡슐화 | private 필드, 프로퍼티 접근, 구현 세부사항 은닉 |
| 상속 | 적절한 상속 계층, 상속보다 합성 우선 |
| 다형성 | 인터페이스/추상 클래스 활용, virtual 메서드 적절성 |
| 추상화 | 적절한 추상화 수준, 불필요한 세부사항 노출 |
SOLID 원칙
| 원칙 | 리뷰 항목 | 위반 징후 |
|---|---|---|
| SRP | 클래스가 단일 책임을 갖는가? | 여러 이유로 클래스가 변경됨, 메서드 과다 |
| OCP | 확장에 열려있고 수정에 닫혀있는가? | 새 기능 추가 시 기존 코드 수정 필요, switch/if-else 체인 |
| LSP | 하위 타입이 상위 타입을 대체할 수 있는가? | 하위 클래스에서 예외 발생, 빈 메서드 오버라이드 |
| ISP | 클라이언트별로 인터페이스가 분리되어 있는가? | NotImplementedException, 미사용 메서드 |
| DIP | 추상화에 의존하는가? | new 직접 인스턴스화, 구체 클래스 타입 의존 |
GoF 디자인 패턴 적용 기회
코드에서 다음 패턴을 적용할 수 있는 부분을 식별합니다:
생성 패턴
- 복잡한 객체 생성 → Builder
- 객체 생성 로직 분리 → Factory Method / Abstract Factory
- 전역 단일 인스턴스 → Singleton (주의: 남용 금지)
구조 패턴
- 호환되지 않는 인터페이스 연결 → Adapter
- 동적 기능 추가 → Decorator
- 복잡한 하위 시스템 단순화 → Facade
- 객체 트리 구조 → Composite
행동 패턴
- 교체 가능한 알고리즘 → Strategy
- 상태에 따른 행동 변경 → State
- 객체 간 통신 → Observer / Mediator
- 요청 처리 체인 → Chain of Responsibility
- 실행 취소/재실행 → Command + Memento
최신 C# 기능 (C# 12/13)
| 기능 | 권장 시점 |
|---|---|
| Primary constructors | 간단한 초기화를 가진 클래스 |
| Collection expressions | 배열/리스트 초기화 [1, 2, 3] |
| required properties | 생성자 없이 필수 초기화 보장 |
| init-only setters | 불변 객체 |
| record types | 값 기반 동등성, DTO |
| Pattern matching | 복잡한 조건문, 타입 검사 |
| File-scoped namespaces | 들여쓰기 축소 |
| Raw string literals | 여러 줄 문자열, JSON, SQL |
성능 리뷰
| 카테고리 | 리뷰 항목 |
|---|---|
| 메모리 할당 | 핫 경로에서 불필요한 할당, Large Object Heap (>= 85KB) |
| Async/Await | 차단 호출 (.Result,.Wait()), ConfigureAwait 누락 |
| 컬렉션 | 잘못된 컬렉션 타입, LINQ 다중 열거 |
| 문자열 | 루프 내 문자열 결합, StringBuilder 미사용 |
| 박싱 | 불필요한 값 타입 박싱 |
| Span/Memory | Span, Memory 없는 버퍼 연산 |
비동기 코드 리뷰
.Result또는.Wait()호출 없음 (데드락 위험)- 라이브러리 코드에서
ConfigureAwait(false)사용 - 적절한 CancellationToken 전파
- 캐시된 결과가 있는 핫 경로에
ValueTask사용 - 스트리밍 데이터에
IAsyncEnumerable사용 - 이벤트 핸들러 외 async void 없음
코드 품질 리뷰
- 명명 규칙 (PascalCase, camelCase, _privateField, Async 접미사)
- null 안전성 (nullable reference types,
?.,??,??=) - 예외 처리 (구체적 예외,
when필터, 적절한 로깅) - IDisposable 패턴 준수 (using 문, Dispose 구현)
- 컬렉션 사용 (적절한 타입 선택, 효율적인 LINQ)
- 매직 넘버/문자열은 상수로 선언
- 중복 코드 제거
- 상속 불필요 클래스에
sealed적절히 사용
보안 리뷰
- 입력 유효성 검사 (SQL Injection, XSS, 경로 탐색)
- 민감 데이터 처리 (하드코딩된 비밀 없음, 적절한 암호화)
- 인증/권한 확인
- 보안 난수 생성 (
Random보안 용도 사용 금지) - XML External Entity (XXE) 방지
3단계: 리뷰 결과 출력
출력 형식
모든 내용은 한국어로 작성합니다. 코드 식별자와 기술 용어는 원문을 유지합니다.
# 코드 리뷰 결과
## 요약
- 파일: {파일 경로}
- 종합 평가: {우수/양호/개선 필요/심각}
- 주요 이슈: {N}건
- .NET 버전 준수: {.NET 8/9 기능 활용도}
## SOLID 원칙 분석
### SRP 위반 (심각도: 높음/보통/낮음)
- 위치: `ClassName.cs:line`
- 문제: {설명}
- 개선안: {코드 예시 포함}
### OCP 위반
...
## 최신 C# 기능 적용 기회
### {기능명} 권장
- 위치: `file.cs:line`
- 현재: {기존 코드}
- 개선: {최신 C# 문법}
- 이점: {설명}
## 성능 이슈
### {이슈 제목} (심각도: 높음/보통/낮음)
- 위치: `file.cs:line`
- 문제: {영향 포함 설명}
- 현재: {문제 코드}
- 개선: {최적화 코드}
- 효과: {예상 개선점}
## 비동기 코드 이슈
### {이슈 제목}
- 위치: `file.cs:line`
- 문제: {설명}
- 위험: {데드락/성능/기타}
- 해결: {코드 수정}
## 적용 가능한 디자인 패턴
### {패턴명} 패턴 권장
- 현재 코드: {문제점}
- 적용 시 이점: {설명}
- 예시 코드: {간략 예시}
## 보안 우려사항
### {이슈 제목} (심각도: 심각/높음/보통/낮음)
- 위치: `file.cs:line`
- 취약점: {설명}
- 해결: {코드 수정}
## 코드 품질 이슈
### {이슈 제목}
- 위치: `file.cs:line`
- 현재: {코드}
- 개선: {코드}
## 우선순위별 개선 사항
1. [심각] {보안 이슈}
2. [높음] {SOLID 위반, 성능 이슈}
3. [보통] {코드 품질, 최신 기능 적용}
4. [낮음] {스타일 개선}가이드라인
- 수정이 필요한 이슈만 보고합니다. 긍정적 피드백은 포함하지 않습니다.
- 구체적인 코드 예시와 함께 개선안을 제시합니다.
- 과도한 엔지니어링을 권장하지 않습니다.
- 맥락을 고려한 실용적인 제안을 합니다.
- 보안 이슈를 최우선으로 다룹니다.
- 대상.NET 버전을 고려하여 기능을 제안합니다.
- 최신 기능과 팀 친숙도 사이의 균형을 고려합니다.