separate-query-modifier

Separate Query from Modifier Skill

값 반환과 부수효과가 혼재된 메서드를 Query(값 반환, 순수)와 Modifier(부수효과, void)로 분리하여 CQS 원칙 적용.

GOAL

• 성공 = Query와 Modifier가 분리되어 커밋 완료됨
• 값을 반환하면서 동시에 상태를 변경하는 메서드가 식별됨
• CQS(Command-Query Separation) 위반 패턴 확인됨
• 사용자 확인 후 Query와 Modifier 분리
• 모든 테스트 통과

CONSTRAINTS

• 계열: System-wide — 후보 제시 후 승인 받고 적용 (../../references/refactoring-procedure.md §0·§3-B)

Hard Rules

• 동작 변경 금지 — 구조 개선만 수행
• 테스트 수정 금지 — 구조 변경이 테스트를 실패시키면 되돌리기
• 사용자 확인 없이 리팩토링 금지 — 모든 후보는 사용자 승인 후 실행
• 커밋 단위 — 1파일 x 1기법 = 1커밋 (논리적으로 연결된 파일은 함께)
• git add -A 금지 — 변경된 파일만 명시적으로 추가

CQS 원칙

Command-Query Separation (CQS):

• Query: 값을 반환하지만 상태를 변경하지 않음 (순수 함수)
• Command (Modifier): 상태를 변경하지만 값을 반환하지 않음 (void)

예외: 원자적 연산 (CAS, pop 등)은 분리하면 thread-safety 파괴

적용 패턴

Separate Query from Modifier 리팩토링 단계:

1. CQS 위반 메서드 식별

   • 값을 반환하면서 동시에 상태 변경
   • 부수효과(side effect)가 숨겨져 있음

2. Query 메서드 추출

   • 상태 변경 없이 값만 반환
   • 순수 함수로 구현 (같은 입력 → 같은 출력)
   • 여러 번 호출해도 안전

3. Modifier 메서드 추출

   • 상태만 변경하고 void 반환
   • 부수효과가 명시적으로 드러남
   • 메서드명으로 변경 의도 표현

4. 호출자 코드 업데이트

   • Query와 Modifier를 순차 호출
   • 필요에 따라 Query 결과를 Modifier에 전달

Before/After 예시

// Before: 값 반환 + 부수효과 혼재
public class OrderService {
    // 총액을 반환하면서 동시에 할인 적용 (부수효과)
    public double getTotalAndApplyDiscount(Order order) {
        double total = order.calculateTotal();
        
        // 숨겨진 부수효과 - 상태 변경
        if (total > 1000) {
            order.setDiscount(0.1);  // 10% 할인
        }
        
        return total;  // 값 반환
    }
    
    // 사용
    public void processOrder(Order order) {
        double total = getTotalAndApplyDiscount(order);  // 부수효과가 숨겨짐
        System.out.println("Total: " + total);
    }
}

// After: Query와 Modifier 분리
public class OrderService {
    // Query: 값만 반환, 상태 변경 없음 (순수)
    public double getTotal(Order order) {
        return order.calculateTotal();
    }
    
    // Modifier: 상태만 변경, void 반환
    public void applyDiscountIfEligible(Order order) {
        double total = order.calculateTotal();
        if (total > 1000) {
            order.setDiscount(0.1);
        }
    }
    
    // 사용: 부수효과가 명시적으로 드러남
    public void processOrder(Order order) {
        applyDiscountIfEligible(order);  // 상태 변경이 명확
        double total = getTotal(order);   // 순수한 값 조회
        System.out.println("Total: " + total);
    }
}