Ошибки при добавлении нескольких элементов объекта

Question

for (int i = 0; i < skus.Count; i++) 
{ 
    sku item = new sku(); 
    item = skus[i]; 
    sku sku = CompanyDbContext.skus.Where(s => s.item_no == item.item_no).FirstOrDefault(); 

    if (sku == null) // ok to insert [no duplicate item numbers] 
    { 
     CompanyDbContext.skus.Add(item);        
    } 

} 
CompanyDbContext.SaveChanges(); 

Я получаю

коллекция была изменена операция перечисления не может выполнить

ошибки. Как я могу это исправить ?

Answer 1

Как уже упоминалось в комментариях, это происходит потому, что вы изменяете коллекцию, которую вы Перебор, как вы выполняете свою работу.

Один из вариантов у вас есть, чтобы создать временную коллекцию и добавить свои sku элементы к этому, и, наконец, добавить содержимое временного List<sku> к вашему CompanyDbContext

// Create a new temporary list 
List<sku> tempSkus = new List<sku>(); 
for (int i = 0; i < skus.Count; i++) 
{ 
    // Let's assign item to skus[i] immediately, we don't need a new instance here when we're later re-pointing to an existing instance 
    sku item = skus[i]; 

    // Use LINQ Any function to determine whether there are any existing SKU's already 
    bool existingSku = CompanyDbContext.skus.Any(s => s.item_no == item.item_no); 

    // There are no duplicates, let's add this sku item to our temporary List 
    if(!existingSku) 
    { 
     tempSkus.Add(item); 
    } 
} 
// Add the Range of tempSkus List to the CompanyDbContext 
CompanyDbContext.skus.AddRange(tempSkus); 
CompanyDbContext.SaveChanges(); 

Или, если вы предпочитаете LINQ

// Create a new temporary list 
List<sku> tempSkus = skus.Where(p => CompanyDbContext.skus.Any(s => s.item_no != p.item_no)).ToList(); 
// Add the Range of tempSkus List to the CompanyDbContext 
CompanyDbContext.skus.AddRange(tempSkus); 
CompanyDbContext.SaveChanges(); 

Answer 2

Проблема в том, что вы изменяете то же самое, что и итерации. Как передовой практики вы должны обновить свой метод что-то вроде этого:

//get search predicat from List<sku> skus 
var item_nos = skus.Select(s=>s.item_no).ToList(); 
//items already in repo 
var addedItems = CompanyDbContext.skus.Where(s => item_nos.Contains(s.item_no)).ToList(); 
var newItems = skus.Except(addedItems).ToList(); 
foreach(var sku in newItems){ 
    CompanyDbContext.skus.Add(item); 
} 
CompanyDbContext.SaveChanges();