코드 단순화
최근 변경된 코드를 리뷰하고 단순화합니다.
Step 1: 변경 범위 파악
git diff --name-only HEAD~1 # 최근 변경 파일 확인Step 2: 리팩토링 패턴 적용
불필요한 추상화 제거
// BEFORE: 한 번만 쓰이는 헬퍼
function formatUserName(user: User) { return `${user.first} ${user.last}`; }
const name = formatUserName(user);
// AFTER: 인라인
const name = `${user.first} ${user.last}`;조건문 단순화
// BEFORE: 깊은 중첩
if (user) {
if (user.isActive) {
if (user.hasPermission) {
doThing();
}
}
}
// AFTER: 얼리 리턴
if (!user?.isActive || !user.hasPermission) return;
doThing();중복 제거
// BEFORE: 반복 패턴
const users = data.filter(d => d.type === 'user').map(d => d.name);
const admins = data.filter(d => d.type === 'admin').map(d => d.name);
// AFTER: 3회 이상 반복될 때만 추출
const namesByType = (type: string) => data.filter(d => d.type === type).map(d => d.name);주의: 2회 반복은 추출하지 않음. 3회부터 고려.
Step 2.5: 중복 코드 탐지
# jscpd — copy-paste 감지 (3줄 이상 중복)
npx jscpd src/ --min-lines 3 --reporters console 2>/dev/null | head -30
# ast-grep — 구조적 패턴 탐지
sg --pattern 'console.log($$$)' --lang ts 2>/dev/null | head -10Step 3: 검증
수정 후 반드시 확인:
npm run typecheck || npx tsc --noEmit # 타입 안전
npm test # 테스트 통과체크리스트
- 함수 50줄 이하
- 파일 800줄 이하
- 중첩 4단계 이하
- 매직 넘버 → 상수
- 명확한 변수명
- console.log 제거
- 사용하지 않는 import 제거
- any 타입 제거
절대 단순화하지 않을 것 (NEVER)
- NEVER 에러 핸들링 제거 — try/catch, 유효성 검사는 의도적 코드
- NEVER 생성된/벤더 파일 수정 — node_modules, generated, vendor 디렉토리
- NEVER 설정 파일 단순화 — tsconfig, eslint, webpack 등은 건드리지 않음
- NEVER 관련 없는 함수 병합 — 비슷해 보여도 도메인이 다르면 분리 유지
- NEVER 테스트 코드 단순화 — 테스트의 명시성은 의도적. DRY 적용 금지
- NEVER 타입 정의 제거 — 중복처럼 보여도 타입 안전성 유지
범위 결정
- 사용자가 특정 파일 지정 시 → 해당 파일만
- 지정 없으면 →
git diff --name-only HEAD~1(최근 커밋 변경 파일) - 변경 범위 밖 코드는 절대 수정하지 않음
원칙
- 동작은 변경하지 않음 (리팩토링만)
- 3줄 비슷한 코드 > 조기 추상화
- 과도한 최적화 지양
- 수정 전
git stash또는 현재 상태 확인 → 수정 후 검증 → 실패 시 원복