Django中如何将注册用户关联并保存为Kunde模型实例

咱们先梳理下你现有代码里的几个核心问题：

1. User创建逻辑错误且位置不对：你把User.objects.create(...)写在了POST请求判断之外，而且写法不符合Django规范，还没正确处理密码加密（UserCreationForm本来就帮你做了这个事）。
2. 未正确关联User与Kunde：你注释掉了bruker = bruker这行，而且创建User的时机不对，导致无法拿到有效的User实例关联到Kunde。
3. 表单处理逻辑顺序混乱：表单初始化和User创建的顺序不符合Django的请求处理流程，容易引发不必要的错误。

下面是修复后的完整解决方案：

修复后的视图代码 #

from django.shortcuts import render, redirect
from .forms import lageBruker
from .models import Kunde

def registrer(request):
    # POST请求处理表单提交
    if request.method == 'POST':
        form = lageBruker(request.POST)
        if form.is_valid():
            # 用表单的save()方法创建User，自动处理密码哈希等逻辑
            bruker = form.save()
            # 创建Kunde并关联刚刚生成的User实例
            Kunde.objects.create(
                bruker=bruker,
                fornavn=bruker.first_name,
                etternavn=bruker.last_name,
                email=bruker.email
            )
            # 注册成功后可跳转至登录页或其他页面
            return redirect('login')
    # GET请求展示空表单
    else:
        form = lageBruker()
    
    context = {'form': form}
    return render(request, 'ebutikk/registrer.html', context)

关键调整说明 #

• 规范表单处理流程：把表单初始化放在了请求分支里，GET请求返回空表单，POST请求处理提交的数据，避免无效的表单初始化。
• 使用表单save()创建User：lageBruker继承自UserCreationForm，它的save()方法会自动完成用户创建、密码哈希等安全操作，比手动从request.POST取值创建User可靠得多。
• 正确关联User与Kunde：用form.save()返回的User实例直接关联Kunde的bruker字段，建立一对一关联关系。
• 复用User字段数据：直接从User实例中取first_name、last_name、email字段填充Kunde，避免重复从request.POST取值，减少冗余和错误。

进阶优化：用信号自动关联User与Kunde #

如果你想让代码更简洁，可以使用Django的信号机制，当User创建时自动生成对应的Kunde，这样视图里就不用写Kunde创建逻辑了：

在你的models.py中添加以下代码：

from django.db.models.signals import post_save
from django.contrib.auth.models import User
from django.dispatch import receiver

# 当User创建时自动创建Kunde
@receiver(post_save, sender=User)
def create_kunde(sender, instance, created, **kwargs):
    if created:
        Kunde.objects.create(
            bruker=instance,
            fornavn=instance.first_name,
            etternavn=instance.last_name,
            email=instance.email
        )

# 当User更新时同步保存Kunde
@receiver(post_save, sender=User)
def save_kunde(sender, instance, **kwargs):
    instance.kunde.save()

此时你的视图可以简化为：

from django.shortcuts import render, redirect
from .forms import lageBruker

def registrer(request):
    if request.method == 'POST':
        form = lageBruker(request.POST)
        if form.is_valid():
            form.save()
            return redirect('login')
    else:
        form = lageBruker()
    
    context = {'form': form}
    return render(request, 'ebutikk/registrer.html', context)

额外建议 #

你的Kunde模型里的fornavn、etternavn、email字段其实和User模型的字段重复了，长期来看可以考虑去掉这些重复字段，直接通过kunde.bruker.first_name这种方式访问用户信息，减少数据冗余。

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