Django同一视图内两个表单仅顶部表单提交生效如何修复

你遇到的问题由3个明确错误导致：

• 视图逻辑错误：所有POST请求会同时绑定到两个表单做校验，只要排在前面的表单校验不通过，代码就直接执行return跳转/返回响应，后面的表单校验逻辑根本不会执行，所以永远只有先校验的表单能正常提交。比如提交评论时出价表单是空值，校验必然失败，代码直接重定向，根本走不到评论保存的逻辑。之前你尝试通过判断POST参数区分表单来源没生效，是因为没在模板表单里加对应name的元素，后端检测不到对应的键，判断永远不成立。
• 表单定义错误：BidsForm的widgets配置写错了字段名，表单字段是bid，widgets里对应的key写成了title，导致出价输入框不会被正确渲染，表单校验逻辑失效。
• 模板层错误：两个表单没有用独立的<form>标签包裹，也没有加区分提交来源的标识，后端没法判断当前请求是哪个表单发起的。

1. 修正forms.py的配置错误 #

把BidsForm里widgets的错误字段名从title改成bid：

class CommentForm(forms.ModelForm):
    class Meta:
        model = Comments
        fields = ['comment']
        widgets = {
            'comment': forms.TextInput(attrs={
                'placeholder': 'Add a comment about this listing here.',
                'class' : 'form-control'
            })
        }

class BidsForm(forms.ModelForm):
    class Meta:
        model = Bids
        fields = ['bid']
        widgets = {
            # 修正之前写错的字段名
            'bid': forms.NumberInput(attrs={
                'placeholder': 'Add a bid for this listing here.',
                'class' : 'form-control'
            })
        }

2. 调整模板结构，给两个表单加独立标识 #

两个表单必须分别用独立的<form>标签包裹，不能嵌套、不能把两个表单的输入框塞到同一个form标签里，给每个表单的提交按钮加不同的name属性，用来给后端判断提交来源：

<!-- 出价表单 -->
<form method="POST">
    {% csrf_token %}
    {{ bidForm }}
    <button type="submit" name="submit_bid" class="btn btn-primary">提交出价</button>
</form>

<!-- 评论表单 -->
<form method="POST">
    {% csrf_token %}
    {{ form }}
    <button type="submit" name="submit_comment" class="btn btn-primary">提交评论</button>
</form>

注意：不要给两个表单加重复的name或id属性，避免参数冲突。

3. 重写视图层POST逻辑 #

先判断提交来源，只实例化、校验当前提交的对应表单，不要同时给两个表单绑定POST数据：

def listing(request, id):
    listing = get_object_or_404(Listings.objects, pk=id)
    listing_price = listing.bid
    comment_obj = Comments.objects.filter(listing=listing)
    # GET请求默认初始化空表单
    comment_form = CommentForm()
    bid_form = BidsForm()

    bid_obj = Bids.objects.filter(listing=listing)
    max_bid = 0
    for bid in bid_obj.all():
        if bid.bid > max_bid:
            max_bid = bid.bid

    if request.method == "POST":
        # 判断是出价表单提交
        if "submit_bid" in request.POST:
            bid_form = BidsForm(request.POST)
            if bid_form.is_valid():
                new_bid = float(bid_form.cleaned_data.get("bid"))
                # 修正原逻辑的or错误，出价必须同时满足两个条件
                if new_bid >= listing_price and new_bid > max_bid:
                    bid = bid_form.save(commit=False)
                    bid.listing = listing
                    bid.user = request.user
                    bid.save()
                    return redirect('listing', id=id)
                else:
                    message = "出价需大于等于挂牌价，且高于当前所有已提交的出价"
                    return render(request, "auctions/listing.html",{
                        "auction_listing": listing,
                        "form": CommentForm(),
                        "comments": comment_obj,
                        "bidForm": bid_form,
                        "bids": bid_obj,
                        "message": message
                    })
            else:
                return render(request, "auctions/listing.html",{
                    "auction_listing": listing,
                    "form": CommentForm(),
                    "comments": comment_obj,
                    "bidForm": bid_form,
                    "bids": bid_obj
                })
        
        # 判断是评论表单提交
        elif "submit_comment" in request.POST:
            comment_form = CommentForm(request.POST)
            if comment_form.is_valid():
                comment = comment_form.save(commit=False)
                comment.listing = listing
                comment.user = request.user
                comment.save()
                return redirect('listing', id=id)
            else:
                return render(request, "auctions/listing.html",{
                    "auction_listing": listing,
                    "form": comment_form,
                    "comments": comment_obj,
                    "bidForm": BidsForm(),
                    "bids": bid_obj
                })

    # GET请求统一返回页面
    return render(request, "auctions/listing.html",{
        "auction_listing": listing,
        "form": comment_form,
        "comments": comment_obj,
        "bidForm": bid_form,
        "bids": bid_obj
    })

• 原逻辑里出价判断用了or连接条件，会导致只要满足「大于等于挂牌价」或者「大于当前最高出价」其中一个就能提交，正确逻辑应该用and，两个条件必须同时满足。
• 表单提交成功后必须走redirect重定向，避免用户刷新页面重复提交。
• 单个表单处理完成后不要直接全局return，保证不同表单的处理逻辑完全隔离，不会互相干扰。

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