Django中表单验证未通过仍重复插入数据库问题

你在DocumentoCreateView的post方法第一行调用了super(DocumentoCreateView, self).post(request)，这行代码会触发Django CreateView默认的表单处理流程——它会自动生成表单、执行验证，一旦验证通过（哪怕你自己后续的表单逻辑有问题）就直接把模型保存到数据库了。也就是说，你自己的表单验证逻辑是在默认保存操作之后才执行的，这就导致哪怕form.is_valid()返回false，前面默认流程已经把数据存进数据库了。

有两种靠谱的修改方式，选一种就行：

方式一：完全自定义post方法流程 #

删掉那行多余的super调用，自己处理表单的验证和保存逻辑，同时记得指定form_class为你的自定义表单（不要再用fields参数）：

class DocumentoCreateView(CreateView):
    model = Documento
    form_class = DocumentoCreateForm  # 指定自定义表单
    template_name = "documentos/documento_new.html"
    success_url = reverse_lazy("documento_new")

    def post(self, request, *args, **kwargs):
        form = self.get_form()
        if form.is_valid():
            documento = form.save()
            # 执行你的文本提取逻辑
            path = f'/media/' + form.files['arquivo_pdf'].__str__()
            extracao_de_texo = TextExtractor(path)
            return self.form_valid(form)
        else:
            print(form.errors)
            return self.form_invalid(form)

方式二：重写form_valid方法（更符合Django类视图的设计） #

不需要重写post方法，而是通过form_valid来扩展表单验证通过后的逻辑——这个方法只有在表单验证通过后才会被调用，自然不会出现提前保存的问题：

class DocumentoCreateView(CreateView):
    model = Documento
    form_class = DocumentoCreateForm
    template_name = "documentos/documento_new.html"
    success_url = reverse_lazy("documento_new")

    def form_valid(self, form):
        # 先保存模型（form_valid默认会调用form.save()，这里显式写出来更清晰）
        documento = form.save()
        # 执行文本提取操作
        path = f'/media/' + form.files['arquivo_pdf'].__str__()
        extracao_de_texo = TextExtractor(path)
        # 调用父类的form_valid完成后续跳转逻辑
        return super().form_valid(form)

1. 不要同时使用fields和form_class参数，选一个就行，用自定义表单时优先用form_class。
2. 你的validate_pdf验证器已经绑定在模型的arquivo_pdf字段上，表单会自动继承这个验证逻辑，不需要额外处理。

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