python - 这是验证 Django 模型字段的方法吗？

Question

据我了解，当创建 Django 应用程序时，数据在插入模型实例之前由表单验证，然后将其写入数据库。但是，如果我想在数据模型层创建一个额外的保护层，我所做的是否低于当前的“最佳实践”？我试图确保审稿人的姓名不能省略，也不能留空。我是否应该像在此处所做的那样在“干净”方法中放置任何自定义验证，然后让“保存”调用“full_clean”来调用“干净”？如果没有，首选方法是什么？谢谢。

class Reviewer(models.Model):
    name = models.CharField(max_length=128, default=None)

    def clean(self, *args, **kwargs):
        if self.name == '':
            raise ValidationError('Reviewer name cannot be blank')
        super(Reviewer, self).clean(*args, **kwargs)

    def full_clean(self, *args, **kwargs):
        return self.clean(*args, **kwargs)

    def save(self, *args, **kwargs):
        self.full_clean()
        super(Reviewer, self).save(*args, **kwargs)

Answer 1

首先，您不应该像以前那样覆盖 full_clean。来自 django docs on full_clean :

Model.full_clean(exclude=None)
This method calls Model.clean_fields(), Model.clean(), and Model.validate_unique(), in that order and raises a ValidationError that has a message_dict attribute containing errors from all three stages.

所以 full_clean 方法已经调用了 clean，但是通过覆盖它，您阻止了它调用其他两个方法。

其次，在 save 方法中调用 full_clean 是一种折衷。请注意，当验证模型表单时，已经调用了 full_clean，例如在 Django 管理员中。因此，如果您在 save 方法中调用 full_clean，那么该方法将运行两次。

通常不期望 save 方法引发验证错误，有人可能调用 save 并没有捕获到产生的错误。但是，我喜欢您调用 full_clean 而不是在 save 方法本身中进行检查 - 这种方法允许模型表单首先发现问题。

最后，您的 clean 方法会起作用，但您实际上可以在模型字段本身中处理您的示例案例。将您的 CharField 定义为

name = models.CharField(max_length=128)

blank选项将默认为 False。如果该字段为空，则在运行 full_clean 时将引发 ValidationError。将 default=None 放入您的 CharField 不会造成任何伤害，但是当您实际上不允许 None 作为一个值。