即使在使用同步根锁定时枚举列表也会引发异常

本文关键字:列表 异常 枚举 定时 同步 锁定 | 更新日期: 2023-09-27 18:14:49

嘿,伙计们,我有以下两个类。Foo类的开始和停止方法被多次调用,有时DoWork方法中的第二个foreach循环会抛出异常"InvalidOperationException: Collection Was Modified"。因为我使用锁,我不知道为什么会发生这种情况。所以,有没有人能给我指出我做错了什么。

感谢
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;
        }
    }
}

即使在使用同步根锁定时枚举列表也会引发异常

我怀疑您省略的" //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对象本身,则不需要这样做——只有在通过添加、删除或插入元素来更改列表时才需要这样做。)