CS50模糊函数实现错误求助:3x3测试像素计算结果不符
模糊函数计算错误的原因及修复方案
你的代码错误的核心原因是在计算过程中直接修改了原始图像数组,导致后续像素的模糊计算使用了已经被修改过的像素值,而非原始的像素数据。
以你提供的3x3测试图为例:
- 程序会先处理左上角的像素
10,计算后将其修改为新值(约53); - 接着处理第一行中间的
40时,会用到已经修改后的左上角像素值(53),而非原始的10; - 当计算中间像素
120时,周围多个像素已经被提前修改,最终总和被拉高,导致平均值错误(你得到的145就是因为混入了修改后的像素值)。
正确的做法是先创建一个原始图像的副本,所有模糊计算都基于这个副本的数据,计算完成后再将结果写入原图像。同时可以优化内层循环的遍历逻辑,提升效率:
void blur(int height, int width, RGBTRIPLE image[height][width]) { // 创建原始图像的副本 RGBTRIPLE copy[height][width]; for (int i = 0; i < height; i++) { for (int j = 0; j < width; j++) { copy[i][j] = image[i][j]; } } for (int row = 0; row < height; row++) { for (int column = 0; column < width; column++) { int sumRed = 0; int sumGreen = 0; int sumBlue = 0; int count = 0; // 遍历当前像素周围3x3范围内的像素 for (int i = row - 1; i <= row + 1; i++) { for (int j = column - 1; j <= column + 1; j++) { // 检查像素是否在图像边界内 if (i >= 0 && i < height && j >= 0 && j < width) { sumRed += copy[i][j].rgbtRed; sumGreen += copy[i][j].rgbtGreen; sumBlue += copy[i][j].rgbtBlue; count++; } } } float averagered = (float)sumRed / count; float averagegreen = (float)sumGreen / count; float averageblue = (float)sumBlue / count; image[row][column].rgbtRed = round(averagered); image[row][column].rgbtGreen = round(averagegreen); image[row][column].rgbtBlue = round(averageblue); } } return; }
内容的提问来源于stack exchange,提问作者Anay Chadha
相关产品推荐
相关产品推荐

