Django注册页面测试报错：视图未返回HttpResponse对象

咱们来拆解你遇到的两个核心问题：视图返回None导致的ValueError，以及手动查询用户名的print语句没执行。

为什么print语句没触发？ #

你写的user_taken = User.objects.filter(username = username).first()这段代码根本没机会运行！如果你的SignupForm是继承自ModelForm（对应Django内置的User模型），Django会自动验证用户名的唯一性——当你输入已存在的用户名时，form.is_valid()直接返回False，代码根本不会进入if form.is_valid()的分支，自然不会执行你的print语句。

为什么会返回None？ #

当request.method == "POST"但form.is_valid()失败时，你的代码没有任何返回逻辑！函数走到这里就结束了，返回了None，这就是Django抛出ValueError的直接原因。

修复后的代码 #

咱们修正这几个问题，同时优化逻辑：

from django.contrib.auth.models import User
from django.contrib.auth.hashers import make_password
from django.contrib.auth import login, redirect
from django.shortcuts import render

def user_signup(request):
    if request.method == "POST":
        form = SignupForm(request.POST)
        if form.is_valid():
            cd = form.cleaned_data
            email = cd['email']
            username = cd['username']
            password = cd['password']
            
            # 这里不需要手动查用户名了，form.is_valid()已经验证过唯一性
            print('username is good')
            secure_password = make_password(password)
            user = User.objects.create(
                username=username,
                email=email,
                password=secure_password
            )
            login(request, user)
            return redirect('user_profile')
        else:
            # 表单验证失败，直接返回带错误信息的表单
            return render(request, 'users/signup.html', {'form': form})
    else:
        # 注意这里之前少了括号，必须实例化表单类
        form = SignupForm()
        return render(request, 'users/signup.html', {'form': form})

额外优化建议 #

1. 利用表单自带的错误提示：不用手动传递error变量，Django的表单会自动收集所有验证错误（包括用户名已存在的错误），在模板里可以这样显示：

   {% if form.errors %}
       <div class="alert alert-danger">
           {% for field in form %}
               {% for error in field.errors %}
                   <p>{{ error }}</p>
               {% endfor %}
           {% endfor %}
           {% for error in form.non_field_errors %}
               <p>{{ error }}</p>
           {% endfor %}
       </div>
   {% endif %}
   

2. 推荐使用内置表单：Django自带UserCreationForm，已经帮你处理了用户名唯一性验证、密码合法性检查等逻辑，直接继承它可以大幅简化你的代码，不用自己实现密码加密和用户名检查。

内容的提问来源于stack exchange，提问作者Omar Jandali