如何避免Django信号重复触发？解决无限递归问题

问题根源 #

1. 无限递归：原信号每次触发后遍历所有QuestionForm实例并调用save()，每个save()又会触发post_save信号，陷入循环。
2. 性能低下：双重循环遍历所有表单和关联问题，时间复杂度为O(n²)，数据量大时性能极差。
3. 信号绑定错误：原代码中sender=form是错误的，应为QuestionForm模型类。

优化方案 #

1. 仅处理当前实例：无需遍历所有表单，只针对触发信号的实例计算总分。
2. 数据库聚合替代循环：用Django的Sum聚合查询直接从数据库计算总分，避免Python层面循环，复杂度降至O(1)。
3. 避免信号递归：使用QuerySet.update()方法更新字段，该方法不会触发模型的save()，因此不会再次触发post_save信号。
4. 覆盖多对多场景：监听m2m_changed信号，处理问题添加/移除时的总分更新。

优化后的代码 #

信号处理代码 #

from django.db.models import Sum
from django.db.models.signals import post_save, m2m_changed
from django.dispatch import receiver
from app.models import QuestionForm, Question

@receiver(post_save, sender=QuestionForm)
def update_form_max_score(sender, instance, **kwargs):
    # 用数据库聚合计算当前表单的总分，不存在则返回0
    total_score = instance.questions.aggregate(total=Sum('points'))['total'] or 0
    # 使用QuerySet.update()直接更新数据库，不会触发post_save信号
    QuestionForm.objects.filter(id=instance.id).update(max_score=total_score)

@receiver(m2m_changed, sender=QuestionForm.questions.through)
def update_form_score_on_m2m_change(sender, instance, action, **kwargs):
    # 仅在多对多关系添加/移除完成后更新总分
    if action in ['post_add', 'post_remove']:
        total_score = instance.questions.aggregate(total=Sum('points'))['total'] or 0
        QuestionForm.objects.filter(id=instance.id).update(max_score=total_score)

可选：保留模型方法的写法 #

如果需要保留原模型的set_score方法，也可以这样实现（需添加判断避免递归）：

@receiver(post_save, sender=QuestionForm)
def update_form_max_score(sender, instance, **kwargs):
    # 避免因更新max_score字段触发的重复计算
    if kwargs.get('update_fields') and 'max_score' in kwargs['update_fields']:
        return
    
    total_score = instance.questions.aggregate(total=Sum('points'))['total'] or 0
    instance.set_score(total_score)
    # 指定update_fields，仅更新max_score字段
    instance.save(update_fields=['max_score'])

关键说明 #

• 聚合查询：instance.questions.aggregate(Sum('points'))直接让数据库计算总和，比Python循环遍历效率高得多，尤其当问题数量较多时。
• 避免递归：QuerySet.update()绕过模型的save()方法，直接执行SQL更新语句，不会触发post_save信号，彻底解决递归问题。
• 多对多场景：多对多关系的修改不会触发QuestionForm的post_save，因此必须监听m2m_changed信号，在关系变更完成后更新总分。

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