C# 코드 리뷰 스킬 Skill

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

성능 리뷰

| 카테고리 | 리뷰 항목 | |----------|-----------| | 메모리 할당 | 핫 경로에서 불필요한 할당, Large Object Heap (>= 85KB) | | Async/Await | 차단 호출 (.Result, .Wait()), ConfigureAwait 누락 | | 컬렉션 | 잘못된 컬렉션 타입, LINQ 다중 열거 | | 문자열 | 루프 내 문자열 결합, StringBuilder 미사용 | | 박싱 | 불필요한 값 타입 박싱 | | Span/Memory | Span<T>, Memory<T> 없는 버퍼 연산 |

비동기 코드 리뷰

[ ] .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 버전을 고려하여 기능을 제안합니다.
최신 기능과 팀 친숙도 사이의 균형을 고려합니다.

Agent Skills: C# 코드 리뷰 스킬

Install this agent skill to your local

Skill Files