我正在使用下面的代码
var processed = new List<Guid>();
Parallel.ForEach(items, item =>
{
processed.Add(SomeProcessingFunc(item));
});
上面的代码线程安全吗?处理列表是否有可能损坏?或者我应该在添加之前使用锁?
var processed = new List<Guid>();
Parallel.ForEach(items, item =>
{
lock(items.SyncRoot)
processed.Add(SomeProcessingFunc(item));
});
谢谢。
最佳答案
不!它根本不安全,因为 processed.Add
不是。您可以执行以下操作:
items.AsParallel().Select(item => SomeProcessingFunc(item)).ToList();
请记住,Parallel.ForEach
主要是为序列中每个元素的命令式 操作创建的。你所做的是映射:投影序列的每个值。 Select
就是为此而创建的。 AsParallel
以最有效的方式跨线程扩展它。
这段代码可以正常工作:
var processed = new List<Guid>();
Parallel.ForEach(items, item =>
{
lock(items.SyncRoot)
processed.Add(SomeProcessingFunc(item));
});
但在多线程方面毫无意义。 lock
在每次迭代时强制完全顺序执行,一堆线程将等待单个线程。
关于c# - List<T> 线程安全,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/5020486/