Django Admin内联表单集生成空项，求助问题原因与解决办法

你的问题核心出在save_formset逻辑错误和表单设计的逻辑混淆，具体问题点如下：

1. 双重循环导致重复操作 #

在save_formset中嵌套了两层循环：

for instance in instances:
    for data in formset.cleaned_data:
        # ... 处理逻辑

这会让每个instance被重复处理len(formset.cleaned_data)次。比如新增1条数据时，cleaned_data里会包含1条有效数据+1条空表单数据（Admin内联添加新行时，即便未填写内容，formset也会保留空条目），最终执行两次保存操作，生成重复空项。

2. 表单逻辑混淆 #

OfferInlineForm基于Offer模型，但额外添加了offer_select字段（用于选择另一个Offer实例），同时Meta字段包含了offer_select和Offer自身的字段。这相当于在编辑一个Offer实例时，同时选择并修改另一个Offer的字段，逻辑上混淆了「当前内联的Offer实例」和「被选中的Offer实例」。

3. 重复调用save生成冗余数据 #

你在循环中手动调用了offer_select.save()和instance.save()，最后又调用formset.save()，这会导致formset再次保存所有实例，包括已处理过的条目甚至空表单数据，进而生成额外空项。

4. 新增实例时initial未设置 #

在OfferInlineForm.__init__中，仅当instance存在（编辑已有条目）时才设置offer_select的初始值，新增条目时instance为None，导致offer_select初始为空，若用户未选择就提交，会产生空值条目。

1. 重构save_formset逻辑，移除双重循环 #

直接遍历formset的cleaned_data，只处理有效数据（排除空条目）：

def save_formset(self, request, form, formset, change):
    feed = form.instance
    # 遍历cleaned_data，跳过空表单
    for data in formset.cleaned_data:
        if not data:
            continue
            
        offer_select = data.get('offer_select')
        offer_current_id = data.get('id')
        partner = data.get('partner')
        logotype = data.get('logotype')
        visible = data.get('visible')
        number = data.get('number')

        if offer_select and offer_current_id != offer_select.id:
            # 解除当前Offer的Feed关联（如果存在）
            if offer_current_id:
                current_offer = Offer.objects.get(id=offer_current_id)
                current_offer.feed = None
                current_offer.save()
            # 更新选中Offer的关联与字段
            offer_select.feed = feed
            offer_select.partner = partner
            offer_select.logotype = logotype
            offer_select.number = number
            offer_select.visible = visible
            offer_select.save()
    # 无需再调用formset.save()，已手动处理有效数据

2. 调整表单逻辑（根据业务需求优化） #

如果你的业务是让Feed关联已有Offer，而非创建新Offer，可简化表单字段：

class OfferInlineForm(forms.ModelForm):
    offer_select = forms.ModelChoiceField(queryset=Offer.objects.all(), label='选择Offer')

    class Meta:
        model = Offer
        fields = ('offer_select',)  # 仅保留选择字段，其他字段通过选中的Offer更新

3. 过滤内联空表单 #

在OfferItemInline中添加表单集过滤逻辑，自动移除空表单：

class OfferItemInline(admin.TabularInline):
    model = Offer
    form = OfferInlineForm
    extra = 0

    class formset(forms.BaseInlineFormSet):
        def clean(self):
            super().clean()
            # 只保留有内容的表单数据
            self.cleaned_data = [item for item in self.cleaned_data if any(item.values())]

关键注意事项 #

• 避免在save_formset中嵌套循环，确保每个表单数据只处理一次
• 手动处理数据后不要重复调用formset.save()，防止冗余保存
• 明确业务逻辑：内联表单是创建新Offer，还是关联/修改已有Offer？当前代码逻辑是修改选中的Offer，若你预期是创建新Offer，需重新设计模型关联

内容的提问来源于stack exchange，提问作者john-doe