Entity Framework Core 在调用 SaveChanges() 时尝试插入重复的记录

Question

我的实体是：

public class Customer
{
   ...
   public virtual ICollection<ShoppingCartItem> ShoppingCartItems { get; set; }
   ...
}

public class ShoppingCartItem
{
   public string CustomerId { get; set; }
   public int ProductId { get; set; }
   public virtual Customer { get; set; }
   public virtual Product{ get; set; }
   ...
}

添加方法是：

public async Task AddAsync(TEntity entity)
{
    await Task.Factory.StartNew(() => this.entities.Add(entity));
}

我要添加的实体是：

ShoppingCartItem()
{
    CustomerId = "xxxxx",
    ProductId = 1,
    Customer = null,
    Product = null 
}

当我调用 SaveChanges() 时，EF 试图为 ShoppingCartItem 插入两条相同的记录。 ShoppingCartItem 仅被创建一次并添加到上下文中。有什么想法可能是错的吗？

编辑：

这就是我调用 AddSync 方法的方式：

public async Task AddNewCartItem(ShoppingCartItem shopingCartItem)
    {
        await this.ShoppingCartItemRepository.AddAsync(shopingCartItem);
        await this.SmartStoreWorkData.CompleteAsync();
    }

Answer 1

DbContext 是 not thread safe。通过 - 毫无意义，如 cmets 中所述 - 在很可能是不同的线程中执行你的 .Add()，你会混淆 DbContext。 Add() 纯粹是内存操作；没有理由尝试使其异步。改变它，我认为它会解决问题。

public void Add(TEntity entity)
{
    this.entities.Add(entity);
}

如果您有任何其他未在问题中显示的类似用法，请将其更改为同步。

您可以与DbContext 进行“适当的”异步，但这仅适用于实际与数据库对话的方法，而不是内存中的方法，并且通常不会涉及Task.<anything> , 只是提供的async 方法。

编辑：为了完整起见，上面的链接适用于 EF6，但在 EF Core 中，DbContext 也是 not thread-safe。

Answer 2

更新：我做了以下事情：

• 克隆了你的回购链接：SmartStoreNETCore
• 使用新的 EF 工具迁移到 .NET Core 1.1（预览版4）
• 添加了如下EF ModelBuilder中指定的配置
• 应用迁移和更新数据库

命令：

dotnet ef --startup-project ../SmartStoreNetCore.Web/ migrations add ChangeShoppingCartItemKey
dotnet ef --startup-project ../SmartStoreNetCore.Web/ database update

• 删除_Layout.cshtml中的以下重复标签

  <script src="~/js/site.js" asp-append-version="true"></script>

site.js 包含 添加到购物车 功能的点击事件处理程序

• 启动了网站，一切都按预期工作，没有重复的购物车商品，并且数量按预期更新

总结

我可以完全确认在删除对site.js 的重复引用之前，以下方法被调用了两次：

 public async Task AddNewCartItem(ShoppingCartItem shopingCartItem)
 {
     await this.ShoppingCartItemRepository.AddAsync(shopingCartItem);
     await this.SmartStoreWorkData.CompleteAsync();
 }

为什么你之前没有通过调试发现这个问题，这对我来说是个谜

EF 模型构建器

您的配置应如下所示：

builder.Entity<ShoppingCartItem>().HasKey(x => x.Id); // Notice this!
builder.Entity<ShoppingCartItem>().Property(x => x.Id).ValueGeneratedOnAdd(); // Also this!
builder.Entity<ShoppingCartItem>().HasOne(s => s.Customer).WithMany(b => b.ShoppingCartItems).OnDelete(DeleteBehavior.Restrict);
builder.Entity<ShoppingCartItem>().HasOne(s => s.Product).WithMany().OnDelete(DeleteBehavior.Restrict);

自动生成一个值不将其定义为主键