Simplify: 코드 리뷰 및 정리
변경된 모든 파일을 재사용성, 품질, 효율성 측면에서 검토합니다. 발견된 문제를 수정합니다.
1단계: 변경 사항 식별
git diff (또는 스테이징된 변경 사항이 있는 경우 git diff HEAD)를 실행하여 변경된 내용을 확인합니다.
2단계: 검토
코드 재사용
- 기존 유틸리티로 새 코드를 대체할 수 있는지 검색합니다.
- 중복 기능을 표시하고 기존 함수를 제안합니다.
- 기존 유틸리티를 사용할 수 있는 인라인 로직을 표시합니다.
코드 품질
- 파생 가능한 중복 상태 또는 캐시된 값.
- 약간의 변형이 있는 복사-붙여넣기 — 공유 추상화로 통합합니다.
- 코드가 무엇을 하는지 설명하는 불필요한 주석.
- 상수나 열거형이 이미 존재하는데 사용된 원시 문자열.
효율성
- 중복 계산, 반복된 파일 읽기.
- 병렬로 실행될 수 있는데 순차적으로 실행되는 독립적인 작업.
- 작업 전에 불필요한 존재 여부 확인(TOCTOU).
3단계: 수정
각 문제를 직접 수정합니다. 오탐(false positive)인 경우 메모하고 넘어갑니다. 완료되면 수정된 내용을 간략히 요약합니다(또는 코드가 이미 깔끔했음을 확인합니다).
동작을 보존합니다. 작은 기계적인 리팩터링을 선호합니다. 관련 없는 재작성은 피합니다.