rimozione di articoli da un elenco generico <t>

Question

Ho il seguente metodo, desidero rimuovere gli elementi dalla mia raccolta che corrispondono all'id del prodotto. Sembra abbastanza semplice, ma ho un'eccezione. Fondamentalmente la mia collezione sta andando fuori sincrono. Quindi qual è il modo migliore per rimuovere un oggetto da una collezione.

public void RemoveOrderItem(Model.Order currentOrder, int productId) 
{ 

    foreach (var orderItem in currentOrder.OrderItems) 
    { 
     if (orderItem.Product.Id == productId) 
     { 
      currentOrder.OrderItems.Remove(orderItem); 
     } 
    } 
} 

Dettagli eccezione: System.InvalidOperationException: Collection è stato modificato; operazione di enumerazione non può essere eseguita

Answer 1

La modifica di una raccolta all'interno di un loop non funziona. Per ovviare a questo, List ha alcuni metodi che consentono modifiche "batch" di una raccolta. Nel tuo caso, utilizzare:

currentOrder.OrderItems.RemoveAll(x => x.Product.Id == productId) 

Answer 2

Non è possibile modificare una raccolta durante l'iterazione. Basta utilizzare un ciclo normale for anziché un ciclo foreach.

Answer 3

Non è possibile rimuovere un elemento da una raccolta che si sta scorrendo, si potrebbe tenere traccia del orderItem, quindi rimuoverlo dopo aver terminato il ciclo

Answer 4

Con loop in questo modo non è possibile rimuovere gli elementi perché la sua in raccolta conserva la traccia degli oggetti memorizzati.

modo semplice per farlo:

authorsList.RemoveAll(x => x.ProductId == productId); 

o

authorsList = authorsList.Where(x => x.ProductId!= productId).ToList(); 

Answer 5

Come vi rendete conto non è possibile rimuovere un elemento da una raccolta mentre si sta loop su di esso. Sono sicuro che qualcuno sarà in grado di fornito una soluzione più ordinato LINQ, ma il seguente dovrebbe farti andare inizialmente:

public void RemoveOrderItem(Model.Order currentOrder, int productId) 
{ 
    var selectedOrderItem = null; 
    foreach (var orderItem in currentOrder.OrderItems) 
    { 
     if (orderItem.Product.Id == productId) 
     { 
      selectedOrderItem = orderItem; 
      break; 
     } 
    } 

    if(selectedOrderItem != null) 
     currentOrder.OrderItems.Remove(selectedOrderItem); 
} 

Answer 6

"foreach" fornisce un "forward-only di sola lettura" l'iterazione di una collezione.

Come soluzione temporanea, è possibile copiare il riferimento a un'altra raccolta e quindi scorrere sulla raccolta copiata e rimuovere gli elementi da quello originale.