Django同一方法内多次调用模型对象save方法是否合理？

结论先行 #

你当前的多次调用save()的写法功能上可以运行，但绝大多数业务场景下属于不合理的实践，优先推荐调整为「子方法返回关联对象、主方法末尾统一赋值后仅调用一次save()」的方案。

两种方案的优缺点对比 #

方案1：同一流程内多次调用save()（你当前的写法） #

优点

• 编码成本低，无需修改子方法返回逻辑，子方法内完成字段赋值后直接落库，不依赖上层调用方的保存操作
• 适合强依赖实时落库的特殊场景：如果子方法执行完成后需要立即触发数据库触发器、异步任务、或者其他进程读取数据库最新状态，该写法可以保证状态实时同步到数据库

缺点

• 性能损耗大：每次save()都会触发一次数据库UPDATE请求，你的代码里共会产生3次数据库IO，高并发场景下会明显拖慢响应速度，同时增加数据库压力
• 数据一致性风险高：如果任意一次save()之后流程报错中断，会出现部分字段更新、部分字段未更新的脏数据。比如sub_method_1的save()成功后，sub_method_2执行报错，那么ModelA实例只有e字段是新值，c/d/f/g/h均为旧值，业务数据不一致
• 信号逻辑异常：Django的pre_save、post_save等模型信号会在每次调用save()时触发，多次调用会导致信号逻辑被重复执行，大概率引发业务bug
• 原子性保障成本高：多次save()天然不具备原子性，要保证所有修改要么全成功要么全失败，需要手动给整个流程加事务包裹，额外增加编码复杂度

方案2：子方法返回关联对象，主方法统一调用一次save() #

优点

• 性能更优：全程仅产生1次数据库UPDATE请求，减少不必要的IO开销，接口响应速度更快
• 数据一致性强：所有字段修改一次性落库，天然具备原子性，要么全部更新成功要么全部失败，不会出现脏数据
• 信号触发符合预期：pre_save、post_save等信号仅会触发一次，避免信号逻辑重复执行的问题
• 代码可维护性更高：子方法仅负责生成关联对象，不修改传入实例的状态，职责更单一，后续迭代修改时不容易产生副作用

缺点

• 需要调整现有代码逻辑，修改子方法返回关联对象，在主方法中完成字段赋值
• 无法满足特殊场景的实时落库需求，如果业务要求子步骤执行后必须立即落库，该方案不适用

最佳实践建议 #

Django官方模型开发规范明确说明：每次save()都会触发一次SQL写入请求，对于同一实例的多次字段更新，优先合并为一次save()调用以提升性能、保障数据一致性。
除非你明确需要多次save()的特殊场景（比如依赖每次save()触发的触发器、信号逻辑，或者中间步骤需要其他进程读取最新库表状态），否则所有场景都优先选择统一save()的方案。如果确实需要多次save()，必须给整个流程加数据库事务包裹，避免出现脏数据。

内容的提问来源于stack exchange，提问作者Sencer H.