Django中如何避免Location模型重复数据入库？

核心问题解答：模型层还是视图层做校验？ #

优先在模型层实现约束，这是最可靠的方案。视图层校验只能拦截前端或常规接口的请求，但如果通过Django Shell、后台脚本或其他直接操作数据库的方式新增数据，视图层校验会失效。而模型层（数据库层面）的约束能从根源上阻止重复数据插入。

你的拼接对比方案是否高效可行？ #

可行但不够高效，且容易出现误差（比如字段拼接时的分隔符冲突、大小写处理不彻底等）。更优的方案是利用Django的联合唯一约束：

• 对于Django 2.2及以上版本，推荐使用UniqueConstraint（更灵活，支持条件约束）
• 旧版本可以用unique_together（已被标记为过时，但仍可使用）

比如，如果你认为country、state、district、taluka、city、address、latitude、longitude组合起来代表“同一地点”，且同一Agency下不能重复，可以在Location模型的Meta类中添加约束：

class Location(models.Model):
    # ... 其他字段 ...
    class Meta:
        constraints = [
            models.UniqueConstraint(
                fields=['agency', 'country', 'state', 'district', 'taluka', 'city', 'address', 'latitude', 'longitude'],
                name='unique_agency_location'
            )
        ]

这样当插入重复数据时，数据库会抛出唯一性错误，你可以在视图中捕获这个错误并返回提示信息。

模型代码中的其他问题及优化建议 #

Agency模型 #

1. BooleanField的choices错误：ngo_sector和govt_registered是BooleanField，但你用了字符串类型的choices值（"True"/"False"），且default设为字符串'False'。BooleanField的choices应该使用布尔值，default也必须是布尔类型：

   yes_no_choices = [(True, "Yes"), (False, "No")]
   ngo_sector = models.BooleanField(choices=yes_no_choices, null=True, blank=True, default=False)
   govt_registered = models.BooleanField(choices=yes_no_choices, default=False)
   

2. agency_sector缺少max_length：CharField必须指定max_length，否则生成迁移文件时会报错：

   agency_sector = models.CharField(choices=agency_sector_choices, max_length=50)
   

3. choices命名不规范：ngo_sector_choices被两个字段共用，改名为yes_no_choices更清晰。

Location模型 #

1. 外键字段名不符合规范：agency_id应该改为agency，Django会自动在数据库中生成agency_id列，字段名用关联模型的小写单数更符合Django惯例：

   agency = models.ForeignKey(Agency, on_delete=models.SET_NULL, null=True)
   

2. on_delete参数错误：models.SET("Deleted User")无效，因为SET需要接收一个可调用对象或能被赋值给外键的合法值。如果希望删除Agency后Location的agency字段设为null，应该用SET_NULL并设置null=True；如果需要设置默认值，要确保默认值是Agency的实例或ID：

   # 示例：删除Agency后设为null
   agency = models.ForeignKey(Agency, on_delete=models.SET_NULL, null=True, blank=True)
   

3. latitude和longitude字段类型错误：用CharField存储经纬度不合适，应该用DecimalField（精度更高）或FloatField：

   latitude = models.DecimalField(max_digits=9, decimal_places=6, default=21.7679)
   longitude = models.DecimalField(max_digits=9, decimal_places=6, default=78.8718)
   

4. __str__方法过于冗长：可以简化为更简洁的格式，比如：

   def __str__(self):
       return f"{self.agency} - {self.city}, {self.state}"
   

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