Django QuerySet.update()疑似延迟执行引发唯一约束报错排查

我有一个包含order字段的Django模型Foo，其Meta类设置了['user', 'order']的unique_together约束。需要在序列化器的update方法中实现对象重排逻辑：将目标对象当前order设为null，调整当前order与新order之间的对象的order值，最后将目标对象order设为新值。

第一次尝试的代码因数据库更新顺序问题触发唯一约束错误：('user', 'order') = ('x', '2') already exist。查阅Django文档后修改代码，在PyCharm逐行调试时正常，但运行项目调用API时仍出现相同报错，疑似qs.update(order=None)执行延迟。请问问题出在哪里？有什么解决建议？

相关代码 #

模型代码 #

class Foo(models.Model):
    user = models.ForeignKey(...)
    order = models.PositiveIntegerField(null=True)

    class Meta:
        unique_together = [
            ['user', 'order'],
        ]

    def set_order(self, order):
        self.order = order
        self.save()

第一次尝试的update方法 #

def update(self, instance, validated_data):
    user = self.context['request'].user
    current_order = instance.order
    new_order = validated_data['order']

    instance.set_order(None)

    if new_order < current_order:
        qs = PIN.objects.exclude(order__isnull=True).filter(
            user_id=user.id, order__gte=new_order, order__lt=current_order
        ).update(order=F('order')+1)
    else:
        qs = PIN.objects.exclude(order__isnull=True).filter(
            user_id=user.id, order__gt=current_order, order__lte=new_order
        ).update(order=F('order')-1)

修改后的代码 #

user = self.context['request'].user
current_order = instance.order
new_order = validated_data['order']
instance.set_order(None)

if new_order < current_order:
    qs = Foo.objects.exclude(order__isnull=True).filter(
        user_id=user.id, order__gte=new_order, order__lt=current_order
    )
    qs_orders = qs.values_list('order', flat=True)
    qs_objs = list(qs)
    qs.update(order=None)           

    for idx, qs_order in enumerate(qs_orders):
        qs_objs[idx].order = qs_order + 1

    Foo.objects.bulk_update(qs_objs, ['order'])
else:
    qs = Foo.objects.exclude(order__isnull=True).filter(
        user_id=user.id, order__gt=current_order, order__lte=new_order
    ).update(order=F('order')-1)
    qs_orders = qs.values_list('order', flat=True)
    qs_objs = list(qs)
    qs.update(order=None)           

    for idx, qs_order in enumerate(qs_orders):
        qs_objs[idx].order = qs_order - 1

    Foo.objects.bulk_update(qs_objs, ['order'])

instance.set_order(new_order)

核心问题 #

1. 事务未显式控制：Django默认在视图/序列化器方法中自动开启事务，但未显式控制提交时机时，可能出现代码执行顺序与数据库实际更新顺序不一致的情况——比如instance.set_order(None)的更新还未提交到数据库，后续批量更新就已开始，引发约束冲突。
2. 修改后代码逻辑错误：else分支中先执行了qs.update(order=F('order')-1)，这一步已触发数据库更新，后续再设置order=None和批量更新属于重复错误操作，直接引发约束冲突。
3. 调试与运行模式差异：PyCharm单步调试时，每个数据库操作会及时提交；而运行时代码连续执行，事务可能延迟提交，导致中间状态未同步。

修复方案 #

1. 显式用事务包裹整个重排逻辑 #

确保所有数据库操作在同一个事务中原子执行，按顺序完成状态切换，避免中间状态触发约束。

2. 修正else分支逻辑错误 #

移除多余的update(order=F('order')-1)调用，统一逻辑为：先清空中间对象order，再重新赋值，最后设置目标对象的新order。

3. 避免直接用F()更新 #

F()表达式直接在数据库层面更新，调整多个对象order时会出现瞬时重复值，触发唯一约束。

修复后的代码 #

from django.db import transaction

def update(self, instance, validated_data):
    user = self.context['request'].user
    current_order = instance.order
    new_order = validated_data['order']

    # 用事务保证所有操作原子性
    with transaction.atomic():
        # 先释放目标对象原有order值
        instance.order = None
        instance.save()

        if new_order < current_order:
            # 向上移动：中间对象order+1
            qs = Foo.objects.filter(
                user=user, 
                order__gte=new_order, 
                order__lt=current_order
            ).order_by('order')
            items = list(qs)
            # 先清空中间对象order，避免赋值冲突
            qs.update(order=None)
            # 重新赋值order
            for item in items:
                item.order += 1
            Foo.objects.bulk_update(items, ['order'])
        else:
            # 向下移动：中间对象order-1，倒序处理避免冲突
            qs = Foo.objects.filter(
                user=user, 
                order__gt=current_order, 
                order__lte=new_order
            ).order_by('-order')
            items = list(qs)
            qs.update(order=None)
            for item in items:
                item.order -= 1
            Foo.objects.bulk_update(items, ['order'])

        # 最后设置目标对象的新order
        instance.order = new_order
        instance.save()

    return instance

额外优化建议 #

• 替换unique_together为Django 2.2+推荐的UniqueConstraint写法：

  class Meta:
      constraints = [
          models.UniqueConstraint(fields=['user', 'order'], name='unique_user_order')
      ]
  

• 可考虑使用django-ordered-model第三方库，其封装了成熟的重排逻辑，避免自行实现踩坑。

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