EF Core重复附加同一实体报错：无法跟踪重复键值的Role实例

这个问题我之前处理多上下文关联实体时也碰到过，核心原因是EF Core的DbContext会维护一个实体跟踪缓存，当你尝试Attach一个和缓存中已有实体主键相同的对象时，就会抛出这个重复跟踪的异常。针对你的代码场景，这里有几个实用的解决办法：

方法一：先检查上下文是否已跟踪该Role，再决定是否Attach #

在每次Attach之前，先判断当前上下文的本地缓存里有没有相同Id的Role实体，如果没有再执行Attach操作。你可以修改循环里的代码：

foreach (var stage in request.Stages) {
  var temp = new StageTemplate();
  temp.Order = stage.Order;
  temp.Name = stage.Name;
  var role = roleService.GetById(stage.RoleId, schemaName);
  
  // 关键：检查当前上下文是否已经在跟踪这个Role
  var existingRole = context.TenantRoles.Local.FirstOrDefault(r => r.Id == role.Id);
  if (existingRole == null) {
    context.TenantRoles.Attach(role);
  } else {
    // 如果已经存在，就用上下文已跟踪的实例
    role = existingRole;
  }
  
  temp.AvailableActions = new List<ApprovalActionTemplate>();
  foreach (var actionId in stage.Actions)
   temp.AvailableActions.Add(context.ApprovalActions.First(a => a.Id == actionId));
  
  temp.Role = role;
  template.Approvers.Add(temp);
}

这个方法不会破坏你现有roleService的逻辑，只是在当前上下文层面做了跟踪检查，简单直接。

方法二：提前批量获取需要的Role，避免重复处理 #

如果你的request.Stages里可能有多个相同的RoleId，可以先把所有需要的RoleId收集起来，一次性获取并存到字典里，循环的时候直接取字典里的实例，这样就不会重复Attach了：

// 先收集所有需要的RoleId并去重
var requiredRoleIds = request.Stages.Select(s => s.RoleId).Distinct().ToList();
// 一次性获取这些Role并统一Attach
var roleDict = requiredRoleIds.ToDictionary(id => id, id => {
  var role = roleService.GetById(id, schemaName);
  context.TenantRoles.Attach(role);
  return role;
});

using (var context = new TenantContext(schemaName, connectionString)) {
 ApprovalTemplates templates = new ApprovalTemplates();
 ApprovalTemplate template = new ApprovalTemplate();
 template.Approvers = new List<StageTemplate>();
 foreach (var stage in request.Stages) {
  var temp = new StageTemplate();
  temp.Order = stage.Order;
  temp.Name = stage.Name;
  // 直接从字典取已Attach的Role
  var role = roleDict[stage.RoleId];
  
  temp.AvailableActions = new List<ApprovalActionTemplate>();
  foreach (var actionId in stage.Actions)
   temp.AvailableActions.Add(context.ApprovalActions.First(a => a.Id == actionId));
  
  temp.Role = role;
  template.Approvers.Add(temp);
 }
 templates.PRApprovalTemplate = template;
 context.ApprovalTemplates.Add(templates);
 context.SaveChanges();
}

这种方法更高效，减少了多次Attach和检查的开销，适合存在大量重复RoleId的场景。

方法三：修改roleService返回无跟踪实体 #

如果你的roleService使用独立的DbContext，可以让它返回无跟踪的实体，这样拿到的Role本身没有被任何上下文跟踪，即使多次获取同一个Role，Attach到当前上下文时也不会有冲突（不过还是建议配合方法一的检查，更稳妥）。修改roleService.GetById的实现：

// 在roleService的GetById方法里添加AsNoTracking()
public TenantRole GetById(int id, string schemaName) {
  using (var serviceContext = new TenantContext(schemaName, connectionString)) {
    return serviceContext.TenantRoles.AsNoTracking().FirstOrDefault(r => r.Id == id);
  }
}

这样返回的Role是 detached状态，你在当前上下文Attach时，只要上下文还没跟踪相同Id的实体，就可以正常Attach；如果已经跟踪了，还是需要方法一的检查来避免冲突。

方法四：直接设置实体状态（谨慎使用） #

如果你确定要强制处理实体状态，可以通过Entry来检查当前跟踪状态，再决定是否设置：

var role = roleService.GetById(stage.RoleId, schemaName);
var entry = context.Entry(role);
if (entry.State == EntityState.Detached) {
  entry.State = EntityState.Unchanged;
}
// 不需要再调用Attach，设置状态已经相当于Attach操作
temp.Role = role;

这个方法和Attach的效果类似，但更直接地控制了实体状态，不过要注意如果实体已经被跟踪，不要强行修改状态，否则还是会报错，所以最好先判断状态。

总结一下，方法一和方法二是最常用也最安全的解决方案，根据你的实际场景选择就行～

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