decompose-conditional

Decompose Conditional

GOAL

복잡한 조건문(if/then/else)을 의미 있는 이름의 메서드로 분해하여:

• 조건식의 의도를 명확한 메서드명으로 표현
• 분기별 로직을 독립적 메서드로 분리
• 가독성 및 재사용성 향상

Guard Clauses와 상호 보완:

• Guard Clauses: 중첩 해결 (early return)
• Decompose Conditional: 조건식 복잡성 해결 (의미 부여)

CONSTRAINTS

• 계열: Tidy — 후보 보고 후 승인 없이 적용 (../../references/refactoring-procedure.md §0·§3-A)
• 동작 변경 금지: 구조 개선만 수행 (기능 변경 없음)
• 테스트 수정 금지: 구조 변경이 테스트를 실패시키면 되돌리기
• 명시적 git add: git add -A 금지, 변경된 파일만 명시
• Extract Method 포함: 단일 클래스 내 완결 시에만 적용

적용 패턴

Before: 복잡한 조건식

if (date.isBefore(SUMMER_START) || date.isAfter(SUMMER_END)) {
    charge = quantity * winterRate + winterServiceCharge;
} else {
    charge = quantity * summerRate;
}

After: 조건식과 분기 모두 추출

if (isWinter(date)) {
    charge = winterCharge(quantity);
} else {
    charge = summerCharge(quantity);
}

private boolean isWinter(LocalDate date) {
    return date.isBefore(SUMMER_START) || date.isAfter(SUMMER_END);
}

private double winterCharge(int quantity) {
    return quantity * winterRate + winterServiceCharge;
}

private double summerCharge(int quantity) {
    return quantity * summerRate;
}

추가 예시: 복합 조건

// Before
if (customer.isPremium() && order.getTotal() > 1000 || 
    customer.isVip() && order.getTotal() > 500) {
    applySpecialDiscount();
} else {
    applyStandardDiscount();
}

// After
if (eligibleForSpecialDiscount(customer, order)) {
    applySpecialDiscount();
} else {
    applyStandardDiscount();
}

private boolean eligibleForSpecialDiscount(Customer customer, Order order) {
    return customer.isPremium() && order.getTotal() > 1000 ||
           customer.isVip() && order.getTotal() > 500;
}

추가 예시: 중첩 조건 + Guard Clause 병행

// Before
if (isActive) {
    if (hasPermission) {
        if (isValidDate) {
            processRequest();
        }
    }
}

// After (Guard Clause + Decompose Conditional)
if (!isActive) return;
if (!hasPermission) return;
if (!isValidDate) return;
processRequest();