0

我有两个 RGB 图像（ppm 格式），我希望能够将顶部图像中不是纯黑色的任何像素叠加到底部图像上。

我可以成功加载图像、保存图像、复制图像......但我无法以上述方式从两个图像中创建图像。

我不会包含我拥有的所有代码，但实现这一点的重要部分是：

struct Pixel
{
    unsigned int r;
    unsigned int g;
    unsigned int b;
}

我重载了它的 == 运算符以便于比较：

bool Pixel::operator==(const Pixel& other)
{
    if(r != other.r)
    {
        return true;
    }
    else if(g != other.g)
    {
        return true;
    }
    else if(b != other.b)
    {
        return true;
    }
    else
    {
        return false;
    }
}

在我的 Pic 类中，我有这个方法：

Pic Pic::overlay(const Pic& top, Pixel mask)
{    
    for(int h = 0; h < height; h++)
    {
        for(int w = 0; w < width; w++)
        {
            if(!(top.pixels[h][w] ==  mask))
            {
                pixels[h][w] = top.pixels[h][w];  // pixels[][] is a Pixel array
            }
        }
    }

    return *this;
}

我的主文件有这个：

Pic top;
Pic bot;
Pic overlay;

Pixel mask:
mask.r = 0;
mask.g = 0;
mask.b = 0;

top.loadimage("top.ppm");  //loadimage() loads the image in and all the data
bot.loadimage("bot.ppm");  //samme thing

overlay = bot.overlay(bot, mask);
overlay.saveimage("overlay.ppm");

显然，Pic 类重载了 = 运算符。

我遇到的问题是：

在覆盖方法中，如果我按照上述方式保留此 if 语句，则顶部图像将显示在保存的文件中。如果我没有 !() 部分，它将显示底部图像。

如果我完全摆脱了 if() 语句，而只是尝试改变单个像素，例如：

pixels[h][w].r = pixels[h][w].r - 50;

由于显而易见的原因，保存的图像将被更改，看起来很古怪。

但是... .b 和 .g 对图像没有影响。

我没有想法......我已经玩了 2 天，但我不知道出了什么问题。在我的程序中，一切都按需要工作，除了这个覆盖方法。

编辑：所以，我在我的代码中发现了一个问题，它回到了我如何加载 PPM P6 格式的图像。我没有将每个像素单独加载为 1 个字节，而是尝试将它们全部加载在一起，因此它创建了结构和从压缩中读取二进制文件时发生的缓冲内容......现在我可以将顶部图像的叠加层放在底部图像，但并非所有颜色都显示。不过，还是比以前好。

这是我修改了覆盖层的嵌套 for() 循环的样子：

for(int h = 0; h < height; h++)
{
    for(int w = 0; w < width; w++)
    {
        if(top.pixels[h][w].r != mask.r &&
           top.pixels[h][w].g != mask.g &&
           top.pixels[h][w].b != mask.b   )
        {
            pixels[h][w].r = top.pixels[h][w].r;
            pixels[h][w].g = top.pixels[h][w].g;
            pixels[h][w].b = top.pixels[h][w].b;
        }

    }
}

显然它仍然需要工作。

4

1 回答 1

1

这一行看起来不对：

overlay = bot.overlay(bot, mask);

不应该是：

overlay = bot.overlay(top, mask);

如果你想要一种更短的方式来编写你的相等测试，那么你可能会喜欢这样：

bool Pixel::operator==(const Pixel& other)
{
    return (r==other.r && g==other.g && b==other.b);
}

最后，既然你有一个相等运算符，那么为什么不做添加和赋值（'='）来让你的编码器像 poss 一样整洁

于 2013-05-22T09:55:55.473 回答