Système.InvalidOperationException: Collection a été modifiée

Je suis une exception suivante lors de l'énumération par une file d'attente:

Système.InvalidOperationException:
La Collection a été modifiée; l'énumération
l'opération ne peut pas exécuter

voici l'extrait de code:

1:    private bool extractWriteActions(out List<WriteChannel> channelWrites)
2:    {
3:        channelWrites = new List<WriteChannel>();
4:        foreach (TpotAction action in tpotActionQueue)
5:        {
6:            if (action is WriteChannel)
7:            {
8:                channelWrites.Add((WriteChannel)action);
9:                lock(tpotActionQueue)
10:               {
11:                  action.Status = RecordStatus.Batched;
12:               }
13:           }
14:       }
15:       return (channelWrites.Count > 0);
16:   }

Je crois que je comprends le problème d'altération de la table de hachage à action.Status = RecordStatus.Batched, qui vis jusqu'à la MoveNext() sur l'énumérateur.
La Question est, comment puis-je la mettre en "modèle" correctement?

  • Pourquoi êtes-vous de verrouillage de la file d'attente? Ce code ne fait pas de sens pour moi.
  • Le plus grand point de l'agent recenseur de la documentation est-ce que "L'agent recenseur n'a pas accès exclusif à la collection; par conséquent, l'énumération d'une collection n'est pas intrinsèquement une procédure thread-safe. Pour garantir la sécurité des threads au cours de l'énumération, vous pouvez verrouiller la collection au cours de l'ensemble de l'énumération. Pour permettre la collecte pour être accessible par plusieurs threads pour la lecture et l'écriture, vous devez mettre en place votre propre synchronisation".
  • La droite. Ce code ne fait rien, même vaguement, comme ça.
  • Envisager l'utilisation d'un lecteur / enregistreur de verrouillage si vous avez besoin de synchroniser des lecteurs et des auteurs sur plusieurs threads.
InformationsquelleAutor kermit_xc | 2009-07-20
  1. 6

    Vous êtes autorisé à changer la valeur d'un élément dans une collection. L'erreur que vous obtenez signifie qu'un article a été ajouté ou retiré à savoir: la collection elle-même a été modifié, et non un élément à l'intérieur de la collection. Ceci est probablement causé par un autre thread ajouter ou supprimer des éléments de cette collection.

    Vous devez verrouiller votre file d'attente au début de votre méthode, pour éviter que d'autres Threads modifiant la collection, tandis que vous y accédez. Ou vous pouvez verrouiller la collection avant même l'appel de cette méthode.

    private bool extractWriteActions(out List<WriteChannel> channelWrites)
    {
      lock(tpotActionQueue)
      {
        channelWrites = new List<WriteChannel>();
        foreach (TpotAction action in tpotActionQueue)
        {
            if (action is WriteChannel)
            {
                channelWrites.Add((WriteChannel)action);

                  action.Status = RecordStatus.Batched;

           }
        }
      }
       return (channelWrites.Count > 0);
   }
    • dang! - juste vérifié, a obtenu un thread différent de pousser des éléments sur la collecte, de la pop sont thread-safe si. Suis un peu inquiet au sujet de la "verrouillage" de la performance sur l'ensemble de l'itération bien. Il est assez sensible - dont elle pourrait avoir besoin réécriture complète :{. Merci bien!
    • vérifié, a obtenu un thread différent de pousser des éléments sur la collection" - c'est votre problème là. change de "foreach" de la boucle "for" n'est pas votre problème et ce n'est pas la réponse. lors de l'utilisation de foreach, vous devez verrouiller votre collection avant le dénombrement. Les agents recenseurs sont pas thread-safe.
    InformationsquelleAutor Stan R.
  2. 8

    Je pense que j'ai eu une exception similaire lors de l'utilisation d'un foreach boucle sur une Collection où j'ai essayé de supprimer des éléments de la Collection (ou il peut avoir été une Liste, je ne me souviens pas). J'ai fini par trouver autour d'elle à l'aide d'un for boucle. Essayez peut-être de quelque chose comme ce qui suit:

    for (int i=0; i<tpotActionQueue.Count(); i++)
{
    TpotAction action = tpotActionQueue.Dequeue();
    if (action is WriteChannel)
    {
        channelWrites.Add((WriteChannel)action);
        lock(tpotActionQueue)
        {
            action.Status = RecordStatus.Batched;
        }
    }
}
    • oui, mais cette méthode n'est pas de retirer ou d'ajouter des éléments à une collection. Le problème est à l'extérieur de cette méthode.
    • Merci beaucoup, ça fait sens ... va donner un coup de feu.
    • Pas de Stan le problème, c'est l'action.Le statut de cession, de lire l'énumération de la documentation pour toute collection. "Un agent recenseur reste valable tant que la collection reste inchangé. Si des modifications sont apportées à la collecte, telles que l'ajout, la modification ou la suppression d'éléments, l'agent recenseur est définitivement invalidé et que son comportement n'est pas défini". Ici, il est la modification d'un élément
    • vous n'avez jamais utilisé une boucle foreach pour initialiser une liste? Liste<Test> t = new List<Test>(); t.Add(new Test()); t.Add(new Test()); foreach (Test t) { objet.Abc = "SomeTest"; }
    • cela a fonctionné parfaitement... merci beaucoup cela m'a sauvé beaucoup de maux de tête!
    • excellent..pour la boucle fait la magie ici.

    InformationsquelleAutor Sarah Vessels
  3. 1

    Vous n'avez pas de définition pour tpotActionQueue, mais si c'est juste normal List<TpotAction> alors que la ligne n'est pas votre problème. Modification de la collecte est de l'ajout ou de la suppression de membres - pas de définition d'une propriété sur un objet contenu.

    Vous avez un lock(tpotActionQueue) et une balise de fil de sécurité, si ma conjecture est qu'il y a un autre thread de l'ajout ou de la suppression d'éléments de tpotActionQueue alors que vous êtes de l'énumération. Vous avez probablement besoin de synchroniser ces accès.

    • c'était mon idée - la modification de la valeur ne devrait pas bousiller la collection. En fin de compte j'ai voulu le supprimer de la file d'attente, mais a plutôt choisi de le marquer comme "lots" de sorte qu'il peut être pris en charge à l'extérieur, dans la file d'attente principale de la boucle - c'est le même fil que.
    • la définition d'une propriété n'comte, msdn.microsoft.com/en-us/library/4a9449ty.aspx
    InformationsquelleAutor Mark Brackett
  4. 0

    Je pense que tout ce que vous devez faire est de cesser d'utiliser le foreach et, au lieu de basculer d'une boucle for

    for(int i = 0; i < tpotActionQueue.Length; i++)
{
     TpotAction action = tpotActionQueue[i];

     if (action is WriteChannel)
     {
        channelWrites.Add((WriteChannel)action);
        lock(tpotActionQueue)
        {
           action.Status = RecordStatus.Batched;
        }
     }
}

    Ce Qui Concerne, Mike.

    • cela va poser un problème dans un environnement multi-thread, il semble tpotActionQueue est un "global" de la variable, et un autre thread peut le modifier si cette méthode est appelée.
    • en outre, vous êtes autorisé à modifier un élément dans une collection, donc je ne vois pas l'intérêt de cette.
    InformationsquelleAutor Michael Ciba
  5. 0

    Comment quelques-uns LINQy bonté?

    private bool extractWriteActions(out List<WriteChannel> channelWrites)
{

   channelWrites= tpotActionQueue.Where<WriteChannel>(x => x is WriteChannel).ToList()

   foreach(WriteChannel channel in channelWrites) {
      channel.Status = RecordStatus.Batched;
   }

  return ( channelWrites.Count > 0);
}
    • ne peut pas faire LINQ .NET 3.0 max. Mais bon 🙂
    InformationsquelleAutor Kirschstein
  6. -1

    Je pense que vous devez avoir un autre thread modifiant la tpotActionQueue alors que vous êtes une itération sur elle. Puisque vous êtes seulement de verrouillage que la file d'attente à l'intérieur de la boucle for, c'est possible.

    InformationsquelleAutor CodeGoat