Django表单验证失败未重渲染，返回None引发ValueError问题

问题原因 #

你写的视图函数存在逻辑漏洞：当请求是POST但表单验证失败时，代码没有返回任何HttpResponse对象，执行完判断后直接走到函数末尾返回了None，这就是报错的根源。你误以为表单无效会进入处理GET请求的else分支，但那个else只针对request.method != 'POST'的情况，POST请求下表单无效不会触发它。

解决方案 #

调整视图逻辑，确保所有代码路径都返回响应。有两种常见写法：

写法一：统一渲染逻辑

把表单渲染代码移到条件判断之外，不管是GET请求还是POST验证失败，都用同一个逻辑返回带表单的页面：

from django.shortcuts import render,redirect
from django.urls import reverse
from .forms import ReviewForm

def rental_review(request):
    form = ReviewForm()  # 初始化空表单
    if request.method=='POST':
        form = ReviewForm(request.POST)
        if form.is_valid():
            form.save()
            return redirect(reverse('cars_form:thank_you'))
    # GET请求或POST验证失败时，都会走到这里渲染表单
    return render(request,'cars_form/rental_review.html',context={'form':form})

写法二：补充POST分支的返回

保持原有结构，在POST请求的表单无效分支里添加返回逻辑：

from django.shortcuts import render,redirect
from django.urls import reverse
from .forms import ReviewForm

def rental_review(request):
    if request.method=='POST':
        form = ReviewForm(request.POST)
        if form.is_valid():
            form.save()
            return redirect(reverse('cars_form:thank_you'))
        # 表单无效时，带着错误信息重新渲染页面
        return render(request,'cars_form/rental_review.html',context={'form':form})
    else:
        form = ReviewForm()
        return render(request,'cars_form/rental_review.html',context={'form':form})

额外说明 #

你给模型字段加的MinValueValidator和MaxValueValidator会被ModelForm自动继承，表单会自动对stars字段做范围验证。要让用户看到错误提示，需要在模板里添加错误展示代码，比如：

<form method="post">
    {% csrf_token %}
    {{ form.as_p }}
    {% if form.errors %}
        <div style="color: red;">
            {% for field in form %}
                {% for error in field.errors %}
                    <p>{{ field.label }}: {{ error }}</p>
                {% endfor %}
            {% endfor %}
        </div>
    {% endif %}
    <button type="submit">提交</button>
</form>

这样当输入的stars超出1-5范围时，表单会重新渲染并显示错误提示，同时返回200状态码。

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