即使在使用同步根锁定时枚举列表也会引发异常
本文关键字:列表 异常 枚举 定时 同步 锁定 | 更新日期: 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
对象本身,则不需要这样做——只有在通过添加、删除或插入元素来更改列表时才需要这样做。)