Django如何创建带外键关联表单及解决视图返回None报错问题

核心报错根因 #

1. 视图无返回的直接原因 #

你在views.py的add_listing函数中，仅处理了GET请求、POST请求且表单校验通过两种场景的返回，当POST请求表单校验失败时走了else: pass分支，没有返回任何HttpResponse对象，这就是你看到的*The view auctions.views.add_listing didn't return an HttpResponse object. It returned None instead.*报错的直接诱因。

2. 表单校验失败的根本原因（和category字段相关） #

• models.py中Listing的category是ManyToManyField多对多字段，不能通过直接给cleaned_data赋值单个id的方式保存，多对多关系需要在主表对象存入数据库后单独处理
• forms.py中将category定义为CharField，重写的save方法完全不符合多对多字段的保存规则
• new.html中select下拉框的option配置冗余，且value传递的是分类对象的文本值，容易出现匹配错误
• 表单保存时调用save(commit=False)后没有手动执行save()方法，生成的Listing对象没有主键，跳转时传参也会报错

修复步骤 #

步骤1：修复views.py的返回逻辑和保存逻辑 #

def add_listing(request):
    if request.method == "POST":
        form = ListingForm(request.POST, request.FILES)
        print('FORM CREATED')
        print(form.is_valid())
        if form.is_valid():
            print('FORM IS VALID')
            print(form.cleaned_data)
            new_listing = form.save(commit=False)
            # 手动保存主表对象，生成主键
            new_listing.save()
            # 保存多对多字段关联
            form.save_m2m()
            # reverse的args参数要用元组，不要用集合
            return HttpResponseRedirect(reverse(listing, args=(new_listing.pk,)))
        else:
            # 校验失败返回页面，带回表单和分类数据用于回显
            return render(request, "auctions/new.html",{
                'form': form,
                'category':Category.objects.all()
            })
    else:
        return render(request, "auctions/new.html",{
            'category':Category.objects.all()
        })

步骤2：修复forms.py的字段定义和保存逻辑 #

class ListingForm(forms.ModelForm):
    # 改成模型选择字段，适配下拉选分类的场景，required=False适配字段blank=True的配置
    category = forms.ModelChoiceField(queryset=Category.objects.all(), required=False)

    class Meta:
        model = Listing
        fields = [
            "title",
            "description",
            "price",
            "photo",
            "category"
        ]

    # 如果需要支持自动创建不存在的分类，保留重写的save方法，调整逻辑如下
    def save(self, commit=True):
        listing = super().save(commit=False)
        if commit:
            listing.save()
        category = self.cleaned_data.get('category')
        if category:
            # 多对多字段调用add方法绑定关联
            listing.category.add(category)
        return listing

    def __str__(self):
        return self.title

步骤3：修复new.html的下拉框配置 #

<select name="category" id="">
    {% for type in category %}
    <!-- value传分类id，删除option上冗余的name属性 -->
    <option value="{{ type.id }}">{{ type.name }}</option>
    {% endfor %}
</select>

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