DRF中Recipe模型实例唯一性验证失效：无法拦截重复创建

问题描述 #

我正在开发一个Recipe模型，需要阻止创建字段值完全相同的重复实例，当存在相同字段值的实例时抛出验证错误。

当前验证代码 #

def validate_recipe(self, value):
    recipes = Recipe.objects.all().filter(author=self.context['request'].user)
    for rec in recipes:
        if rec._meta.fields == value._meta.fields:
            raise serializers.ValidationError(
                {'errors': 'Do you have this recipe!'}
            )
    return value

当前输出截图 #

更新说明 #

我尝试过使用unique_together但无效，我的模型代码如下：

class Recipe(models.Model):
    author = models.ForeignKey(
        User, on_delete=models.CASCADE, related_name='recipes')
    name = models.CharField(max_length=200)
    image = models.ImageField(
        upload_to='recipe/', null=True, blank=True)
    text = models.TextField()
    ingredients = models.ManyToManyField(
        Ingredients,
        through='RecipeIngredients'
    )
    tags = models.ManyToManyField(
        Teg
    )
    cooking_time = models.IntegerField(help_text='Cooking time')
    
    class Meta:
        unique_together = ('author', 'name', 'image',
                                            'text', 'cooking_time')

问题根源 #

1. 你当前的验证逻辑完全错误：rec._meta.fields == value._meta.fields比对的是模型的字段定义结构，不是字段的实际值，所以永远不会触发错误。
2. unique_together无效的原因：
   • image字段允许null=True，数据库里的NULL会被视为不同值，无法通过唯一约束限制；
   • unique_together不支持多对多字段（ingredients、tags），所以即使设置了也覆盖不了关联字段的重复判断。

正确的解决方法 #

方案1：序列化器中实现完整的重复校验

修改验证方法，逐个比对字段实际值，同时处理多对多字段的比对：

def validate_recipe(self, value):
    user = self.context['request'].user
    # 先过滤核心单值字段匹配的食谱
    query = Recipe.objects.filter(
        author=user,
        name=value.name,
        text=value.text,
        cooking_time=value.cooking_time
    )
    # 处理图片字段：当前实例有图就匹配图，没图就匹配空图
    if value.image:
        query = query.filter(image=value.image)
    else:
        query = query.filter(image__isnull=True)
    
    # 比对多对多字段的关联对象
    if query.exists():
        for recipe in query:
            # 转成集合，忽略顺序影响
            recipe_ing_ids = set(recipe.ingredients.values_list('id', flat=True))
            current_ing_ids = set(value.ingredients.values_list('id', flat=True))
            recipe_tag_ids = set(recipe.tags.values_list('id', flat=True))
            current_tag_ids = set(value.tags.values_list('id', flat=True))
            
            if recipe_ing_ids == current_ing_ids and recipe_tag_ids == current_tag_ids:
                raise serializers.ValidationError(
                    {'errors': '你已经创建过完全相同的食谱了！'}
                )
    return value

方案2：优化模型约束（仅覆盖核心单值字段）

如果不需要严格校验多对多字段，可以调整模型的唯一约束，避开图片字段的空值问题：

class Recipe(models.Model):
    # 字段定义保持不变
    class Meta:
        # Django 2.2+推荐用unique_constraints替代unique_together
        unique_constraints = [
            models.UniqueConstraint(
                fields=['author', 'name', 'text', 'cooking_time'],
                name='unique_recipe_core'
            )
        ]

这种方式只能限制核心单值字段的重复，多对多字段的重复仍需在序列化器中额外校验。

注意点 #

• 多对多字段用集合比对，避免因为关联顺序不同误判为不重复；
• 图片字段如果是新上传的文件，直接比对image字段可能有问题，建议额外计算图片哈希值再比对。

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