【问题标题】:Using Validators in Django not working properly在 Django 中使用验证器无法正常工作
【发布时间】:2021-02-26 00:01:33
【问题描述】:

我最近了解了验证器及其工作原理,但我正在尝试在我的博客项目中添加一个函数,以便在使用坏词时引发错误。 我在 txt 中有一个坏词列表,并将代码添加到 models.py 中,问题是由于某种我不确定的原因没有任何内容被阻止。

这是models.py

class Post(models.Model):
       title = models.CharField(max_length=100, unique=True)
       ---------------other unrelated------------------------

def validate_comment_text(text):
    with open("badwords.txt") as f:
        censored_word = f.readlines()
    words = set(re.sub("[^\w]", " ", text).split())
    if any(censored_word in words for censored_word in CENSORED_WORDS):
        raise ValidationError(f"{censored_word} is censored!")

class Comment(models.Model):
    user = models.ForeignKey(User, on_delete=models.CASCADE)
    post = models.ForeignKey(Post, on_delete=models.CASCADE)
    content = models.TextField(max_length=300, validators=[validate_comment_text])
    updated = models.DateTimeField(auto_now=True)
    created = models.DateTimeField(auto_now=True)

这里是views.py:

class PostDetailView(DetailView):
    model = Post
    template_name = "blog/post_detail.html"  # <app>/<model>_<viewtype>.html

    def get_context_data(self, *args, **kwargs):
        context = super(PostDetailView, self).get_context_data()
        post = get_object_or_404(Post, slug=self.kwargs['slug'])
        comments = Comment.objects.filter(
            post=post).order_by('-id')
        total_likes = post.total_likes()
        liked = False
        if post.likes.filter(id=self.request.user.id).exists():
            liked = True

        if self.request.method == 'POST':
            comment_form = CommentForm(self.request.POST or None)
            if comment_form.is_valid():
                content = self.request.POST.get('content')
                comment_qs = None

                comment = Comment.objects.create(
                    post=post, user=self.request.user, content=content)
                comment.save()
                return HttpResponseRedirect("blog/post_detail.html")
        else:
            comment_form = CommentForm()

        context["comments"] = comments
        context["comment_form"] = comment_form
        context["total_likes"] = total_likes
        context["liked"] = liked
        return context

    def get(self, request, *args, **kwargs):
        res = super().get(request, *args, **kwargs)
        self.object.incrementViewCount()
        if self.request.is_ajax():
            context = self.get_context_data(self, *args, **kwargs)
            html = render_to_string('blog/comments.html', context, request=self.request)
            return JsonResponse({'form': html})
        return res

class PostCommentCreateView(LoginRequiredMixin, CreateView):
    model = Comment
    form_class = CommentForm

    def form_valid(self, form):
        post = get_object_or_404(Post, slug=self.kwargs['slug'])
        form.instance.user = self.request.user
        form.instance.post = post
        return super().form_valid(form)

这是我的试用版,但没有成功

def validate_comment_text(sender,text, instance, **kwargs):
    instance.full_clean()
    with open("badwords.txt") as f:
        CENSORED_WORDS = f.readlines()

    words = set(re.sub("[^\w]", " ", text).split())
    if any(censored_word in words for censored_word in CENSORED_WORDS):
        raise ValidationError(f"{censored_word} is censored!")

pre_save.connect(validate_comment_text, dispatch_uid='validate_comment_text')

我是新手,如果你能对答案提供一些解释,我将不胜感激,这样我就可以避免重复同样的错误。

【问题讨论】:

  • (1) 您的缩进已关闭。我假设这只是对 SO 的复制/粘贴问题。 (2) 此检查将区分大小写 - 如果您将单词存储为全部大写/小写或其他内容,则很可能很多都不匹配。建议将您的文本全部转换为小写,并将所有小写存储在CENSORED_WORDS 中,看看是否有帮助。
  • @michjnich 这只是一个复制/粘贴错误导致缩进错误,但这不是不工作的问题

标签: python django


【解决方案1】:

验证器仅在您使用 ModelForm 时运行。如果直接调用comment.save(),验证器将不会运行。 link to docs

因此,您需要使用ModelForm 验证该字段,或者您可以添加pre_save 信号并在那里运行验证(您需要手动调用该方法,或使用full_clean 来运行验证) . 比如:

from django.db.models.signals import pre_save

def validate_model(sender, instance, **kwargs):
    instance.full_clean()

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

【讨论】:

  • 不清楚为什么这被否决了?问题中没有足够的信息来判断它是否是实际的解决方案,但似乎是一个合理的建议!
  • 我同意:这个答案很有用,所以我会支持它;)出于个人喜好,我宁愿覆盖 save() ,因为这对我来说似乎更明确。当您需要在另一个应用程序中声明的模型的钩子时,信号更合适
  • @Aman Garg 我一直在尝试使用 pre_save 但我没有成功实施它,我已经用我的试用版更新了帖子。
  • @michjnich 我已经包含了帖子模型和视图,以便为问题添加更多细节。谢谢
  • @Shiko validate_comment_text(sender,text, instance, **kwargs) 没有正确的签名。第二个参数是instance,因此只需更新答案中提到的签名即可。 link
【解决方案2】:

我确信有很多方法可以处理这个问题,但我最终决定在我所有的 Django 项目中采用一种通用做法:

当模型需要验证时,我重写 clean() 以将所有验证逻辑收集在一个地方并提供适当的错误消息。

在clean()中,可以访问所有模型字段,不需要返回任何东西;只需根据需要提出 ValidationErrors:

from django.db import models
from django.core.exceptions import ValidationError


class MyModel(models.Model):

    def clean(self):
         
        if (...something is wrong in "self.field1" ...) {
            raise ValidationError({'field1': "Please check field1"})
        }
        if (...something is wrong in "self.field2" ...) {
            raise ValidationError({'field2': "Please check field2"})
        }

        if (... something is globally wrong in the model ...) {
            raise ValidationError('Error message here')
        }

管理员已经从中受益,从 ModelAdmin.save_model() 调用 clean(), 并在更改视图中显示任何错误;当一个字段被 ValidationError 寻址时, 相应的小部件将在表单中突出显示。

要在以编程方式保存模型时运行相同的验证,只需按如下方式覆盖 save():

class MyModel(models.Model):

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

证明:

文件models.py

from django.db import models


class Model1(models.Model):

    def clean(self):
        print("Inside Model1.clean()")

    def save(self, *args, **kwargs):
        print('Enter Model1.save() ...')
        super().save(*args, **kwargs)
        print('Leave Model1.save() ...')
        return

class Model2(models.Model):

    def clean(self):
        print("Inside Model2.clean()")

    def save(self, *args, **kwargs):
        print('Enter Model2.save() ...')
        self.full_clean()
        super().save(*args, **kwargs)
        print('Leave Model2.save() ...')
        return

文件 test.py

from django.test import TestCase
from project.models import Model1
from project.models import Model2

class SillyTestCase(TestCase):

    def test_save_model1(self):
        model1 = Model1()
        model1.save()

    def test_save_model2(self):
        model2 = Model2()
        model2.save()

结果:

❯ python manage.py test
Creating test database for alias 'default'...
System check identified no issues (0 silenced).
Enter Model1.save() ...
Leave Model1.save() ...
.Enter Model2.save() ...
Inside Model2.clean()
Leave Model2.save() ...
.
----------------------------------------------------------------------
Ran 2 tests in 0.002s

OK
Destroying test database for alias 'default'...

【讨论】:

  • 您可以将self.instance.full_clean() 添加到def clean(self) 以运行模型验证。如果有问题,它会像任何其他问题一样提出ValidationError
  • 正如另一个答案所说...切换到 ModelForm 也很好,模型验证将自动进行以及设置字段。
  • @TimTisdall ... 既然您说“并将代码添加到 models.py 中”,这就是我所做的。而且我确实相信这是有充分理由的:如果您以编程方式保存模型,那将享受相同的验证。只要模型的代码保护自己,您的数据将始终处于良好状态,无论用户界面(即模型表单)表现如何好或坏
  • @TimTisdall "你可以将 self.instance.full_clean() 添加到 def clean(self)" ...我确实相信 Django 的工作方向相反:首先你在 Model 中调用 full_clean() .save(),然后(结果)框架将调用 clean()。我同意这里的方法名称相当混乱;)
  • @TimTisdall 老实说,我不明白这个概念是从哪里来的,ValidationErrors 是针对表单的。 DRF 还在序列化程序和模型中使用它们,这里是 official documentation 列表,特别是应该使用 ValidationError,以什么顺序调用事物以及为什么除了单独的字段清理方法之外还需要 clean()。
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2015-02-16
  • 2022-12-22
  • 1970-01-01
相关资源
最近更新 更多