无效的操作数可转换为二进制表达式（＆＃x27; rgbtriple＆＃x27; and＆＃x27; int€＃x27;）

Question

当我尝试查看特定值是否存在时，我会遇到此错误。我使用了CS50为我提供的自定义数据类型，称为rgbtriple。如果该值不存在在内存中价值。因此，我使用了一个称为isnull的函数。它将检查该值是否存在在内存中。如果不是零，则将返回1，否则为0。

这是我的代码

// Blur image
void blur(int height, int width, RGBTRIPLE image[height][width])
{
    // entering the array of the image
    for (int i = 0; i < height; i++)
    {
        // entering the array of the row
        for (int j = 0; j < width; j++)
        {
            /* blur: the element we select we need to get the value
               of its neighbour value and adding this all including it
               then get the avg value of pixel that need to set
             */

            int element = 0;
            int blueSum = 0;
            int greenSum = 0;
            int redSum = 0;

            RGBTRIPLE pixels[] = {
                image[i][j], image[i][j - 1], image[i][j + 1],
                image[i - 1][j - 1], image[i - 1][j], image[i - 1][i + 1],
                image[i + 1][j - 1], image[i + 1][j], image[i + 1][j + 1]
            };

            for (int k = 0; k < 9; k++)
            {
                if (isNull(pixels[k]) == 1)
                {
                    element++;
                    blueSum += pixels[k].rgbtBlue;
                    greenSum += pixels[k].rgbtGreen;
                    redSum += pixels[k].rgbtRed;
                }
            }

            image[i][j].rgbtBlue = round(blueSum / element);
            image[i][j].rgbtGreen = round(greenSum / element);
            image[i][j].rgbtRed = round(redSum / element);
        }
    }
    return;
}

// check whether it is null or not
int isNull(RGBTRIPLE pixel)
{
    if (pixel != 0)
    {
        return 1;
    }

    return 0;
}

错误我得到了：

$ make filter
helpers.c:142:15: error: invalid operands to binary expression ('RGBTRIPLE' and 'int')
    if (pixel != 0)
        ~~~~~ ^  ~
1 error generated.
make: *** [Makefile:2: filter] Error 1

Answer 1

类型rgbtriple是在 bmp.h 中定义的：

/**
 * RGBTRIPLE
 *
 * This structure describes a color consisting of relative intensities of
 * red, green, and blue.
 *
 * Adapted from http://msdn.microsoft.com/en-us/library/aa922590.aspx.
 */
typedef struct
{
    BYTE  rgbtBlue;
    BYTE  rgbtGreen;
    BYTE  rgbtRed;
} __attribute__((__packed__))
RGBTRIPLE;

它是一个结构：您不能将结构与==操作员进行比较，您必须比较会员单独。

问题是：您是什么意思，请检查它是否为 是什么？

“像素是黑色的？

// check whether it is black or not
int isBlack(RGBTRIPLE pixel) {
    return ((pixel.rgbtBlue | pixel.rgbtGreen | pixel.rgbtRed) == 0);
}

如果您的意思是 >：

• 第六个初始化器映像[i -1] [i + 1]有一个错字，
• 您必须为图像边界制作特殊情况（i == 0，<代码> j == 0 ，<代码> i ==高度 - 1 和j == width -1）。

这是一个简单的修复：

int min(int a, int b) { return a < b ? a : b; }
int max(int a, int b) { return a < b ? b : a; }
// Blur image
void blur(int height, int width, RGBTRIPLE image[height][width])
{
    // entering the array of the image
    for (int i = 0; i < height; i++) {
        // entering the array of the row
        for (int j = 0; j < width; j++) {
            /* blur: compute the new color by averaging the components
               of all pixels in a 3x3 area around the pixel.
               assume that pixel colors continue beyond the image
               borders.
             */
            unsigned blueSum = 0;
            unsigned greenSum = 0;
            unsigned redSum = 0;
            int i1 = max(0, i - 1);
            int i2 = min(height - 1, i + 1);
            int j1 = max(0, j - 1);
            int j2 = min(width - 1, j + 1);

            RGBTRIPLE pixels[] = {
                image[i][j1], image[i][j], image[i][j2],
                image[i1][j1], image[i1][j], image[i1][j2],
                image[i2][j1], image[i2][j], image[i2][j2]
            };

            for (int k = 0; k < 9; k++) {
                blueSum += pixels[k].rgbtBlue;
                greenSum += pixels[k].rgbtGreen;
                redSum += pixels[k].rgbtRed;
            }
            image[i][j].rgbtBlue = round(blueSum / 9);
            image[i][j].rgbtGreen = round(greenSum / 9);
            image[i][j].rgbtRed = round(redSum / 9);
        }
    }
}

但是，请注意，上面的函数无法按编码工作，因为它覆盖将用于下一列和下一行的像素值。要执行此转换，您可以使用3行缓冲区来保持先前的值。

这是一个修改版本：

typedef unsigned char BYTE;

typedef struct {
    BYTE rgbtBlue;
    BYTE rgbtGreen;
    BYTE rgbtRed;
} RGBTRIPLE;

// Blur image
void blur(int height, int width, RGBTRIPLE image[height][width])
{
    RGBTRIPLE row[3][width + 2];

    // populate the initial the row cache
    row[1][0] = row[2][0] = image[0][0];
    row[1][width + 1] = row[2][width + 1] = image[0][width - 1];
    for (int j = 0; j < width; j++) {
        row[1][j + 1] = row[2][j + 1] = image[0][j];
    }

    // entering the array of the image
    for (int i = 0; i < height; i++) {
        // update the row cache
        for (int j = 0; j < width + 2; j++) {
            row[0][j] = row[1][j];
            row[1][j] = row[2][j];
        }
        if (i < height - 2) {
            row[2][0] = image[i + 1][0];
            row[2][width + 1] = image[i + 1][width - 1];
            for (int j = 0; j < width; j++)
                row[2][j + 1] = image[i + 1][j + 1];
        }
        // update the image row
        for (int j = 0; j < width; j++) {
            /* blur: compute the new color by averaging the components
               of all pixel values in a 3x3 area around the pixel.
               Assume that pixel colors are extended beyond the image
               borders.
             */
            unsigned blueSum  = 0;
            unsigned greenSum = 0;
            unsigned redSum   = 0;

            for (int ii = 0; ii < 3; ii++) {
                for (int jj = j; jj < j + 3; jj++) {
                    blueSum  += row[ii][jj].rgbtBlue;
                    greenSum += row[ii][jj].rgbtGreen;
                    redSum   += row[ii][jj].rgbtRed;
                }
            }
            image[i][j].rgbtBlue  = (BYTE)((blueSum + 4)  / 9);
            image[i][j].rgbtGreen = (BYTE)((greenSum + 4) / 9);
            image[i][j].rgbtRed   = (BYTE)((redSum + 4)   / 9);
        }
    }
}

Answer 2

您不能将标量类型与结构或数组进行比较。这就是错误消息告诉您的。

将多个无关字节的结构与数值进行比较应该是什么条件？
那不是它的工作方式。您只能分别比较字段或将整个结构与同一结构类型的另一个变量进行比较。

除此之外，您的代码中有一些误解。

如果该值不存在在内存中，则我会得到“分割故障”

您会得到一个分段故障，因为要读取的内存不是您的内存。您没有访问特权可以阅读或写作。

它将检查该值是否存在在内存中。

如果您为函数提供一些价值，它将始终“存在于内存中”。您已经在调用功能中提供了一个值。怎么不存在？
您无法通过查看复制值来检测复制的内存位置是否有效。

---

您的根本问题是，您没有在阅读值之前之前与数组的界限进行验证。
您必须将行和列索引与限制进行比较，并且仅在不超出界限时才访问数组元素。

您需要重新设计方法以收集像素以模糊。