c++ - OpenMP/C++ 如何并行递增一个变量？

Question

我有一个问题，大问题=。 我有两个图像(使用 GDIplus)，我想比较像素像素。 当 pixelA = pixelB 时，变量 cont 应该递增。

今天我比较两个相等的图像，我的返回应该是100%，但是这个返回是70%。

为什么？我该如何解决这个问题？

看

    #pragma omp parallel for schedule(dynamic) 
    for (int x = 0; x < height; x++){
        for (int y = 0; y < width; y++){
            int luma01 = 0, luma02 = 0;


            Gdiplus::Color pixelColorImage01;
            Gdiplus::Color pixelColorImage02;

                myImage01->GetPixel(x, y, &pixelColorImage01);
                luma01 = pixelColorImage01.GetRed() + pixelColorImage01.GetGreen() + pixelColorImage01.GetBlue();
                myImage02->GetPixel(x, y, &pixelColorImage02);
                luma02 = pixelColorImage02.GetRed() + pixelColorImage02.GetGreen() + pixelColorImage02.GetBlue();

                #pragma omp critical
                if (luma01 == luma02){
                    cont++;
                }
            }
        }

图像之间的相等百分比

谢谢 =)

Answer 1

在并行化您的解决方案之前，请确保您可以按顺序解决它。在这种情况下，这意味着注释掉 #pragma 并首先对其进行调试。

首先，

for (int x = 0; x < height; x++){
    for (int y = 0; y < width; y++){
        ...
        myImage01->GetPixel(x, y, &pixelColorImage01);

您调换了 width 和 height，所以对于任何非正方形的图像，您都会得到错误的答案。

其次，您的像素平等指标容易发生冲突。由于您将各个颜色的亮度相加然后比较该总和，因此它会认为，例如，全红色像素等于全蓝色像素。
改为做这样的事情:

if (red1 == red2 && green1 == green2 && blue1 == blue2)
    cont++;

至于您的并行化，它在技术上是正确的，但会给您带来糟糕的性能。您在 if 周围放置了一个关键部分，这意味着是否所有工作人员都在不断尝试获取该锁。换句话说，您有并行工作人员，但每个工作人员都必须等待所有其他工作人员。换句话说，您已经序列化了您的并行代码。要解决此问题，请查找 OpenMP reducer 。