CS50 Pset4 Filter模块blur函数Check50报错求助

你的代码核心问题是边界判断不完整：只检查了邻接像素的行/列号是否非负，但没限制它们不能超过图像的高度和宽度。当处理最右边缘（j = width-1时，j+1 = width）或底部边缘（i = height-1时，i+1 = height）的像素时，会访问超出数组范围的内存，导致模糊计算的总和和计数错误，这就是Check50报错的原因。

错误代码的问题点 #

你用p[x][y] >= 0 && q[x][y] >= 0判断像素是否有效，但忽略了：

• 行号必须小于图像高度：p[x][y] < height
• 列号必须小于图像宽度：q[x][y] < width

另外，你定义的p和q二维数组完全是冗余的，直接通过循环生成邻接的行和列更简洁高效。

修正后的代码 #

void blur(int height, int width, RGBTRIPLE image[height][width])
{
    RGBTRIPLE temp[height][width];

    for (int i = 0; i < height; i++)
    {
        for (int j = 0; j < width; j++)
        {
            float sumRed = 0, sumGreen = 0, sumBlue = 0;
            int counter = 0;

            // 遍历当前像素周围3x3的区域
            for (int row = i - 1; row <= i + 1; row++)
            {
                for (int col = j - 1; col <= j + 1; col++)
                {
                    // 检查当前行和列是否在图像范围内
                    if (row >= 0 && row < height && col >= 0 && col < width)
                    {
                        sumRed += image[row][col].rgbtRed;
                        sumGreen += image[row][col].rgbtGreen;
                        sumBlue += image[row][col].rgbtBlue;
                        counter++;
                    }
                }
            }

            // 计算模糊后的像素值并赋值给临时数组
            temp[i][j].rgbtRed = round(sumRed / counter);
            temp[i][j].rgbtGreen = round(sumGreen / counter);
            temp[i][j].rgbtBlue = round(sumBlue / counter);
        }
    }

    // 将临时数组的结果复制回原图像
    for (int i = 0; i < height; i++)
    {
        for (int j = 0; j < width; j++)
        {
            image[i][j] = temp[i][j];
        }
    }
}

关键修改说明 #

1. 简化邻接像素遍历：直接用row从i-1到i+1，col从j-1到j+1，不需要额外定义二维数组，代码更易读。
2. 完整的边界检查：确保只有在row和col都在合法索引范围内时，才累加像素值并计数。
3. 变量拆分：把sumRGB拆成单独的sumRed、sumGreen、sumBlue，代码逻辑更清晰，避免数组索引出错。

这样修改后，边缘和角落的像素会正确计算周围有效像素的平均值，Check50的测试用例就能通过了。

内容的提问来源于stack exchange，提问作者Emma R