Django批量创建对象失败：NOT NULL约束报错排查

问题描述 #

使用ModelForm实现批量将数字存入Nbr模型时，控制台能打印日志但数据未保存至数据库，执行super().save(self)触发IntegrityError，提示NOT NULL constraint failed: app_nbr.number。尝试两种方案均失败：

• 创建Nbr实例列表调用bulk_create，返回列表时提示'list' object has no attribute 'save'
• 返回最后一个实例，提示'NbrSetForm' object has no attribute 'save_m2m'

问题根源 #

1. 原save方法中，循环创建实例后调用super().save(self)属于错误用法：父类save会尝试创建一个缺失number字段的Nbr实例，违反非空约束；且调用方式错误，正确应为super().save(commit=commit)而非传入self。
2. ModelForm的save方法约定返回单个模型实例，bulk_create返回列表不符合预期，导致后续save_m2m调用失败。

修复后的代码 #

调整后的ModelForm实现 #

class NbrSetForm(forms.ModelForm):
    start_number = forms.IntegerField(required=True)
    end_number = forms.IntegerField(required=True)
    
    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)
        # 实时获取最新的最后一个数字，避免类加载时查询导致数据不更新
        last_nbr = Nbr.objects.order_by("-number").first()
        last_number = last_nbr.number if last_nbr else 0
        self.fields['start_number'].initial = last_number + 1
        self.fields['end_number'].initial = last_number + 1
    
    class Meta:
        model = Nbr
        # 列出表单需要处理的其他字段，排除自动生成或不需要的字段
        fields = ["start_number", "end_number", "somefield", "someotherfield"]
        exclude = ["number", "created_date"]

    def clean(self):
        cleaned_data = super().clean()
        s = cleaned_data.get("start_number")
        e = cleaned_data.get("end_number")
        
        if s is None or e is None:
            return cleaned_data
        if s > e:
            raise forms.ValidationError("起始数字不能大于结束数字")
        
        # 直接用range生成数字序列，无需转列表，提升效率
        self.number_list = range(s, e + 1)
        return cleaned_data
        
    def save(self, commit=True):
        # 收集表单中需要传递给模型的其他字段数据
        other_field_data = {
            field: self.cleaned_data[field]
            for field in ["somefield", "someotherfield"]  # 替换为你的实际字段名
        }
        
        # 批量生成Nbr实例
        instances = [
            Nbr(number=n, **other_field_data)
            for n in self.number_list
        ]
        
        if commit:
            # 批量插入数据库，比循环create效率更高
            Nbr.objects.bulk_create(instances)
        
        # 返回最后一个实例，符合ModelForm save方法的返回约定
        return instances[-1] if instances else None

关键修复点说明 #

• 初始化逻辑移至__init__：避免类加载时就查询数据库导致初始值不实时，每次实例化表单时重新查询最新的最后一个数字。
• 移除错误的父类save调用：原代码中super().save(self)会创建无效实例，直接删除该逻辑。
• 正确处理批量创建：用bulk_create一次性插入所有实例，提升数据库操作效率；同时收集表单中其他字段的数据，确保实例完整。
• 返回单个实例：按照ModelForm的约定返回最后一个实例，避免save_m2m报错（若模型存在多对多字段，该方法可正常执行）。

额外注意事项 #

• 若somefield是外键字段，需确保表单中传入了合法的外键实例或ID，保证cleaned_data中的值有效。
• bulk_create不会触发模型的save方法和信号，若模型依赖这些逻辑，需手动处理，或改用循环create（效率较低）。

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