Question

嘿伙计们，我有以下两节课。 Foo类的start和stop方法被多次调用，有时DoWork方法中的第二个foreach循环抛出异常＆＃34; InvalidOperationException：Collection was Modified＆＃34;。由于我使用锁定，我不知道为什么会发生这种情况。所以请一些人指出我正确的方向，我做错了。

由于

class Item 
{
}

class Foo
{
    private List<Item> items;
    private volatile bool bContinue = true;
    private object locker = new object();


    Foo()
    {
        items = new List<Item>();
    }
    public void Add (Item item)
    {
        lock (((ICollection)items).SyncRoot)
        {
            items.Add(item);
        }
    }

    public void Remove(Item item)
    {
        lock (((ICollection)items).SyncRoot)
        {
            items.Remove(item);
        }
    }

    public void Stop()
    {
        lock(locker)
        {
            bContinue = false;
        }
    }

    public void Start()
    {
        Thread worker = new Thread(DoWork);
        worker.Start();
    }

    private void DoWork()
    {
        while (bContinue)
        {
            lock (((ICollection)items).SyncRoot)
            {

               items.ForEach((o) =>
                    {
                       //access memeber variables of o
                    }
                );

                foreach (Item it in items)
                {

                    //Call member methods of it
                }
            }

        }

        lock (locker)
        {
            bContinue = true;
        }
    }

}

Answer 1

我怀疑你省略的“//do some work”代码就是问题所在。可能这段代码正在修改items集合。

您在枚举时无法修改集合，甚至不是来自同一个线程。这是IEnumerable界面的合同。

foreach (Item it in items)
{
    // The following line will cause an exception to be thrown on the next
    // loop iteration, because the iterator will detect that the collection
    // was modified.  You cannot modify a collection while enumerating it.
    items.Remove(it);
}

如果您需要在枚举集合的内容时添加或删除集合中的项目，则必须首先制作集合的副本，然后枚举副本。例如：

foreach (Item it in items.ToList())
{
    //do some work
}

ToList()将创建列表的副本，然后您将枚举此副本。然后您可以自由修改items列表。

（请注意，如果您只修改Item个对象，则不需要这样做 - 只有在通过添加，删除或插入元素来更改列表时才需要这样做。）

枚举列表时抛出异常，即使在使用syncroot锁定时也是如此

1 个答案: