Perché sto ricevendo "La raccolta è stata modificata, l'operazione di enumerazione potrebbe non essere eseguita" quando non si modifica la raccolta enumerata?

Question

Ho due raccolte di stringhe: CollectionA è una proprietà StringCollection di un oggetto memorizzato nel sistema, mentre CollectionB è una lista generata in fase di esecuzione. La raccolta deve essere aggiornata per corrispondere alla raccolta B se vi sono differenze. Quindi ho ideato quello che mi aspettavo fosse un semplice metodo LINQ per eseguire la rimozione.

var strDifferences = CollectionA.Where(foo => !CollectionB.Contains(foo)); 
foreach (var strVar in strDifferences) { CollectionA.Remove(strVar); } 

Ma sto ottenendo un errore di "Collection was modified; enumeration operation may not execute" su strDifferences ... anche se è enumerabile un separato dalla collezione in corso di modifica! Inizialmente ho ideato questo in modo esplicito per eludere questo errore, in quanto la mia prima implementazione lo avrebbe prodotto (mentre stavo enumerando su CollectionA e rimuovendolo solo quando !CollectionB.Contains(str)). Qualcuno può far luce sul perché questa enumerazione sta fallendo?

Answer 1

I due non sono completamente separati. Where non crea una copia separata della raccolta. Mantiene internamente un riferimento alla collezione originale e recupera elementi da esso come richiesto.

È possibile risolvere il problema aggiungendo ToList() per forzare Where per iterare immediatamente attraverso la raccolta.

var strDifferences = CollectionA 
    .Where(foo => !CollectionB.Contains(foo)) 
    .ToList(); 

Answer 2

Where passa indietro IEnumerable<T> e poi si utilizza che nel vostro foreach (var strVar in strDifferences)

Si sta poi cercando di rimuoverlo dalla collezione che ha creato il IEnumerable<T>. Non hai creato un nuovo elenco, fa riferimento a CollectionA per estrarre l'elemento successivo da, quindi non puoi modificare CollectionA.

È possibile farlo anche:

var strDifferences = CollectionA.Where 
    (foo => CollectionB.Contains(foo)).ToList(); 

CollectionA = strDifferences; 
//or instead of reassigning CollectionA 
CollectionA.Clear(); 
CollectionA.AddRange(strDifferences); 

Dal momento che si sta rimuovendo quelli che non sono in CollectionB. Cerca solo quelli che sono, crea una lista e assegna quella lista alla variabile CollectionA.

Answer 3

provare a modificare la prima linea un po ':

var strDifferences = 
    CollectionA.Where(foo => !CollectionB.Contains(foo)).ToList(); 

Penso LINQ sta usando esecuzione pigro della query. La chiamata a ToList imporrà l'esecuzione della query prima dell'enumerazione.