code-refactor

程式碼重構技能

描述

主動偵測並執行程式碼重構，維持 DDD 架構和程式碼品質。

觸發條件

• 「重構這段程式碼」、「refactor」
• 「這個檔案太長了」
• 「模組化」、「拆分」
• 主動觸發：偵測到程式碼超過閾值時

---

核心原則

📜 依據憲法第 7.3 條「主動重構原則」

重構不是改天換地，而是持續的小步快跑
每次提交都應該比上次更乾淨

---

閾值設定

📏 長度閾值

類型	警告	強制重構
檔案	> 200 行	> 400 行
類別	> 150 行	> 300 行
函數	> 30 行	> 50 行
目錄檔案數	> 10 個	> 15 個

🔄 複雜度閾值

指標	警告	強制重構
圈複雜度	> 10	> 15
巢狀深度	> 3 層	> 4 層
參數數量	> 4 個	> 6 個
依賴數量	> 5 個	> 8 個

---

重構模式庫

1️⃣ Extract Method（提取方法）

觸發條件：函數過長、重複邏輯

# Before
def process_order(order):
    # 驗證訂單 (10 行)
    if not order.items:
        raise ValueError("Empty order")
    if order.total < 0:
        raise ValueError("Invalid total")
    # ... 更多驗證
    
    # 計算價格 (15 行)
    subtotal = sum(item.price * item.qty for item in order.items)
    tax = subtotal * 0.05
    total = subtotal + tax
    # ... 更多計算
    
    # 儲存訂單 (10 行)
    # ...

# After
def process_order(order):
    self._validate_order(order)
    total = self._calculate_total(order)
    self._save_order(order, total)

def _validate_order(self, order):
    """驗證訂單有效性"""
    if not order.items:
        raise ValueError("Empty order")
    # ...

def _calculate_total(self, order) -> Decimal:
    """計算訂單總金額（含稅）"""
    subtotal = sum(item.price * item.qty for item in order.items)
    return subtotal * Decimal("1.05")

2️⃣ Extract Class（提取類別）

觸發條件：類別職責過多、超過 150 行

# Before: User 類別包含太多職責
class User:
    def __init__(self, name, email, ...):
        self.name = name
        self.email = email
        self.address_line1 = ...
        self.address_line2 = ...
        self.city = ...
        self.postal_code = ...
    
    def validate_email(self): ...
    def format_address(self): ...
    def calculate_shipping(self): ...

# After: 提取 Address 值物件
@dataclass(frozen=True)
class Address:
    """地址值物件"""
    line1: str
    line2: str | None
    city: str
    postal_code: str
    
    def format(self) -> str:
        return f"{self.line1}\n{self.city} {self.postal_code}"

class User:
    def __init__(self, name: str, email: Email, address: Address):
        self.name = name
        self.email = email
        self.address = address