Django重写model的save方法生成PDF时出现无限循环如何解决？

根因分析 #

Django的FileField调用save()方法时，默认携带save=True参数，保存文件后会自动触发所属模型实例的save()方法。你在Fattura自定义save方法中生成PDF后调用self.pdf.save()，会重新触发Fattura.save()形成无限递归，直到达到Python默认递归深度阈值后抛出异常，整个事务回滚，因此无实例写入数据库，固定生成499个文件是递归达到深度限制前的生成次数。

解决方案 #

方案1：修改FileField保存参数（最简方案） #

调用self.pdf.save()时显式传入save=False，禁止保存文件时触发父模型保存，修改后逻辑如下：

def save(self, *args, **kwargs):
    # 其他前置逻辑
    # 可选判断：仅PDF为空/需要更新时生成，避免重复操作
    if not self.pdf:
        # 你的reportlab生成PDF逻辑，得到pdf文件对象
        self.pdf.save(f"fattura_{self.pk or 'new'}.pdf", pdf_file, save=False)
    super().save(*args, **kwargs)

该方案对原有逻辑侵入极小，不会影响ordine等其他字段的正常保存。

方案2：新增递归拦截标志位 #

如果需要在特定条件下重复生成PDF，可新增私有标志位拦截重复生成逻辑：

class Fattura(models.Model):
    # 原有字段定义（pdf、ordine等）
    _generating_pdf = False

    def save(self, *args, **kwargs):
        if self._generating_pdf:
            super().save(*args, **kwargs)
            return
        
        self._generating_pdf = True
        # PDF生成逻辑
        self.pdf.save(f"fattura_{self.pk or 'new'}.pdf", pdf_file)
        super().save(*args, **kwargs)
        self._generating_pdf = False

优化建议 #

如果仅当ordine关联字段变更时才需要重新生成PDF，可增加字段变更判断，减少不必要的IO操作：

def save(self, *args, **kwargs):
    need_regenerate = False
    if not self.pk:
        need_regenerate = True
    else:
        original_instance = Fattura.objects.get(pk=self.pk)
        if original_instance.ordine != self.ordine:
            need_regenerate = True
    
    if need_regenerate:
        # 执行PDF生成逻辑

内容的提问来源于stack exchange，提问作者H3x