为什么 django 的 model.save() 不调用 full_clean() ？

Question

我只是好奇是否有人知道 Django 的 ORM 不在模型上调用 full_clean 是否有充分的理由，除非它被保存为模型表单的一部分？

请注意，当您调用模型的 save() 方法时，不会自动调用 full_clean()。当您想要为自己手动创建的模型运行一步模型验证时，您需要手动调用它。
^{- 来自 Django 的 full_clean 文档}

（注意：更新了 Django 1.6 的引用...之前的 Django 文档也对 ModelForms 提出了警告。）

人们不希望这种行为有充分的理由吗？我认为如果您花时间向模型添加验证，您会希望每次保存模型时都运行验证。

我知道如何让一切正常工作，我只是在寻找解释。

Answer 1

AFAIK，这是因为向后兼容性。带有排除字段的 ModelForms、带有默认值的模型、pre_save() 信号等也存在问题。

您可能感兴趣的来源：

• http://code.djangoproject.com/ticket/13100
• http://groups.google.com/group/django-developers/browse_frm/thread/b888734b05878f87

Answer 2

由于兼容性考虑，django 内核中未启用保存时自动清理功能。

如果我们正在开始一个新项目并希望模型上的默认 save 方法可以自动清理，我们可以在保存每个模型之前使用以下信号进行清理。

from django.dispatch import receiver
from django.db.models.signals import pre_save, post_save

@receiver(pre_save)
def pre_save_handler(sender, instance, *args, **kwargs):
    instance.full_clean()

Answer 3

调用 full_clean 方法的最简单方法就是重写模型中的 save 方法：

class YourModel(models.Model):
    ...  
    
    def save(self, *args, **kwargs):
        self.full_clean()
        return super(YourModel, self).save(*args, **kwargs)

Answer 4

评论@Alfred Huang 的回答和评论。人们可以通过在当前模块（models.py）中定义类列表并在 pre_save 挂钩中检查它来将 pre_save 挂钩锁定到应用程序：

CUSTOM_CLASSES = [obj for name, obj in
        inspect.getmembers(sys.modules[__name__])
        if inspect.isclass(obj)]

@receiver(pre_save)
def pre_save_handler(sender, instance, **kwargs):
    if type(instance) in CUSTOM_CLASSES:
        instance.full_clean()

Answer 5

如果您希望始终确保模型验证，则全局 pre_save 信号可以很好地发挥作用。但是，它会在当前版本 (3.1.x) 中遇到 Django 身份验证问题，并可能导致您正在使用的其他应用程序的模型出现问题。

详细说明@Peter Shannon的答案，这个版本只会验证您执行它的模块内的模型，跳过验证 "raw" 保存 并将 dispatch_uid 添加到 避免重复信号。

from django.db.models.signals import pre_save
import inspect
import sys

MODELS = [obj for name, obj in
    inspect.getmembers(sys.modules[__name__], inspect.isclass)]

def validate_model(sender, instance, **kwargs):
    if 'raw' in kwargs and not kwargs['raw']:
        if type(instance) in MODELS:
            instance.full_clean()

pre_save.connect(validate_model, dispatch_uid='validate_models')

Answer 6

如果您想要确保一个模型至少具有一个 FK 关系，并且您不想使用 null=False 因为这需要设置默认 FK（这将是垃圾数据），我想出的最好方法是添加自定义 .clean() 和 .save() 方法。 .clean() 引发验证错误，.save() 调用 clean。这样，从表单和其他调用代码、命令行和测试中都可以强制执行完整性。如果没有这个，（AFAICT）就无法编写一个测试来确保模型与专门选择的（非默认）其他模型具有 FK 关系。

class Payer(models.Model):

    name = models.CharField(blank=True, max_length=100)
    # Nullable, but will enforce FK in clean/save:
    payer_group = models.ForeignKey(PayerGroup, null=True, blank=True,)

    def clean(self):
        # Ensure every Payer is in a PayerGroup (but only via forms)
        if not self.payer_group:
            raise ValidationError(
                {'payer_group': 'Each Payer must belong to a PayerGroup.'})

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

    def __str__(self):
        return self.name

Answer 7

我们可以将应用程序用作 settings.py 中的 INSTALLED_APPS 部分，而不是插入一段声明接收器的代码，

INSTALLED_APPS = [
    # ...
    'django_fullclean',
    # your apps here,
]

在此之前，您可能需要安装 django-fullclean 使用 PyPI：

pip install django-fullclean