C#中的翻转位图函数

Question

作为练习，我尝试使用 C# 中的不安全代码和指针翻转位图 但是问题是我得到的是原始位图而不是翻转的位图，似乎该函数什么也没做，所以我想知道我的代码有什么问题！

请记住我要翻转pictureBox1.Image并将其设置为pictureBox2.Image

    private bool Flip_H()
    {
        try
        {
            b = new Bitmap(pictureBox1.Image);
            bmdata = b.LockBits(new Rectangle(0, 0, b.Width, b.Height),ImageLockMode.ReadWrite, PixelFormat.Format24bppRgb);
            int offset = bmdata.Stride - b.Width * 3;
            byte back_up;
            int BGRwidth = b.Width * 3;
            unsafe
            {
                byte* p = (byte*)bmdata.Scan0;
                for (int y = 0; y < b.Height; y++)
                {
                    for (int x = 0; x < BGRwidth / 2; x += 3)
                    {
                        back_up = p[x];
                        p[x] = p[BGRwidth - x - 3];
                        p[BGRwidth - x - 1] = back_up;
                        back_up = p[x + 1];
                        p[x + 1] = p[BGRwidth - x - 2];
                        p[BGRwidth - x - 2] = back_up;
                        back_up = p[x + 2];
                        p[x + 2] = p[BGRwidth - x - 1];
                    }
                    p += offset;
                }
            }
            b.UnlockBits(bmdata);
            pictureBox2.Image = b;
            return true;
        }
        catch
        {
            return false;
        }
    }

我已经用 GetPixel() 和 SetPixel() 函数完成了这项工作，但是你知道它们太慢了，所以我正在尝试用指针改进我的代码！

问题是Offset 参数！感谢所有帮助过的人

Answer 1

您的问题是您从未在 Y 方向前进。您始终只更改第一条扫描线上的像素，因此其余的不会更改。发生这种情况是因为您执行 p += offset 但您将 offset 定义为：

int offset = bmdata.Stride - b.Width * 3;

您应该将偏移量定义为：

 int offset = bmdata.Stride;

因为您想添加步幅宽度（以字节为单位）以到达下一个扫描线。

---

您的交换代码中也存在错误。你有：

back_up = p[x + 0];
p[x + 0] = p[BGRwidth - x - 3];
p[BGRwidth - x - 1] = back_up;  // Error!

back_up = p[x + 1];
p[x + 1] = p[BGRwidth - x - 2];
p[BGRwidth - x - 2] = back_up;

back_up = p[x + 2];
p[x + 2] = p[BGRwidth - x - 1];
    // Missing!

你应该有：

back_up = p[x + 0];
p[x + 0] = p[BGRwidth - x - 3];
p[BGRwidth - x - 3] = back_up;

back_up = p[x + 1];
p[x + 1] = p[BGRwidth - x - 2];
p[BGRwidth - x - 2] = back_up;

back_up = p[x + 2];
p[x + 2] = p[BGRwidth - x - 1];
p[BGRwidth - x - 1] = back_up;