def process_claim(claim_data):
    if not claim_data.get("id"):
        return "Error"
    db = connect_to_db("prod_db")
    db.execute(f"INSERT INTO claims VALUES ({claim_data['id']})")
    total = claim_data['amount'] * 1.15
    send_email("admin@company.com", f"Claim {total} processed")
    return "Success"

class ClaimValidator:
    def validate(self, data):
        if not data.get("id"):
            raise ValidationError("Missing ID")

class TaxCalculator:
    def calculate(self, amount, region_code):
        rate = self._get_rate(region_code)
        return amount * rate

class ClaimService:
    def __init__(self, validator, calculator, repository, notifier):
        self.validator = validator
        self.calculator = calculator
        self.repository = repository
        self.notifier = notifier

    def execute(self, claim_data):
        self.validator.validate(claim_data)
        total = self.calculator.calculate(claim_data['amount'], "US")
        self.repository.save(claim_data)
        self.notifier.send(f"Claim {total} processed")

Soni 说这是最重要的教训。重构前必须先写特征测试（characterization test）。

很多人的直觉是：代码太烂了，先重构再补测试。这个顺序简直就是灾难。烂代码的"正确行为"往往不是设计出来的，而是历史沉淀出来的——某个边界条件在线上跑了三年没出过事，但你自己看代码根本看不出这个条件。重构的时候把它改掉了，上线就炸。

特征测试跟单元测试不一样：它不验证"结果对不对"，只记录"当前行为是什么"。输入 X，当前输出 Y，记录下来。重构后输入同样的 X，输出还是 Y，就说明没改坏。

具体做法：从最常出问题的模块开始。跑一遍现有输入输出，把结果录成测试。然后用 LLM 重构这个模块，跑测试，通过就继续，不通过就回退。

LLM 重构的一个特别问题：AI 太喜欢上设计模式。你给它一段简单的 if/else，它可能还你一个策略模式 + 工厂模式 + 依赖注入容器，三重抽象套在一起。

Soni 发现 Gemini 3 也有这个倾向。原文里的代码被加了四个类和一个构造函数注入，在 2000 行的单体脚本背景下，这个抽象层级可能是合适的。但如果是 50 行的工具函数，同样的做法就是浪费。

原则很简单：只引入能解决当前问题的复杂度。简单函数不需要类，一个类不需要三层抽象。"万一以后要扩展呢"不是加抽象的理由。大部分"以后"根本不会来，真来了再改也来得及。

这个陷阱对 LLM 重构尤其危险。"一把梭"的意思是：把整个项目全喂给 AI，期望它吐出一个完整的重构版。听起来省事，实际是给自己挖坑。

别这么干。Soni 说他见过的最惨案例就是团队把整个系统丢给 AI，拿回来一个不能跑的 Frankenstein。

正确做法：一次重构一个模块。改完就跑测试，确认系统还在跑。再改下一个。AI 能帮你加速每个模块，但控制节奏的是你。

import pytest

def test_process_claim_missing_id():
    """输入缺 id → 当前返回 'Error'"""
    result = process_claim({"amount": 100})
    assert result == "Error"

def test_process_claim_normal():
    """正常输入 → 当前返回 'Success'"""
    result = process_claim({"id": "A001", "amount": 200})
    assert result == "Success"

def test_process_claim_tax_calculation():
    """税额 = amount * 1.15，需要 mock 掉 DB 和邮件才能验证"""
    pass