Django Formset无法保存：移除特定代码才可正常保存的问题排查

核心结论 #

Fields.objects.filter()本身不会直接干扰保存逻辑，但你基于该查询结果的后续操作（尤其是旧数据字典的更新逻辑）会导致事务回滚，最终表现为没有任何记录被保存。

具体问题点 #

1. 事务内的旧数据快照问题
   你在函数开头就查询了Fields的旧数据并存在fields_dict里，这是事务开始时的数据库快照。当你保存新的Fields实例后，后续用fields_dict里的旧实例去更新排序，若此时出现数据冲突（比如旧实例的字段已经被新操作修改），会触发事务回滚，把之前保存的所有内容都撤销。

2. 排序更新逻辑的潜在异常
   如果ordered_ids中包含了不在fields_dict里的ID（比如新创建字段的临时ID，或者前端传递的错误ID），会抛出KeyError异常。因为函数加了@transaction.atomic，整个事务会直接回滚，导致所有保存操作失效。而你的视图虽然捕获了异常，但可能没输出具体错误，导致你误以为是filter的问题。

3. 新字段排序的逻辑漏洞
   新字段的order被设为旧数据的长度+1，但后续的排序循环跳过了id == 'None'的项，这些新字段的排序不会被重新调整。如果order字段有唯一约束，可能导致重复值，进而触发保存失败。

修复方案 #

1. 调整查询时机，获取最新数据
   把查询Fields的代码移到formset保存之后，确保字典里包含新创建的字段：

@transaction.atomic
def update_fields_and_fields_group(formset, fg_id, fg_name, ordered_ids):
    fg = FieldsGroup.objects.get(pk=fg_id)
    fg.name = fg_name
    fg.save()

    # 先保存formset
    instances = formset.save(commit=False)
    for fields in instances:
        fields.fields_group_id = fg_id
        fields.save()
    
    # 保存后再查询最新的Fields数据
    fields_list = Fields.objects.filter(fields_group__pk=fg_id)
    fields_dict = {f.id: f for f in fields_list}
    
    # 处理排序，包含新字段
    current_order = 1
    for id_str in ordered_ids:
        if id_str == 'None':
            # 找到对应的新实例（从formset的new_objects中获取）
            for new_field in formset.new_objects:
                new_field.order = current_order
                new_field.save()
                current_order += 1
            continue
        field_id = int(id_str)
        if field_id in fields_dict:
            fields_dict[field_id].order = current_order
            fields_dict[field_id].save()
            current_order += 1

2. 完善异常排查
   在视图的异常捕获中添加错误打印，方便定位问题：

if formset.is_valid():
    try:
        ordered_ids = request.POST.get("fields_orders")
        ordered_ids = ordered_ids.split(", ")
        update_fields_and_fields_group(formset, fields_group_id, fields_group_name, ordered_ids)
        messages.success(request, "success")
    except Exception as e:
        print(f"保存失败原因: {str(e)}")  # 打印具体错误
        messages.error(request, str(e))

3. 确保排序字段无冲突
   如果order字段设置了唯一约束，要保证所有字段的order值唯一，避免保存时抛出唯一性异常。

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