Django Admin 中 inline formset 出现多余空记录，通常源于 save_formset 逻辑错误及 ModelChoiceField 初始化不当；本文详解问题成因，并提供安全、可维护的替代实现。
django admin 中 inline formset 出现多余空记录，通常源于 `save_formset` 逻辑错误及 `modelchoicefield` 初始化不当；本文详解问题成因，并提供安全、可维护的替代实现。
在 Django Admin 中使用 TabularInline 管理关联模型时，若出现“新增一项却保存了两条记录，其中一条 offer_select 为空”的现象，根本原因在于两个层面：
表单字段初始化错误：OfferInlineForm.__init__() 中将 self.initial['offer_select'] = instance.id，但 offer_select 是 ModelChoiceField，其初始值应为 模型实例对象（如 Offer 实例），而非主键 ID。Django 表单在渲染时无法正确解析该 ID 为有效选项，导致字段回填失败，进而触发空值提交。
save_formset 逻辑混乱：原代码双重嵌套循环（for instance in instances + for data in formset.cleaned_data），且错误地将 offer_select 当作待更新目标（offer_select.feed = feed），而 offer_select 实际是另一个 Offer 对象——这不仅语义错乱，更会意外创建或修改无关记录。此外，未区分新增/编辑状态，也未正确处理 commit=False 后的关联赋值。
✅ 正确做法如下：
- 移除 __init__ 中对 offer_select 的错误初始化，让 Django 自动根据 instance.offer_select 填充；
- 在 save_formset 中，逐个匹配表单与实例（推荐 zip(formset.forms, instances)），仅对非空 offer_select 执行业务逻辑；
- 使用 form.cleaned_data.get(..., instance.field) 保持字段默认值，避免覆盖空提交；
- 调用 formset.save_m2m() 处理多对多字段（如有）；
- 关键原则：inline model 的实例本身（即 instance）才是当前要保存的对象，offer_select 是它的一个外键字段值，而非操作主体。
以下是修正后的完整代码：
class OfferInlineForm(forms.ModelForm):
offer_select = forms.ModelChoiceField(
queryset=Offer.objects.all(),
label='Offer select',
required=False # 显式声明可选，避免必填校验干扰
)
class Meta:
model = Offer
fields = ('offer_select', 'partner', 'logotype', 'number', 'visible')
@admin.register(Feed)
class FeedAdmin(admin.ModelAdmin):
class OfferItemInline(admin.TabularInline):
model = Offer
form = OfferInlineForm
extra = 0
# 可选：禁用删除/添加按钮以简化流程
# can_delete = False
list_display = ('name', 'utm_source', 'number')
ordering = ('name', 'number')
inlines = (OfferItemInline,)
fields = ('name',)
def save_formset(self, request, form, formset, change):
instances = formset.save(commit=False)
feed = form.instance
# 一一对应表单与实例，确保逻辑清晰
for form_obj, instance in zip(formset.forms, instances):
if form_obj.has_changed() and form_obj.is_valid():
cleaned = form_obj.cleaned_data
offer_select = cleaned.get('offer_select')
# 仅当 offer_select 有值时才更新关联关系
if offer_select:
instance.feed = feed # 关联到当前 Feed
instance.partner = cleaned.get('partner', instance.partner)
instance.logotype = cleaned.get('logotype', instance.logotype)
instance.number = cleaned.get('number', instance.number)
instance.visible = cleaned.get('visible', instance.visible)
instance.save()
else:
# 若 offer_select 为空，可选择跳过保存，或显式设为 None
instance.feed = None
instance.save()
# 保存 m2m 字段（如有）
formset.save_m2m()? 注意事项：
- extra = 0 是良好实践，避免 Admin 默认渲染空白行引发误提交；
- required=False 显式设置 offer_select 可选，配合前端校验更可控；
- 永远通过 form_obj.is_valid() 和 form_obj.has_changed() 判断是否需处理，防止空表单干扰；
- 不要直接操作 formset.cleaned_data 全局列表——它可能包含已删除/空表单，应始终绑定具体 form_obj；
- 若 Offer 模型中 feed 字段为 ForeignKey（指向 Feed），请确保其 on_delete 行为合理（如 models.CASCADE 或 models.SET_NULL）。
通过以上重构，可彻底消除冗余空记录，使 inline 表单行为符合预期：每行对应一个真实 Offer 实例，offer_select 正确回显与保存，逻辑清晰且易于维护。
