Django性能优化：如何高效批量为Provider添加关联Skill对象？

原代码存在几个直接导致执行缓慢的问题：

• 冗余数据库查询：嵌套循环中每次Skill.objects.get()都会发起单独的数据库请求，当目标车辆和技能数量较多时，查询次数会呈N×M级增长，这是性能瓶颈的核心原因。
• 不必要的save操作：provider.skills.add()方法会自动处理多对多关系的保存，手动调用provider.save()完全多余，还会增加额外的数据库写入开销。
• 逻辑缩进错误：遍历车辆和技能的代码块没有放在form.is_valid()的判断范围内，即使表单验证失败也会执行，存在逻辑bug。

优化方案 #

核心思路是将多次数据库交互合并为少数几次，通过批量查询和批量添加来提升性能：

def skills_add(request, id):
    provider = get_object_or_404(Provider, id=id)
    if request.method == 'POST':
        form = AddSkill(request.POST)
        if form.is_valid():
            data = form.cleaned_data
            # 1. 批量获取目标品牌下所有车辆的ID（仅取ID，减少数据传输）
            car_ids = Car.objects.filter(brand=data['brand']).values_list('id', flat=True)
            # 2. 一次性查询出所有符合条件的Skill对象
            target_skills = Skill.objects.filter(
                car_id__in=car_ids,
                skill__in=data['skills']
            )
            # 3. 批量添加多对多关联，无需循环逐个处理
            provider.skills.add(*target_skills)
    # 统一重定向逻辑，添加默认值避免空引用
    return redirect(request.META.get('HTTP_REFERER', "managers:dashboard_manager"))

优化细节说明 #

• 批量获取车辆ID：使用values_list('id', flat=True)直接提取车辆ID列表，避免加载完整的Car对象，减少内存占用和数据库返回的数据量。
• 一次性筛选Skill：通过car_id__in和skill__in的组合条件，将原本N×M次的查询缩减为1次，大幅降低数据库交互次数。
• 批量添加关联：provider.skills.add(*target_skills)支持一次性传入多个对象，Django内部会批量处理多对多关系的写入操作，进一步减少数据库请求。
• 修复逻辑bug：将核心业务逻辑完全包裹在form.is_valid()判断内，避免表单验证失败时执行无效代码。
• 健壮的重定向：给redirect设置默认值"managers:dashboard_manager"，防止HTTP_REFERER为空时出现异常。

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