1

以下代码：

#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

我想我应该使用“可组合”，但我找到的关于它的文档不是很好。你知道如何使这段代码正常工作吗？如果 vett 是二维向量怎么办？

既然我想学习并行计算，那么值得学习这个新的微软库还是有更好的选择？

4

2 回答 2

2

您的代码的主要问题是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循环完成后的最终值。

于 2012-02-14T14:18:00.243 回答
1

并行代码的一个好的经验法则是尽量减少共享变量的数量。

因此，您永远不应该将(or )[&]用作 lambda 捕获。仅捕获需要共享的变量。parallel_forstd::thread

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

于 2012-02-14T15:23:41.033 回答