Django 1.11中on_delete=models.SET_NULL未生效问题咨询

嘿，我来帮你理清楚这个问题——你遇到的情况其实是对on_delete的作用方向理解有偏差，再加上可能代码里有额外的删除逻辑在捣乱，咱们一步步拆解：

一、先搞懂on_delete的作用方向 #

首先纠正一个常见误区：on_delete是单向生效的，它只定义「当被关联的对象（也就是ForeignKey指向的模型，这里是Track）被删除时，当前模型（Disc）里的这个外键字段该怎么处理」，完全不是反过来的逻辑。

拿你的代码举例：

a_side_disc = models.ForeignKey(Track, null=True, on_delete=models.SET_NULL)

这句话的意思是：当某条Track记录被删除时，所有关联了这条Track的Disc记录，它们的a_side_disc字段会被设为NULL，而不是把Disc本身删掉。

所以你遇到的Disc被删除的情况，绝对不是on_delete=SET_NULL导致的，肯定是你的删除逻辑里有额外操作。

二、排查Disc被意外删除的原因 #

你提到自己写了Track的删除接口，还会先检查Track是否关联到in_jukebox=False的Disc。那大概率是你的删除逻辑里，除了删Track，还主动把关联的Disc也删了？或者有没有用Django的信号（比如pre_delete/post_delete）在监听Track删除事件，然后触发了Disc的删除？

建议你先检查这几个点：

• 翻看你的删除接口代码，是不是在调用track.delete()之后，又写了类似track.disc_set.all().delete()这种主动删Disc的代码？
• 检查项目里有没有注册针对Track的删除信号，比如下面这种：

  from django.db.models.signals import post_delete
  from django.dispatch import receiver
  from .models import Track
  
  @receiver(post_delete, sender=Track)
  def delete_discs_when_track_deleted(sender, instance, **kwargs):
      # 如果有这个信号，就会在Track删除时删掉所有关联的Disc
      instance.disc_set.all().delete()
  

• 另外，虽然你说数据库Schema是SET_NULL，但Django 1.11确实会在应用层处理部分on_delete逻辑（而非完全依赖数据库约束），所以核心还是要排查代码里的额外操作。

三、怎么实现「双向」的关联处理 #

如果你想要的是：

1. 删除Track时，关联Disc的外键自动设为NULL（这已经通过on_delete=SET_NULL配置好了，只要去掉额外删除逻辑就能生效）
2. 删除Disc时，对关联的Track做相应处理（比如如果Track没有其他Disc关联就删掉它）

那针对第二种情况，你可以通过以下两种方式实现：

方法1：用信号处理反向操作 #

写一个信号接收器，监听Disc的删除事件，然后处理关联的Track：

from django.db.models.signals import post_delete
from django.dispatch import receiver
from django.db.models import Q
from .models import Disc, Track

@receiver(post_delete, sender=Disc)
def handle_disc_deletion(sender, instance, **kwargs):
    # 处理a_side关联的Track
    if instance.a_side_disc:
        # 检查这个Track是否还有其他Disc关联
        has_other_discs = Disc.objects.filter(
            Q(a_side_disc=instance.a_side_disc) | Q(b_side_disc=instance.a_side_disc)
        ).exists()
        if not has_other_discs:
            instance.a_side_disc.delete()
    # 同理处理b_side关联的Track
    if instance.b_side_disc:
        has_other_discs = Disc.objects.filter(
            Q(a_side_disc=instance.b_side_disc) | Q(b_side_disc=instance.b_side_disc)
        ).exists()
        if not has_other_discs:
            instance.b_side_disc.delete()

方法2：覆盖Disc的delete()方法 #

直接在Disc模型里重写delete()方法，删除Disc时先处理关联的Track：

from django.db.models import Q

class Disc(models.Model):
    a_side_disc = models.ForeignKey(Track, null=True, on_delete=models.SET_NULL)
    b_side_disc = models.ForeignKey(Track, null=True, on_delete=models.SET_NULL)
    in_jukebox = models.BooleanField()

    def delete(self, *args, **kwargs):
        # 先处理关联的Track
        for track in [self.a_side_disc, self.b_side_disc]:
            if track:
                # 排除当前Disc，检查是否还有其他Disc关联这个Track
                has_other_discs = Disc.objects.filter(
                    Q(a_side_disc=track) | Q(b_side_disc=track)
                ).exclude(id=self.id).exists()
                if not has_other_discs:
                    track.delete()
        # 再删除当前Disc
        super().delete(*args, **kwargs)

⚠️ 注意：如果用批量删除（比如Disc.objects.filter(in_jukebox=False).delete()），重写的delete()方法不会被触发，这时候还是得用信号来处理。

总结 #

回到你的原始问题：

• on_delete=SET_NULL的作用方向是Track被删 → Disc的外键设为NULL，它绝对不会导致Disc被删除，所以你的问题根源是代码里有额外的删除逻辑。
• 先排查删除接口和信号，找到主动删Disc的代码，去掉后就能看到预期的SET_NULL效果。
• 如果需要双向处理关联对象，通过信号或重写模型delete()方法来实现反向逻辑。

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