Pourquoi je reçois "La collection a été modifiée, l'opération d'énumération peut ne pas s'exécuter" si elle ne modifie pas la collection énumérée?

Question

J'ai deux collections de chaînes: CollectionA est une propriété StringCollection d'un objet stocké dans le système, tandis que CollectionB est une liste générée lors de l'exécution. CollectionA doit être mise à jour pour correspondre à CollectionB s'il y a des différences. J'ai donc conçu ce que je pensais être une simple méthode LINQ pour effectuer la suppression.

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

mais je reçois une erreur "Collection was modified; enumeration operation may not execute" sur strDifferences ... même si elle est un dénombrable séparé de la collection en cours de modification! J'ai initialement conçu ce explicitement pour éviter cette erreur, car ma première implémentation le produirait (comme je l'énumérais à travers CollectionA et juste en supprimant quand !CollectionB.Contains(str)). Quelqu'un peut-il nous expliquer pourquoi cette énumération échoue?

Answer 1

Les deux ne sont pas complètement séparés. Where ne fait pas une copie distincte de la collection. Il conserve en interne une référence à la collection d'origine et récupère les éléments à partir de celui-ci lorsque vous le demandez.

Vous pouvez résoudre votre problème en ajoutant ToList() pour forcer Where à parcourir la collection immédiatement.

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

Answer 2

Where passe en arrière IEnumerable<T> puis vous utilisez ce dans votre foreach (var strVar in strDifferences)

Vous êtes alors essayez de le retirer de la collection qui a créé le IEnumerable<T>. Vous n'avez pas créé de nouvelle liste, elle fait référence à CollectionA pour extraire l'élément suivant, vous ne pouvez donc pas modifier CollectionA.

Vous pouvez le faire aussi:

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

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

Puisque vous supprimez ceux qui ne sont pas Collectionb. Cherchez simplement ceux qui sont, créez une liste et assignez cette liste à la variable CollectionA.

Answer 3

Essayez de modifier la première ligne un peu:

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

Je pense que LINQ utilise l'exécution paresseuse de votre requête. L'appel à ToList forcera l'exécution de la requête avant l'énumération.