c# - List 线程安全

Question

我正在使用下面的代码

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));
});

谢谢。

Answer 1

不!它根本不安全，因为 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在每次迭代时强制完全顺序执行，一堆线程将等待单个线程。