如何更正此 LockBits 数学错误？

Question

使用 GetPixel/SetPixel，我使用以下内容为强调红色和紫色的图像过滤器提供动力：

for (int y = 0; y < bmpMain.Height; y++)
    for (int x = 0; x < bmpMain.Width; x++)
    {
        bmpMain.GetPixel(x, y);
        Color c = bmpMain.GetPixel(x, y);
        int myRed = c.R, myGreen = c.G, myBlue = c.B;
        myGreen -= 128;
        if (myGreen < 0) myGreen = 0;
        bmpMain.SetPixel(x, y, Color.FromArgb(255, myRed, myGreen, myBlue));
    }

使用 LockBits，我将其替换为以下内容：

for (int counter = 1; counter < rgbValues.Length; counter += 3)
{
    rgbValues[counter] -= 128;
    if (rgbValues[counter] < 0) rgbValues[counter] = 0;  
}

但不是绿色像素值减去 128，而是将 128 添加到绿色值。

如果我这样做：

for (int counter = 1; counter < rgbValues.Length; counter += 3)
{
    rgbValues[counter] += 128;
    if (rgbValues[counter] < 0) rgbValues[counter] = 0;  
}

128也加到绿色值上。生成的图像与我减去 128 的图像相同。

那么，我如何获得应该是简单的数学来在 LockBits 中正常工作？

Answer 1

假设rgbValues是字节数组，语句

rgbValues[counter] -= 128;

等同于

rgbValues[counter] = (byte)(rgbValues[counter] - 128);

因此，如果 rgbValues[counter] 等于零，它将设置为 (byte)(-128)。问题是，与 int 不同，byte 数据类型是 unsigned 并且不能表示负值。正如 EBrown 指出的那样，减法溢出并返回到 128。

修复代码的一种方法是引入类型为 int 的中间变量，以便您可以安全地容纳负值：

int myGreen = rgbValues[counter];
myGreen -= 128;
if (myGreen < 0) myGreen = 0;  
rgbValues[counter] = (byte)myGreen;

另一种方法是重写代码并首先避免负值：

rgbValues[counter] = rgbValues[counter] > 128
    ? (byte)(rgbValues[counter] - 128)
    : (byte)0;

如何更正此 LockBits 数学错误？

How do I correct this LockBits math error?

c#

rgb

image-processing

lockbits

getpixel