AI 写的代码你看着不舒服——风格冲突的三种解法

# 你的风格：显式，逐行展开
def process_order(order):
    if not order.items:
        raise ValueError("订单无商品")
    total = 0
    for item in order.items:
        if item.stock < item.quantity:
            raise InsufficientStock(item.name)
        total += item.price * item.quantity
    return total

# AI 的风格：函数式，链式调用
def process_order(order):
    if not order.items:
        raise ValueError("订单无商品")
    return sum(
        item.price * item.quantity
        for item in order.items
        if item.stock >= item.quantity
    ) if all(item.stock >= item.quantity for item in order.items) else None

# ❌ 模糊的风格约束
你：帮我写一个订单处理函数，注意代码风格
AI：[给你一堆你看不懂的写法]
你：...算了，我自己改吧

# ✅ 精确的风格约束
你：帮我写一个订单处理函数。
   风格要求：
   - 不要用嵌套生成器表达式（展开成 for 循环）
   - 不要用三元表达式嵌套（拆成 if-else）
   - 早期返回（guard clause）优先于深层嵌套
   - 每个函数不超过 20 行

## 我的风格偏好（写在 SOUL.md 里，每次让 AI 写代码时自动生效）

### 必须遵守
- 不要嵌套生成器表达式（展开成 for 循环）
- 不要三元表达式嵌套（拆成 if-else 块）
- guard clause（早期返回）优先于深层 if 嵌套
- 避免单行函数体（除非确实只有一行）

### 优先选择
- 显式 for 循环 > 列表推导式（除非非常简单）
- 显式变量 > 链式调用（分配给变量再操作）
- 按逻辑分组（验证 → 计算 → 执行），用空行分隔

### 永远不要
- 用 `_` 作为"不重要但需要"的变量名（用真实名字）
- 在同一个表达式里做超过 3 件事

# 这些差异，建议接受
- AI 用 tuple unpacking 而不是索引访问 → 都可以
- AI 用 f-string 而不是 .format() → f-string 更好，接受
- AI 用 list comprehension 而不是 map() → 都可以

# 这些差异，必须统一
- AI 用递归代替迭代处理大数据集 → 栈溢出风险，必须改
- AI 把配置写在代码里而不是配置文件 → 部署时没法改，必须改
- AI 用异常代替条件判断控制流程 → 语义不清晰，必须改

def style_difference_is_acceptable(diff):
    # 会降低可读性吗？（对你，不是对所有人）
    if diff.reduces_readability_for_you:
        return False

    # 会埋下维护风险吗？
    if diff.introduces_maintenance_risk:
        return False

    # 会在团队里引起混淆吗？
    if diff.causes_confusion_in_team:
        return False

    # 纯粹审美差异 → 接受
    return True

# ❌ 模糊的重构指令
你：把这段代码的风格改成跟我一致
AI：[改了 3 处，漏了 5 处，你不知道它到底改了哪些]

# ✅ 精确的重构指令
你：把 process_order 函数做以下重构：
   1. 把嵌套生成器表达式展开为 for 循环
   2. 把三元表达式拆成 if-else
   3. 把 validator 抽成独立函数
   4. 不要改变任何业务逻辑
   验证方式：跑现有的 test_process_order，全部通过

# 这个模式特别有效
你：把下面这段代码，按我的风格改写。
   风格要求：
   1. [具体约束1]
   2. [具体约束2]
   不改变任何业务逻辑。

[粘贴 AI 之前写的代码]

AI：[给你改写后的版本，风格一致，逻辑不变]

# 风格不是你一个人的事——是你和 AI 之间的协议
class StyleProtocol:
    # 你应该约定的
    must_haves = [
        "不要嵌套生成器",
        "不要三元嵌套",
        "guard clause 优先",
        "配置文件分离",
        "函数不超过 20 行"
    ]

    # 你应该接受的（只是"不同"，不是"更差"）
    acceptables = [
        "tuple unpacking vs 索引",
        "f-string vs .format()",
        "list comprehension vs map()",
    ]

    # 你永远不要接受的
    never_accept = [
        "异常当控制流",
        "配置写死在代码里",
        "递归处理大数据集",
    ]