Question

以下代码：

#include <ppl.h>
int i;
vector<int> val(10),summ(10,0);
for(i=0;i<10;i++) val[i]=i;

parallel_for(0, 10, [&] (int y){
    vector<int> vett(1000);
    double vall=val[y];

    for(i=0;i<vett.size();i++)
        vett[i]=vall;

    for(i=0;i<vett.size();i++)
        summ[y]+=vett[i];
 });

for(i=0;i<10;i++)
cout<<summ[i]<<endl;

生成随机输出，如： 0 1000 1468 204 3600 25 5898 7000 7456 1395

我应该使用＆＃34;可组合＆＃34;我想，但我发现的文件并不是很好。你知道怎么让这段代码正常工作吗？如果vett是一个二维矢量怎么办？

由于我想学习并行计算，是否值得学习这个新的微软库或者有更好的替代方案？

Answer 1

代码的主要问题是i下标变量。它由多个并行任务同时使用。随之而来的是混乱，这是结果不稳定的原因。最简单的解决方法是在lambda中声明循环，如下所示：

for(int i=0;i<vett.size();i++)
    vett[i]=vall;

for(int i=0;i<vett.size();i++)
    summ[y]+=vett[i];

请注意，每个循环索引都在for循环初始化中声明。这是更惯用的用法，可以在所有循环中使用，除非在循环结束后你真的需要i的最终值。

Answer 2

并行代码的一个好的经验法则是最小化共享变量的数量。

因此，您不应该使用[&]作为parallel_for（或std::thread）的lambda捕获。仅捕获需要共享的变量。

如果你这样做，你会发现@Blastfurnace指出的问题，即i在所有工人之间共享。

人，如何正确使用它？

2 个答案: