c# - 将 for 循环转换为 Task.Parallel.For

Question

我有一个程序 bool IsExistImage(int i) 。程序的任务是检测图像并返回 bool 值，无论它是否存在。

我有一个 100 多页的 PDF，我通过该方法拆分并仅发送文件名。文件名实际上是主 PDF 文件的页码。像 1,2,3,...,125,..

检测图像后，我的方法正确保存了页面列表。为此，我使用了这段代码:

ArrayList array1 = new ArrayList();
for(int i=1;i<pdf.length;i++)
{
   if(isExistImage(i))
   {
       array1.add(i);
   }
}

这个过程运行了 1 个多小时(显然是因为 isExistImage() 方法的内部工作。)。我可以向您保证，在方法范围之外没有任何对象/变量是全局的。

所以，为了缩短时间，我使用了 Task.Parallel For 循环。这是我所做的:

System.Threading.Tasks,Parallel.For(1,pdf.Length,i =>
{
    if(isExistImage(i))
        array1.Add(i);
}

但这不能正常工作。有时图像检测是正确的。但大多数时候是错误的。当我使用非并行 for 循环时，它总是正确的。

我不明白这里的问题是什么。我应该在这里申请什么。我缺少什么技术吗？

Answer 1

你的问题是ArrayList (以及大多数其他 .Net 集合)不是线程安全的。

有几种方法可以解决这个问题，但我认为在这种情况下，最好的选择是使用 PLINQ:

List<int> pagesWithImages = ParallelEnumerable.Range(1, pdf.Length)
    .Where(i => isExistImage(i))
    .ToList();

这将使用多个线程来调用(奇怪的名字)isExistImage方法，这正是你想要的，然后返回一个 List<int>包含符合条件的索引。

返回的列表不会被排序。如果需要，请添加 AsOrdered()在 Where() 之前.

顺便说一句，你真的不应该使用 ArrayList .如果你想要一个整数列表，使用 List<int> .