作为一种练习我尝试在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
参数!感谢所有帮助过的人
答案 0 :(得分:4)
你的问题是你从未在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;