请帮忙检查我未使用Django Forms编写的注册函数是否正确

核心问题清单 #

• 数据验证逻辑顺序颠倒：先保存Customers实例到数据库，再做密码匹配、用户名/邮箱唯一性检查，会导致无效数据（如密码不匹配、用户名重复）被存入数据库，后续需手动清理。
• 密码处理错误：make_password加密后的密码传给User.objects.create_user会导致二次加密，登录时密码验证失败。create_user会自动处理密码加密，只需传入明文密码。
• 重定向参数错误：redirect(register)应改为redirect("register")，否则会传入视图函数对象而非路由名称，引发异常。
• POST数据读取不安全：直接用request.POST['key']读取字段，若前端未传该字段会抛出KeyError，导致服务崩溃。
• 重复创建用户实例：若Customers是自定义用户模型，无需再创建默认User实例；若为关联模型，应先创建User，再关联Customers，而非反过来。

修正后的代码示例 #

from django.contrib.auth.hashers import check_password, make_password
from django.contrib.auth.models import User
from django.shortcuts import render, redirect
from django.contrib import messages
from .models import Customers  # 假设Customers在当前app的models里

def register(request):
    if request.method == 'POST':
        # 安全读取POST数据，设默认值避免KeyError
        fname = request.POST.get('fname', '')
        lname = request.POST.get('lname', '')
        email = request.POST.get('email', '')
        password = request.POST.get('password', '')
        repassword = request.POST.get('repassword', '')
        username = request.POST.get('username', '')

        # 检查必填字段是否为空
        if not all([fname, lname, email, password, repassword, username]):
            messages.info(request, '所有字段都不能为空')
            return redirect("register")
        
        # 检查密码是否匹配
        if password != repassword:
            messages.info(request, '两次输入的密码不一致')
            return redirect("register")
        
        # 检查用户名和邮箱是否已存在
        if User.objects.filter(username=username).exists():
            messages.info(request, '用户名已被占用')
            return redirect("register")
        if User.objects.filter(email=email).exists():
            messages.info(request, '邮箱已被占用')
            return redirect("register")
        
        # 验证通过后创建实例
        # 先创建User实例（create_user自动加密密码）
        user = User.objects.create_user(
            username=username,
            password=password,  # 传入明文密码
            email=email,
            first_name=fname,
            last_name=lname
        )
        
        # 若Customers是关联User的模型，关联已创建的用户
        db = Customers()
        db.fname = fname
        db.lname = lname
        db.email = email
        db.username = username
        # 无需存储密码，User模型已保存加密后的密码
        db.save()
        
        messages.success(request, '注册成功，请登录')
        return redirect("login_user")
    else:
        return render(request, "compte/register.html")

额外建议 #

• 优先学习Django Forms/ModelForms，它能自动处理字段验证、数据类型转换、错误信息展示，比手动写验证更高效且不易出错。
• 若使用自定义用户模型，需在settings.py中配置AUTH_USER_MODEL = 'your_app.Customers'，避免同时使用默认User模型造成数据冗余。

内容的提问来源于stack exchange，提问作者badr ahrir