并行使用 ConcurrentBag 与简单数组

Question

这是一个假设的问题/用例，基于在此特定用例的并行 for 循环中使用（在这种情况下）ConcurrentBag 而不是简单数组的好处。

该场景基于使用通用管道模式来分析从 1 到 total 结果的数字，并根据其中一个管道操作的输出存储 result，而不是为空。

结果列表的实际顺序很重要，因此使用简单的列表（字符串类型）。add 会根据每个线程决定返回结果的时间而产生奇怪的结果。

我有以下工作代码：

    public IList<string> Execute(int total)
    {
        var items = new ConcurrentBag<AnalyzerResult>();

        Parallel.ForEach(Iterate(1, (total + 1)), d =>
        {
            foreach (IOperation<T> operation in operations)
            {
                var result = operation.Execute(d);
                if (result != null)
                {
                    items.Add(new AnalyzerResult(d, result));
                    break;
                }
            }
        });

        return items.OrderBy(o=>o.SortOrder).Select(d => d.Result).ToList();
    }

AnalyzerResult 是一个简单的不可变类，并且代码只会将新项目推送到袋子中（因此理论上不会有 items 列表中的某些内容被更改的危险）。

基于此，一个简单的数组是否足够（并且包含更少的代码噪音）？或者使用并发类型会被认为是更好的实践/更高的性能？例如：

    public IList<string> Execute(int total)
    {
        var items = new string[total];

        Parallel.ForEach(Iterate(1, (total + 1)), d =>
        {
            foreach (IOperation<T> operation in operations)
            {
                var result = operation.Execute(d);
                if (result != null)
                {
                    items[(d - 1)] = result;
                    break;
                }
            }
        });

        return items.ToList();
    }

注意：这不是并发问题，这两种方法都是合法的，并且可以毫无问题地产生所需的结果。

Answer 1

我最初回答“您需要并发保护”，但后来重新阅读了您问题的第二部分。

这看起来应该可以工作，因为您不会尝试从两个不同的线程写入内存中的同一位置。因此，消除锁和线程关联（ConcurrentBag 提供的）应该会显着提高性能。

真正的问题是 - 增加了多少，是否需要增加（需要分析），您将来是否会更改此设置，以便您需要并发保护。

按原样，它应该没问题，而且可读性很强。您可能想要评论这段代码，说明您为什么这样做，以确保有人不会随便浏览它并认为“并发问题”（就像我刚才所做的那样）并“修复”它。