Django：如何通过循环正确向数据库添加对战记录？

核心问题排查 #

你的代码无法保存对战数据，主要有以下几个关键错误：

1. 循环提前终止
   你在for p in participants_pairs循环内部直接调用了return HttpResponseRedirect(...)，这会导致循环只执行第一次迭代就跳转页面，后续的对战记录根本不会被创建。

2. 表单类使用混淆
   POST请求时用的是AddGroupForm，GET请求时却引用AddFightsForm（还没实例化，少了括号），表单类不统一会导致验证失败或字段无法正确获取。

3. 重复创建对象且更新错误

   • 你通过form.save(commit=False)获取的obj是同一个实例，循环中修改它再save()只会不断更新这个对象，而不是创建新的对战记录。
   • 同时又调用group.fights.create(...)，属于重复创建逻辑，容易造成数据混乱。

4. GET请求处理逻辑位置错误
   当前逻辑中，若用户已认证但不是POST请求，会直接返回表单；但如果用户未认证，没有任何返回逻辑，会导致报错。

修正后的视图函数 #

import itertools
from django.shortcuts import get_object_or_404, render, redirect
from django.urls import reverse
from .models import Group, Fight  # 确保导入对应的模型
from .forms import AddFightsForm  # 统一使用AddFightsForm

def add_fights(request, group_id):
    group = get_object_or_404(Group, pk=group_id)
    tournament = group.tournament
    participants = group.participants.all()
    only_ids_ls = [p.id for p in participants]  # 简化ID提取方式
    participants_pairs = list(itertools.combinations(only_ids_ls, 2))
    
    if request.user.is_authenticated:
        if request.method == "POST":
            form = AddFightsForm(request.POST)
            if form.is_valid():
                rounds = form.cleaned_data['rounds']
                # 循环创建所有对战记录
                for p1_id, p2_id in participants_pairs:
                    fighter_one = participants.get(id=p1_id)
                    fighter_two = participants.get(id=p2_id)
                    # 直接创建Fight对象，避免重复逻辑
                    Fight.objects.create(
                        group=group,
                        rounds=rounds,
                        tournament=tournament,
                        fighter_one=fighter_one,
                        fighter_two=fighter_two
                    )
                # 所有记录创建完成后再跳转
                return redirect(reverse("tournaments:tournament_details", args=[group_id]))
        else:
            # GET请求时实例化表单
            form = AddFightsForm()
        return render(request, "add_fights.html", {
            'form': form,
            'group_id': group_id,
        })
    # 未认证用户的处理，比如跳转登录页
    return redirect('login')

额外注意事项 #

1. 补充缺失的模型
   你提供的模型代码中缺少Group和Fight模型，确保Fight模型包含以下关键字段：

   class Group(models.Model):
       tournament = models.ForeignKey(Tournament, on_delete=models.CASCADE, related_name='groups')
       participants = models.ManyToManyField('Participant', related_name='groups')  # 假设存在Participant模型
       # 其他字段...
   
   class Fight(models.Model):
       group = models.ForeignKey(Group, on_delete=models.CASCADE, related_name='fights')
       tournament = models.ForeignKey(Tournament, on_delete=models.CASCADE)
       rounds = models.IntegerField()
       fighter_one = models.ForeignKey('Participant', on_delete=models.CASCADE, related_name='fights_as_one')
       fighter_two = models.ForeignKey('Participant', on_delete=models.CASCADE, related_name='fights_as_two')
       # 其他字段，比如对战结果等...
   

2. 表单类验证
   确保AddFightsForm包含rounds字段且验证逻辑正确，示例：

   from django import forms
   from .models import Fight
   
   class AddFightsForm(forms.ModelForm):
       class Meta:
           model = Fight
           fields = ['rounds']
   

3. 优化性能
   若分组内选手较多，循环中多次调用participants.get()会产生重复数据库查询，可以提前将选手ID映射为对象：

   participant_map = {p.id: p for p in participants}
   # 循环中直接使用
   fighter_one = participant_map[p1_id]
   fighter_two = participant_map[p2_id]
   

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