重构应先确认坏味道：频繁出错、测试易挂、扩展困难的代码才需动；提取函数需重命名变量、控制副作用、封装参数；类型判断应改为协议或注册表；测试是重构刹车系统，须覆盖边界、冻结时间、全量验证。
重构前先确认「坏味道」是否真需要动
不是所有看起来不顺眼的代码都值得重构。比如 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() 判断类型往往是重构起点
当看到类似这样的代码，就是典型的「类型分支坏味道」：
立即学习“Python免费学习笔记（深入）”；
织梦团购管理系统 x1 beta
下载
织梦团购管理系统拥有客户上千家，以其完美用户体验与极佳性能，为用户提供最值得信赖的团购系统平台，是您身边团购系统专家。新版本Dede5x1在v3基础上重构团购程序，优化团购源码,满足您个性化需求,为您提供最佳的团购系统解决方案. X1织梦团购源码主要特点： ·首家基于优雅的HMVC PHP5框架开发； ·遵循DRY，高安全性、轻量级代码 &mi
if isinstance(obj, PDFDocument):
return obj.render_pdf()
elif isinstance(obj, MarkdownDocument):
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。
