supprimer des éléments d'une Liste générique<t>

J'ai la méthode suivante, je souhaite supprimer des éléments de ma collection qui correspondent à l'Id de produit. Semble assez simple, mais j'obtiens une exception. Fondamentalement, ma collection est hors de synchronisation. Alors, quelle est la meilleure façon de supprimer un élément d'une collection.

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

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

Détails De L'Exception: System.InvalidOperationException: Collection a été modifiée; opération d'énumération peut pas exécuter

OriginalL'auteur frosty | 2010-06-08

  1. 28

    De la modification d'une collection à l'intérieur d'une boucle ne fonctionne pas. Pour contourner ce problème, List a quelques méthodes qui permettent de “lot” modification d'une collection. Dans votre cas, utilisez:

    currentOrder.OrderItems.RemoveAll(x => x.Product.Id == productId)
    grâce à conrad, bizarrement je ne peux pas obtenir le lamda de travail. Il ne reconnaît pas le "x.Produit.Id de la partie". Étrange parce que les œuvres suivantes var query = de x dans currentOrder.OrderItems où x.Produit.Id == productId sélectionnez x; Le type de collection est ISet.
    ok, j'ai répondu à ma propre question 🙂 j'ai changé à la Liste<t>

    OriginalL'auteur Konrad Rudolph

  2. 5

    Vous ne pouvez pas modifier une collection lors de l'itération. Il suffit d'utiliser un normal for boucle au lieu d'un foreach boucle.

    🙂 c'était la meilleure solution pour moi

    OriginalL'auteur Hans Olsson

  3. 3

    Par le bouclage de cette façon, vous ne pouvez pas supprimer des éléments en raison de sa collection, il garde la trace des éléments stockés.

    Moyen facile de faire cela : 

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

    ou

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

    OriginalL'auteur Pranay Rana

  4. 2

    Vous ne pouvez pas supprimer un élément d'une collection en parcourant, vous pouvez garder une trace de la orderItem, puis l'enlever après avoir terminer la boucle

    OriginalL'auteur BenW

  5. 1

    Que vous vous rendez compte que vous ne pouvez pas supprimer un élément d'une collection, tandis que vous êtes en boucle sur elle. Je suis sûr que quelqu'un sera en mesure de bénéficier d'une plus soignée LINQ solution, mais les points suivants devraient vous aider à aller d'abord:

    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);
}
    Konrad Rudolph a donné la solution LINQ
    Notez que List<T>.RemoveAll est disponible dans .NET 2.0 et est techniquement pas LINQ, mais il ressemble. C'est un énorme plus sur LINQ pour ces pauvres âmes qui ont pour maintenir la compatibilité Windows 2000.

    OriginalL'auteur openshac

  6. 0

    "foreach" fournit un "Avant uniquement en lecture seule" itération d'une collection.

    Comme une solution de contournement, vous pouvez copier la référence à une autre collection, puis itérer sur la copie de collecte et de supprimer les éléments de l'original.

    OriginalL'auteur Manish Basantani