Python 代码重构方法与案例

重构应先确认坏味道：频繁出错、测试易挂、扩展困难的代码才需动；提取函数需重命名变量、控制副作用、封装参数；类型判断应改为协议或注册表；测试是重构刹车系统，须覆盖边界、冻结时间、全量验证。

重构前先确认「坏味道」是否真需要动

不是所有看起来不顺眼的代码都值得重构。比如 if-else 嵌套三层，如果逻辑稳定、测试覆盖全、没人改它，强行拆成策略模式反而增加维护成本。真正该动的是：频繁修改却总出错的函数、单元测试一跑就挂的模块、新需求加进去要改五六个地方的类。

常见信号包括：duplicate code（相同逻辑在三处以上）、long method（单个函数超 40 行且含多层条件）、feature envy（一个方法总在调用另一个类的多个属性）。遇到这些，再动手不迟。

提取函数时别只剪切粘贴

把一段 30 行的逻辑剪出来单独成函数，只是第一步。关键在参数设计和副作用控制：

• 优先用 return 传递结果，避免通过 list.append() 或 dict.update() 修改传入的可变对象——这会让调用方难以预料状态变化
• 参数超过 3 个时，考虑封装成 dataclass 或命名元组，比如把 process_order(user_id, item_id, discount_rate, is_vip, timestamp) 改成 process_order(order: OrderRequest)
• 原函数里临时变量名如 tmp、res 搬过去后必须重命名，否则重构后更难读

用 isinstance() 判断类型往往是重构起点

当看到类似这样的代码，就是典型的「类型分支坏味道」：

if isinstance(obj, PDFDocument):
    return obj.render_pdf()
elif isinstance(obj, Markdo
wnDocument):
    return obj.render_html()
elif isinstance(obj, EPUBDocument):
    return obj.export_epub()

这不是不能运行，而是每次加新文档类型都要改这个 if-elif 链，违反开闭原则。可行路径有两条：

• 引入协议（Protocol）或抽象基类，让各类实现统一接口 render()，调用方完全不关心具体类型
• 用注册表模式：定义 RENDERERS = {PDFDocument: pdf_render, ...}，新增类型只需往字典里塞键值对，不碰原有逻辑

选哪条取决于扩展频率——如果新格式每月加一种，注册表更轻量；如果已稳定在 4 种以内，协议更利于 IDE 类型推导。

测试不是重构的负担，是唯一刹车系统

没有测试就重构 Python，等于蒙眼过独木桥。重点不在覆盖率数字，而在「边界是否被卡住」：

• 对输入做 assert 校验的函数，必须测 ValueError 是否如期抛出
• 涉及时间的逻辑（比如 is_expired()），别用 datetime.now() 硬写，用 freezegun 冻结时间，否则测试会随系统时钟飘移
• 重构后若发现某个测试从「通过」变「跳过」，不是漏写了，很可能是断言里依赖了被删掉的私有属性名

最常被忽略的一点：重构完别只跑自己改的那几个测试，至少执行一次完整模块的 pytest tests/ -k "not slow"——有些耦合藏得深，表面无关的改动会意外触发老 bug。

# 的是  # 多个  # 而在  # python  # markdown  # 都要  # 只需  # 没人  # 难以预料  # app  # 重命名  # 注册表  # 对象  # if  # html  # 接口  # 重构  # red  # bug  # 键值对  # 封装  # pdf  # ide  # elif  # append  # timestamp  # pytest 

相关栏目： <？ｍｕｍａ $count = M('archives')->where(['typeid'=>$field['id']])->count(); ?> 【 AI推广<？ｍｕｍａ echo $count; ?> 】 <？ｍｕｍａ $count = M('archives')->where(['typeid'=>$field['id']])->count(); ?> 【 SEO优化<？ｍｕｍａ echo $count; ?> 】 <？ｍｕｍａ $count = M('archives')->where(['typeid'=>$field['id']])->count(); ?> 【 技术百科<？ｍｕｍａ echo $count; ?> 】 <？ｍｕｍａ $count = M('archives')->where(['typeid'=>$field['id']])->count(); ?> 【 谷歌推广<？ｍｕｍａ echo $count; ?> 】 <？ｍｕｍａ $count = M('archives')->where(['typeid'=>$field['id']])->count(); ?> 【 百度推广<？ｍｕｍａ echo $count; ?> 】 <？ｍｕｍａ $count = M('archives')->where(['typeid'=>$field['id']])->count(); ?> 【 网络营销<？ｍｕｍａ echo $count; ?> 】 <？ｍｕｍａ $count = M('archives')->where(['typeid'=>$field['id']])->count(); ?> 【 案例网站<？ｍｕｍａ echo $count; ?> 】 <？ｍｕｍａ $count = M('archives')->where(['typeid'=>$field['id']])->count(); ?> 【 精选文章<？ｍｕｍａ echo $count; ?> 】