C＃中的Parallel.foreach

Question

我对C＃中的Parallel.Foreach有疑问。 我是多线程的新手，并且从一个月开始学习它。 我正在使用一个Foreach循环，在那个foreach循环中，我正在找到一个整数列表的平均值，并将每个平均值添加到一个当前包中。 但是我面临的问题不是所有迭代的结果都被添加了。 我的列表有10个条目，但是我要添加的当前包在foreach循环的末尾只有不到10个条目。 谁能让我知道我在做什么错。

下面是我的代码：

static void Main(string[] args)
        {
            List<List<int>> intList = new List<List<int>>();
            List<int> ints = new List<int>();
            Random rand = new Random();
            for (int k = 1; k <= 10; k ++)
            {
                ints.Clear();
                for (int i = 0; i < 10000000; i++)
                {
                    ints.Add(rand.Next(10000000));
                }
                List<int> copy = new List<int>(ints);
                intList.Add(copy);
            }
            ConcurrentBag<double> results = new ConcurrentBag<double>();
            Parallel.ForEach<List<int>, double>(intList,
                () => 0,
                (eachIterationList, pls, threadLocal) =>
                {
                    threadLocal = eachIterationList.Average();
                    return threadLocal;
                },
                    (result) =>
                    {
                        lock (results)
                        {
                            results.Add(result);
                        }
                    });
            Console.WriteLine("No of results in results variable: {0}", results.Count);
            Console.ReadLine();
        }

当写入控制台时，结果变量包含少于十个元素，而我认为应该包含十个元素。

Answer 1

为什么要锁定结果ConcurrentBag？ 像这样简单地工作：

        Parallel.ForEach(intList,
            list =>
            {
                results.Add(list.Average());
            });

锁定结果并不能保证您的商品顺序正确。 如果您希望两者都同步，请改用此方法：

List<double> results = new List<double>(Enumerable.Repeat(0.0d, intList.Count));
Parallel.For(0, intList.Count,
(i) =>
{
   results[i] = intList[i].Average();
});

或使用.ForEach（）：

List<double> results = new List<double>(Enumerable.Repeat(0.0d, intList.Count));
Parallel.ForEach(intList,
    (item, loop, i) =>
    {
        results[(int)i] = item.Average();
    });

Answer 2

您可以看到本地的final委托在用于处理Parallel.ForEach的每个任务的最终动作上运行。 每次可以使用不同的数字。 因此，如果您按如下所示修改代码，则会看到用于列表迭代的任务数量与运行最终操作的任务数量相同。 此完成委托的用途不同于您的情况。

您需要了解.NET中的线程和任务之间的区别。 同样，当您使用锁将其添加到ConcurrentBag时，您的代码是错误的，因为它已经支持多线程“添加到集合”，并且您不需要锁定。

 List<List<int>> intList = new List<List<int>>();
        List<int> ints = new List<int>();
        Random rand = new Random();
        for (int k = 1; k <= 10; k++)
        {
            ints.Clear();
            for (int i = 0; i < 10000000; i++)
            {
                ints.Add(rand.Next(10000000));
            }
            List<int> copy = new List<int>(ints);
            intList.Add(copy);
        }
        HashSet<int?> resultTaskIds = new HashSet<int?>();
        HashSet<int?> iterationTaskIds = new HashSet<int?>();
        ConcurrentBag<double> results = new ConcurrentBag<double>();
        Parallel.ForEach<List<int>, double>(intList,
            () => 0,
            (eachIterationList, pls, threadLocal) =>
            {
                iterationTaskIds.Add(Task.CurrentId);

                return eachIterationList.Average();
            },
                (result) =>
                {
                    resultTaskIds.Add(Task.CurrentId);
                    results.Add(result);
                });