리뷰할 수 있는 크기로 변경 나누기, 변경 전후를 비교하는 방법

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

개선한 코드가 더 좋아 보인다는 인상과 실제 동작의 개선은 구분해야 한다. 입력과 실행 조건이 다르면 차이의 원인을 알기 어렵다. 이번 기록에서는 비교 기준을 먼저 정하고, 바꾼 부분이 기대한 효과를 냈는지 확인하는 절차를 다룬다.

같은 조건에서 얻은 증거

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

이번에 확인할 질문: 관찰한 차이를 이번 변경 때문이라고 설명할 만큼 비교 조건이 같은가?

작게 확인하는 예제

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

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. 효과가 확인되지 않으면 추가 복잡도를 받아들일 이유를 다시 검토한다.

비교 결과는 특정 조건에서 얻은 증거다. 이를 모든 사용자와 모든 실행 환경으로 곧바로 확대하지 않는다. 입력의 크기와 조건을 함께 남겨야 다른 상황에서 다시 검증할 때도 출발점으로 사용할 수 있다.