代碼質量追求：基於度量指標的重構實戰——提取方法模式降低圈複雜度

「寫作就是重寫」，代碼亦然。重構不是重寫，而是在保持外部行為不變的前提下，通過結構優化提升可維護性。本文基於圈複雜度等客觀度量指標，實戰演示「提取方法」模式如何將圈複雜度 23 的高風險方法拆解為可測試、可維護的小單元，並解決私有方法測試難題，構建「度量→識別→重構→驗證」閉環。

重構核心認知

圈複雜度風險識別

PMD 等靜態分析工具報告揭示高風險方法：

// 報告示例
Method 'updateContent' has cyclomatic complexity of 23 (threshold 10)
Class: AccountAction
Risk: High defect density, difficult to test (needs 23+ test cases)

圈複雜度本質：測試覆蓋所有路徑所需的最小測試用例數。複雜度 23 意味著需要 23+ 個獨立測試用例才能達到分支覆蓋，實際維護成本極高。

提取方法模式實戰

重構前：高複雜度方法片段

public void updateContent(Account account) {
    // ... 大量其他代碼 ...
    
    // 風險片段：狀態文本生成邏輯 (複雜度貢獻 +5)
    String retstatus = null;
    if (lastChangedStatus != null && lastChangedStatus.size() > 0) {
        if (status.getId() == ((IStatus)lastChangedStatus.get(0)).getId()) {
            retstatus = "Change in Current status";
        } else {
            retstatus = "Account Previously Changed in: " + 
                ((IStatus)lastChangedStatus.get(0)).getStatusIdentification();
        }
    } else {
        retstatus = "No Changes Since Creation";
    }
    
    // ... 後續使用 retstatus ...
}

重構後：提取為獨立方法

// 原方法調用點簡化
String statusText = getStatusText(status, lastChangedStatus);

// 新增獨立方法：單一職責、易測試、可復用
private String getStatusText(IStatus status, List lastChangedStatus) {
    if (lastChangedStatus != null && !lastChangedStatus.isEmpty()) {
        IStatus lastStatus = lastChangedStatus.get(0);
        if (status.getId() == lastStatus.getId()) {
            return "Change in Current status";
        } else {
            return "Account Previously Changed in: " + lastStatus.getStatusIdentification();
        }
    }
    return "No Changes Since Creation";
}

重構效果量化

私有方法測試難題與解決方案

提取出的 `getStatusText` 為 `private`，JUnit 無法直接調用。四種解決方案對比：

PrivateAccessor 實戰用法

import junitx.util.PrivateAccessor; // JUnit-addons 模塊

public class AccountActionTest {
    @Test
    public void testGetStatusText_NoChanges() throws Throwable {
        AccountAction action = new AccountAction();
        
        // 反射調用私有方法
        String result = (String) PrivateAccessor.invoke(action, 
            "getStatusText", 
            new Class[]{IStatus.class, List.class},  // 參數類型
            new Object[]{null, null}                  // 參數值
        );
        
        assertEquals("No Changes Since Creation", result);
    }
    
    @Test
    public void testGetStatusText_SameStatus() throws Throwable {
        AccountAction action = new AccountAction();
        IStatus status = mock(IStatus.class);
        when(status.getId()).thenReturn(1L);
        
        IStatus lastStatus = mock(IStatus.class);
        when(lastStatus.getId()).thenReturn(1L);
        List list = Arrays.asList(lastStatus);
        
        String result = (String) PrivateAccessor.invoke(action,
            "getStatusText",
            new Class[]{IStatus.class, List.class},
            new Object[]{status, list}
        );
        
        assertEquals("Change in Current status", result);
    }
}

PrivateAccessor 核心 API

// 靜態調用 (私有靜態方法)
Object result = PrivateAccessor.invoke(AccountAction.class, "staticMethod", paramTypes, paramValues);

// 實例調用 (私有實例方法) - 最常用
Object result = PrivateAccessor.invoke(instance, "methodName", paramTypes, paramValues);

// 設置私有字段
PrivateAccessor.setField(instance, "fieldName", value);

// 獲取私有字段
Object value = PrivateAccessor.getField(instance, "fieldName");

重構最佳實踐清單

1. 紅燈重構：先運行測試確保綠燈 → 重構 → 運行測試驗證綠燈
2. 小步提交：每次提取方法後立即運行測試，而非批量修改後再測試
3. 命名即文檔：方法名應表達「做什麼」而非「怎麼做」，如 `calculateLateFee` 優於 `compute`
4. 參數對象化：參數超過 3 個考慮引入 Parameter Object 或 Builder
5. 避免過度提取：單行代碼或極簡邏輯無需提取，以可讀性為準

度量指標體系建設

持續集成集成策略

# Maven PMD 插件配置片段

    org.apache.maven.plugins
    maven-pmd-plugin
    
        
            /rulesets/java/design.xml  
        
        1.8
        true  
        5  // 閾值 10
    

結語

重構是對抗代碼熵增的唯一武器。通過圈複雜度等客觀指標識別風險熱點，應用「提取方法」等經典模式進行微創手術，配合 PrivateAccessor 解決私有方法測試難題，納入 CI 門禁形成「度量→重構→驗證」閉環，才能在交付壓力下保持代碼庫健康。記住：小就是美，測試是盾，度量是眼。