EF中批量添加实体后SaveChanges报错，分批保存正常，求原因及建议

先还原下你的场景：你创建了Photo和Comment两个实体的列表，代码如下：

// Photo
var photos = new List<Photo>{ 
    new Photo { Title = "Title1", Description = "Description1", CreatedDate = DateTime.Today } 
}; 
// Comment
var comments = new List<Comment> { 
    new Comment{ PhotoId = 1, UserName = "User1", Subject = "Comment1" } 
};

当你尝试把两个列表一起添加到上下文后调用一次SaveChanges()时，出现了「更新条目时发生错误」的异常：

photos.ForEach(s => context.Photos.Add(s)); 
comments.ForEach(s => context.Comments.Add(s)); 
context.SaveChanges();

但如果分两次添加并分别调用SaveChanges()，就完全正常：

photos.ForEach(s => context.Photos.Add(s)); 
context.SaveChanges(); 
comments.ForEach(s => context.Comments.Add(s)); 
context.SaveChanges();

问题原因分析 #

核心问题出在外键关联与数据库主键生成的时机上：

• 你的Comment实体通过PhotoId关联Photo，而Photo的主键大概率是数据库自增类型（比如SQL Server的IDENTITY）。
• 当你一次性添加两个列表并调用SaveChanges()时，EF会尝试同时把Photo和Comment提交到数据库，但此时Photo还没被持久化，数据库还没给它分配真实的自增主键。你手动给Comment设置的PhotoId = 1，在数据库里还不存在对应的Photo记录，直接触发了外键约束的错误。
• 而分两次调用SaveChanges()时，第一次提交会把Photo保存到数据库，数据库生成主键后，EF会把这个真实主键同步回上下文里的Photo实体。这时候再添加Comment，它的PhotoId = 1就对应了数据库中已经存在的Photo记录，外键约束满足，自然能正常保存。

是否需要每次修改后都调用SaveChanges？ #

答案是不需要。SaveChanges()的作用是把上下文追踪的所有变更一次性提交到数据库，通常在完成一组相关操作后调用一次即可。

但针对这种存在依赖关系的实体，你有两种更优雅的处理方式，不用强制分两次调用：

1. 使用导航属性关联实体：不要手动设置PhotoId，而是直接把Comment关联到对应的Photo实例，EF会自动处理主键的同步：

var photo = new Photo { Title = "Title1", Description = "Description1", CreatedDate = DateTime.Today };
var photos = new List<Photo>{ photo };
var comments = new List<Comment> { 
    new Comment{ Photo = photo, UserName = "User1", Subject = "Comment1" } 
};

// 一起添加后调用一次SaveChanges即可
photos.ForEach(s => context.Photos.Add(s)); 
comments.ForEach(s => context.Comments.Add(s)); 
context.SaveChanges();

2. 配置实体的主键生成策略：如果你的Photo主键是手动赋值而非数据库自增，那一次性提交也不会有问题，但这种场景在实际开发中比较少见。

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