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

Question

嘿，伙计们，我有以下两节课。Foo类的start和stop方法被多次调用，有时DoWork方法中的第二个foreach循环抛出异常"InvalidOperationException: InvalidOperationException已被修改“。由于我使用锁，我不知道为什么这是happening.So，请您给我指出一个正确的方向，我做错了什么。

谢谢

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对象本身，则不需要这样做--只有在通过添加、删除或插入元素来更改列表时才需要这样做。)