Django表单实现用户创建可加成员的俱乐部（用户组）功能异常排查

现存导致数据库无记录的关键错误 #

• 表单类继承错误：Django 表单体系中不存在 forms.Model 基类，你编写的 AddClubForm(forms.Model) 属于无效类定义，会直接导致表单校验、模型映射、save方法完全失效，根本无法执行数据库写入操作。
• 视图传参错位：ModelForm 的 instance 参数仅用于绑定已有模型实例做更新操作，需要传入对应模型类的对象，你当前传入的是 request.user（Account 类型对象），和绑定的 Club 模型类型完全不匹配，会导致表单初始化异常。
• 逻辑冗余：owner 字段本质是当前登录的俱乐部创建者，完全不需要让用户手动输入文本，暴露为可编辑表单字段既可能引入伪造创建者的安全问题，也增加了无意义的校验流程。
• 字段类型不匹配：模型中 start/end 定义为 DateTimeField，表单对应字段却写为 DateField，类型不一致会直接导致表单校验失败，无法通过校验流程。
• 错误处理逻辑缺失：当前POST分支中，无论 form.is_valid() 返回校验通过还是失败，都会直接执行重定向跳转，既不会把校验失败的错误反馈给用户，也不会执行 save 操作，自然数据库不会生成记录。
• 模板渲染不完整：当前模板仅渲染了 form.owner 字段，topic、start、end 字段根本没有被渲染到表单中，就算前期bug全部修复，用户也无法提交这些字段的合法值。
• 模型设计缺陷：原代码中 owner 字段用 CharField 存文本用户名，应该改用外键关联 Account 模型，否则用户修改用户名后，俱乐部的归属信息会和实际账号脱节。

对应优化修复方案 #

1. 修正模型设计 #

class Club(models.Model):
    # 关联当前创建者账号，替代原来的CharField存文本
    owner = models.ForeignKey(Account, on_delete=models.CASCADE, related_name="owned_clubs")
    topic = models.CharField(max_length=30, blank=True)
    start = models.DateTimeField(verbose_name='start date')
    end = models.DateTimeField(verbose_name='end date')
    # 明确多对多关联的反向查询名，预留添加成员的字段
    members = models.ManyToManyField(Account, related_name="joined_clubs", blank=True)

2. 重写表单类 #

from django import forms
from .models import Club

# 修正继承基类，使用ModelForm实现模型关联映射
class AddClubForm(forms.ModelForm):
    # 将DateField替换为DateTimeField，和模型字段类型对齐
    start = forms.DateTimeField(required=False)
    end = forms.DateTimeField(required=False)
    topic = forms.CharField(required=False, max_length=30)

    class Meta:
        model = Club
        # 移除owner字段，该字段在视图中手动赋值，不开放给用户提交
        fields = (
            'topic',
            'start',
            'end',
        )

3. 修正视图逻辑 #

from django.shortcuts import redirect, render
from django.contrib.auth.decorators import login_required
from .forms import AddClubForm

@login_required
def add_club(request):
    if request.method == "POST":
        # 初始化表单仅传入POST请求参数，移除错位的instance传值
        form = AddClubForm(request.POST)
        if form.is_valid():
            # 先获取表单保存的实例，暂不提交数据库
            club = form.save(commit=False)
            # 自动绑定当前登录用户为俱乐部创建者
            club.owner = request.user
            # 完成字段赋值后再写入数据库
            club.save()
            # 保存成功后再执行重定向
            return redirect(request.path_info)
        # 校验失败时将携带错误信息的表单返回给模板渲染，不再直接重定向
        return render(request, 'page/club.html', {"form": form})
    # GET请求时初始化空表单即可
    form = AddClubForm()
    return render(request, 'page/club.html', {"form": form})

4. 补全模板渲染 #

<form method="POST" action="">
    {% csrf_token %}
    <div class="col-md-6">
        <label class="labels">Create a club</label>
        <!-- 补全所有表单字段渲染，不再仅渲染owner字段 -->
        {{ form.topic.label }}
        {{ form.topic }}
        <br>
        {{ form.start.label }}
        {{ form.start }}
        <br>
        {{ form.end.label }}
        {{ form.end }}
        <br>
        <input class="btn" type="submit" value="Add club">
    </div>
    <!-- 渲染表单错误提示，方便用户定位填写问题 -->
    {% if form.errors %}
        <div class="alert alert-danger">
            {% for error in form.errors.values %}
                <p>{{ error }}</p>
            {% endfor %}
        </div>
    {% endif %}
</form>

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