Django多对一关系下用户、Profile、组织模型写法正确性咨询

结论 #

两种写法均存在错误，第二种写法的逻辑更接近需求，但有语法、拼写和命名问题。

两种写法的问题梳理 #

第一种写法错误点 #

• 组织和用户的关联逻辑完全颠倒：ForeignKey是多对一关系，你将外键放在organization类中指向User，会变成「一个组织只能绑定一个用户，一个用户可以属于多个组织」，完全不符合「每个用户仅归属一个组织」的要求。
• 存在基础语法问题：Class 关键字应该小写为class，类定义末尾缺失冒号，organization类名不符合Python大驼峰命名规范。

第二种写法错误点 #

• 存在拼写错误：User类中的外键字段名写为organizatiob（多了字母b），外键指向的Organization类实际定义的是全小写的organization，无法匹配。
• 外键级联逻辑不合理：organization外键的on_delete设为CASCADE，会导致删除组织时直接删除所有关联的用户，业务风险极高。
• 同样存在基础语法问题：Class关键字小写错误，类定义末尾缺失冒号，organization类名不规范。
• Profile的on_delete设为PROTECT会导致删除用户时必须先手动删除关联的Profile，若需求是用户删除后资料同步删除，改为CASCADE更合理。

符合需求的正确写法示例 #

from django.db import models
from django.contrib.auth.models import AbstractUser

# 组织模型
class Organization(models.Model):
    name = models.CharField(max_length=100)
    # 其他组织相关字段自行补充

    def __str__(self):
        return self.name

# 自定义用户模型，推荐继承Django自带的AbstractUser避免重复造轮子
class User(AbstractUser):
    full_name = models.CharField(max_length=50, blank=True)
    # 每个用户归属一个组织，允许用户无组织的话保留null=True、blank=True，否则删掉这两个参数
    organization = models.ForeignKey(
        Organization,
        on_delete=models.PROTECT, # 存在关联用户的组织不允许删除，避免误操作
        null=True,
        blank=True,
        related_name="members" # 反向查询组织下所有用户可用 org.members.all()
    )

# 用户资料模型
class Profile(models.Model):
    user = models.OneToOneField(
        User,
        on_delete=models.CASCADE, # 用户删除时同步删除关联资料
        related_name="profile"
    )
    address = models.CharField(max_length=100)
    # 其他用户资料字段自行补充

    def __str__(self):
        return f"{self.user.username}的个人资料"

额外注意点 #

• 如果你不想自定义用户模型，直接使用Django默认的auth.User即可，将Profile和用户关联的外键指向auth.User就行。
• 如果要在创建用户时自动生成对应的Profile，可以通过Django信号实现。

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