Django学生注册时报Field 'id' expected a number but got (<Group: STUDENT>, False)错误

错误原因 #

• Django的get_or_create()方法返回值是二元组(model_instance, created)，第一个元素是查询/创建的模型实例，第二个是布尔值表示是否为新创建的对象
• 原代码直接把返回的完整元组赋值给student_group，后续调用add()方法时传入的是元组而非Group实例，因此触发类型错误。

修复步骤 #

1. 修正Group获取逻辑 #

拆分get_or_create()的返回值，仅提取Group实例对象即可：

# 原错误代码
# student_group = Group.objects.get_or_create(name='STUDENT')
# 修正后代码，_ 为占位符，不需要用到是否创建的布尔值
student_group, _ = Group.objects.get_or_create(name='STUDENT')

2. 完整修正后的视图代码 #

同时删除原代码中冗余的Form实例化逻辑，也符合仅通过Student实例处理业务的需求：

from django.contrib.auth.models import Group
from django.contrib import messages
from django.shortcuts import render, redirect
from student.models import Student
from quiz.forms import RegistrationForm

def register_student_view(request):
    if request.method == 'POST':
        form = RegistrationForm(request.POST, request.FILES)
        if form.is_valid():
            user = form.save(commit=False)
            user.is_student = True
            user.save()
            # 实例化Student关联用户，后续所有操作通过student实例处理
            student = Student.objects.create(user=user)
            # 拆分get_or_create返回值，仅取Group实例
            student_group, _ = Group.objects.get_or_create(name='STUDENT')
            student.user.groups.add(student_group)
            messages.success(request, 'Registration successful!')
            return redirect('login')
    # GET请求仅初始化一次form
    else:
        form = RegistrationForm()
    return render(request, 'accounts/register.html', {'form': form })

额外优化建议 #

如果要严格满足不直接操作CustomUser，仅通过Student实例处理业务逻辑的需求，可以把用户字段赋值、保存逻辑都封装到Student模型的自定义方法中，避免在视图层直接操作CustomUser实例。

内容的提问来源于stack exchange，提问作者sam hassan