C++ CS50 PSET反射滤波器

发布于06月17日

我正在试着解决CS50 Week 4 Pset中的反射滤镜，但我被卡住了.我的逻辑是使用RGBTRIPLE类型的缓冲区在整个图像中将最后一个像素从image[n][width - 1]复制到第一个像素image[n][n].但是，我的代码似乎有问题:

void reflect(int height, int width, RGBTRIPLE image[height][width])
{
    for (int h = 0; h < height; h++)
    {
        RGBTRIPLE buffer;
        for (int w = 0; w < width; w++)
        {
            buffer = image[h][w];
            image[h][w] = image[h][width - w];
            image[h][width - w] = buffer;
        }
    }
    return;
}

推荐答案

您的代码至少是不正确的，因为当h等于height - 1,w等于0时，表达式image[h][width - w]可以访问图像以外的内存，因为数组一行的有效索引范围是[0, width).

但是，即使您将使用表达式image[h][width - w - 1]，循环

for (int w = 0; w < width; w++)

将形成相同的原始行.

如果您想要反转图像的每一行，则该函数可以如下所示

void reflect( size_t height, size_t width, RGBTRIPLE image[height][width] )
{
    for ( size_t h = 0; h < height; h++ )
    {
        for ( size_t w = 0; w < width / 2; w++ )
        {
            RGBTRIPLE buffer;
            buffer = image[h][w];
            image[h][w] = image[h][width - w - 1];
            image[h][width - w - 1] = buffer;
        }
    }
}