Question

我正在为CS50开发边缘程序。我在此程序上遇到了很多问题，但是我感觉自己越来越近了。如果有人可以帮助我解决可能出现的问题，我将非常感激。

这是我的代码：

// Detect edges
void edges(int height, int width, RGBTRIPLE image[height][width])
{
RGBTRIPLE copy[height][width];

for (int h = 0; h < height; h++)
{
    for (int w = 0; w < width; w++)
    {
        copy[h][w] = image[h][w];
    }
}

for (int y = 0; y < height; y++)
{
    for (int x = 0; x < width; x++)
    {
        int gxred = 0;
        int gxgreen = 0;
        int gxblue = 0;
        int gyred = 0;
        int gygreen = 0;
        int gyblue = 0;
        for (int r = -1; r < 2; r++)
        {
            for (int s = -1; s < 2; s++)
            {
                if ((y + r < height && y + r >= 0) && (x + s < width && x + s >= 0))
                {
                    if (r == -1 || r == 1)
                    {
                        gxred += copy[y + r][x + s].rgbtRed * s;
                        gxgreen += copy[y + r][x + s].rgbtGreen * s;
                        gxblue += copy[y + r][x + s].rgbtBlue * s;
                    }
                    if (r == 0)
                    {
                        gxred += copy[y + r][x + s].rgbtRed * (s * 2);
                        gxgreen += copy[y + r][x + s].rgbtGreen * (s * 2);
                        gxblue += copy[y + r][x + s].rgbtBlue * (s * 2);
                    }

                }
                if ((y + s < height && y + s >= 0) && (x + r < width && x + r >= 0))
                {
                    if (s == -1 || s == 1)
                    {
                        gyred += copy[y + s][x + r].rgbtRed * s;
                        gygreen += copy[y + s][x + r].rgbtGreen * s;
                        gyblue += copy[y + s][x + r].rgbtBlue * s;
                    }
                    if (s == 0)
                    {
                        gyred += copy[y + s][x + r].rgbtRed * (s * 2);
                        gygreen += copy[y + s][x + r].rgbtGreen * (s * 2);
                        gyblue += copy[y + s][x + r].rgbtBlue * (s * 2);
                    }
                }
            }
        }
        int red = round(sqrt((gxred * gxred) + (gyred * gyred)));
        int green = round(sqrt((gxgreen * gxgreen) + (gygreen * gygreen)));
        int blue = round(sqrt((gxblue * gxblue) + (gyblue * gyblue)));

        if (red > 255)
        {
            red = 255;
        }
        if (blue > 255)
        {
            blue = 255;
        }
        if (green > 255)
        {
            green = 255;
        }

        image[y][x].rgbtRed = red;
        image[y][x].rgbtGreen = green;
        image[y][x].rgbtBlue = blue;

    }
}
return;
}

这是我的错误：

:(边缘正确过滤中间像素预期为“ 210150 60 \ n”，而不是“ 199150 60 \ n”

:(边缘正确过滤边缘上的像素应该是“ 213228255 \ n”，而不是“ 191198255 \ n”

:(边缘正确过滤了角落的像素应该是“ 76117255 \ n”，而不是“ 5892248 \ n”

:(边缘正确过滤3x3图像应该是“ 76117255 \ n21 ...”，而不是“ 5892248 \ n191 ...”

:(边缘正确过滤4x4图像应该是“ 76117255 \ n21 ...”，而不是“ 5892248 \ n191 ...”

Answer 1

R，G和B的gx，gy可能会通过运行if而改变。

测试：您可能希望在“将值传回图像”循环之前打印出它们的值

Answer 2

如果问题仍然存在：

评论将非常有帮助。.也许可以使用浮点数进行计算。我不知道sqrt是否会自动转换其结果，或者您是否得到sqrt-int。

目前我是自己做的，因此尝试时我将了解我的工作原理。

Answer 3

我刚刚通过了check50并提交了我的代码。我猜您收到的错误消息表明在“ if”条件下存在一些逻辑缺陷，无法很好地处理边缘和角落像素。昨天我花了几个小时试图弄清楚如何处理那些边缘和角落而没有成功，这确实很痛苦。突然间，我想到了一个绝妙的主意：制作原始图像的副本时，还可以在副本中添加四个黑色边缘，因此在副本图像中，原始图像位于中间，被四个黑色边缘包围。然后，您只需要处理中间像素，而不再需要处理边缘或角像素。考虑到这个想法，编写代码简直是小菜一碟。如果需要，我可以在这里发布我的代码。

我的CS50边缘过滤器程序的测试失败

3 个答案: