c# - Entity Framework 4.1。更新多对多关系。这是正确的方法吗？

Question

下面的代码有效，但是，我怀疑我遗漏了什么。有“更好”的方法吗？

    private void UpdateNew(MarketProduct marketproduct)
    {
        context.MarketCategories.Load();
        MarketProduct dbProd = context.MarketProducts.Find(marketproduct.Id);
        dbProd.Categories.Clear();
        foreach (var c in marketproduct.Categories ?? Enumerable.Empty<MarketCategory>())
        {
            var cc = context.MarketCategories.Find(c.Id);
            dbProd.Categories.Add(cc);
        }
        context.Entry(dbProd).CurrentValues.SetValues(marketproduct);
    }

我认为不使用 Find 就可以做到这一点

Answer 1

您有三个数据库查询:1) context.MarketCategories.Load()(希望类别表很小，否则这将是不行的，因为它将整个表加载到内存中) , 2) ...Find 和 3) dbProd.Categories.Clear():这里肯定是涉及到了延迟加载，否则会崩溃，因为dbProd.类别 将为null。

使用单个数据库查询更新的替代方法如下:

private void UpdateNew(MarketProduct marketproduct)
{
    MarketProduct dbProd = context.MarketProducts
        .Include(p => p.Categories)
        .Single(p => p.Id == marketproduct.Id);

    var categories = marketproduct.Categories 
                     ?? Enumerable.Empty<MarketCategory>();
    foreach (var category in categories)
    {
        if (!dbProd.Categories.Any(c => c.Id == category.Id))
        {
            // means: category is new
            context.MarketCategories.Attach(category);
            dbProd.Categories.Add(category);
        }
    }
    foreach (var category in dbProd.Categories.ToList())
    {
        if (!categories.Any(c => c.Id == category.Id))
            // means: category has been removed
            dbProd.Categories.Remove(category);
    }

    context.Entry(dbProd).CurrentValues.SetValues(marketproduct);

    // context.SaveChanges() somewhere
}