Django创建员工时save()执行两次致部门员工数翻倍问题求助

问题根源 #

你遇到的重复计数问题，核心原因是CreateView的form_valid方法里执行了两次保存操作：你手动调用了emp.save()，之后又调用父类的form_valid——而父类的form_valid内部会自动执行form.save()，这导致同一个员工对象被保存两次，进而触发了两次post_save信号，部门的personal字段因此被加了两次。

方案一：修改视图逻辑，避免重复保存 #

直接在commit=False的对象上设置full_name，要么交给父类处理保存，要么手动保存后跳过父类的保存流程：

写法1：交给父类处理保存

class EmpleadoCreateView(CreateView):
    model = Empleado
    form_class = EmpleadoForm
    success_url = reverse_lazy('empleado_list')

    def form_valid(self, form):
        emp = form.save(commit=False)
        # 设置full_name字段
        emp.full_name = f"{emp.nombre} {emp.apellido}"
        self.object = emp  # 将对象赋值给self.object，父类会自动完成保存
        return super().form_valid(form)

写法2：手动保存后直接重定向

class EmpleadoCreateView(CreateView):
    model = Empleado
    form_class = EmpleadoForm
    success_url = reverse_lazy('empleado_list')

    def form_valid(self, form):
        emp = form.save(commit=False)
        emp.full_name = f"{emp.nombre} {emp.apellido}"
        emp.save()  # 手动完成一次保存
        return redirect(self.get_success_url())  # 直接返回重定向，绕过父类的保存逻辑

方案二：优化信号处理函数（补漏方案） #

如果不想修改视图逻辑，可以在信号函数里增加判断，只在员工首次创建时执行计数更新：

from django.db.models.signals import post_save
from django.dispatch import receiver
from .models import Empleado, Oficina

@receiver(post_save, sender=Empleado)
def actualizar_contador_oficina(sender, instance, created, **kwargs):
    if created:  # 仅当员工是新创建时才执行计数增加
        oficina = instance.oficina
        oficina.personal += 1
        oficina.save(update_fields=['personal'])  # 只更新目标字段，提升性能

注：此方案只是补漏，优先推荐方案一从根源解决重复保存的问题。

额外建议 #

• 尽量保持代码的单一职责，避免视图和信号重复处理保存逻辑
• 如果性能允许，部门的员工计数可以通过反向查询实时计算（如oficina.empleado_set.count()），彻底避免数据不一致的问题；只有当查询性能成为瓶颈时，再考虑存储字段+信号维护的方式

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