Csharp Code Review

C# 코드 리뷰 스킬

OOP 원칙, SOLID 원칙, GoF 디자인 패턴, 최신 C# 기능, 성능 관점에서 C# 코드를 체계적으로 리뷰합니다.

중요: 모든 리뷰 결과는 반드시 한국어로 작성합니다. 코드 식별자, 기술 용어, 패턴 이름 등은 원문 그대로 유지하되, 설명·문제점·개선안 등 서술 부분은 한국어를 사용합니다.

인자

• $ARGUMENTS[0]: 대상 파일 또는 디렉토리 경로 (선택, 미지정 시 최근 수정된 .cs 파일을 탐색)

실행 단계

1단계: 리뷰 대상 식별

사용자가 파일을 지정하지 않은 경우:

• 최근 수정된 .cs 파일 확인
• 또는 사용자에게 리뷰할 파일/디렉토리 지정 요청

2단계: 코드 분석

대상 코드를 읽고 아래 관점에서 분석합니다.

리뷰 체크리스트

OOP 4대 원칙

SOLID 원칙

GoF 디자인 패턴 적용 기회

코드에서 다음 패턴을 적용할 수 있는 부분을 식별합니다:

생성 패턴

• 복잡한 객체 생성 → Builder
• 객체 생성 로직 분리 → Factory Method / Abstract Factory
• 전역 단일 인스턴스 → Singleton (주의: 남용 금지)

구조 패턴

• 호환되지 않는 인터페이스 연결 → Adapter
• 동적 기능 추가 → Decorator
• 복잡한 하위 시스템 단순화 → Facade
• 객체 트리 구조 → Composite

행동 패턴

• 교체 가능한 알고리즘 → Strategy
• 상태에 따른 행동 변경 → State
• 객체 간 통신 → Observer / Mediator
• 요청 처리 체인 → Chain of Responsibility
• 실행 취소/재실행 → Command + Memento

최신 C# 기능 (C# 12/13)

성능 리뷰

비동기 코드 리뷰

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