리뷰할 수 있는 크기로 변경 나누기, 대안을 선택하는 기준 정리

기능 추가와 이름 변경, 포맷 수정이 한 번에 섞이면 리뷰어가 실제 동작 변화를 찾기 어렵다. 변경을 나누는 기준은 파일 수보다 설명 가능한 목적이다.

문제를 푸는 방법이 둘 이상이면 기능의 개수만 비교하기 쉽다. 하지만 운영 중 설명할 수 있는지, 실패했을 때 되돌릴 수 있는지, 다음 동료가 고칠 수 있는지도 선택의 일부다. 이번에는 현재 조건에서 필요한 성질과 감수할 비용을 나누어 적는다.

장점과 비용을 함께 보기

하나의 변경이 해결하는 문제와 검증 방법을 먼저 적는다. 독립적인 정리는 별도 변경으로 분리하고 의존성이 있으면 적용 순서를 명시한다.

이번에 확인할 질문: 선택한 방법의 장점이 지금의 제약과 실제로 연결되어 있는가?

작게 확인하는 예제

다음 예제는 핵심 조건을 드러내기 위한 출발점이다. 서비스의 전체 구현을 대신하지 않으므로 실제 입력 범위와 실패 처리 규칙을 함께 읽어야 한다.

Change A: rename without behavior changes
Change B: implement the new validation rule
Change C: migrate existing callers

Each change includes its own observable check.

리뷰 설명을 한 문단으로 적었을 때 서로 다른 목적이 반복해서 등장하는지 확인한다.

놓치기 쉬운 조건

줄 수만 줄이려고 서로 의존하는 코드를 반쪽씩 나누면 중간 상태가 깨질 수 있다. 각 변경이 검증 가능한 상태인지 함께 봐야 한다.

확인할 부분	살펴볼 내용
기준 동작	리뷰 설명을 한 문단으로 적었을 때 서로 다른 목적이 반복해서 등장하는지 확인한다.
비교 조건	입력과 실행 순서를 고정하고 한 번에 한 조건만 바꾼다.
적용 범위	예제가 생략한 의존성과 실제 서비스의 제약을 구분한다.

다음 변경에 적용할 순서

1. 필수 조건과 있으면 좋은 조건을 분리한다.
2. 선택하지 않은 대안이 유리해지는 상황도 한 가지 적는다.
3. 조건이 바뀌면 결정을 다시 검토할 시점을 남긴다.

다른 방법을 배제한 이유가 기록되어 있으면 같은 논의를 처음부터 반복하지 않아도 된다. 선택은 환경에 따라 달라질 수 있다. 무엇을 포기했는지 설명하는 문장이 구현을 이해하는 데 오히려 더 직접적인 도움이 될 때가 많다.