2013-03-15 71 views
2

当我试着翻动,在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

介意告诉我们什么是错在先?请解释你不期待发生的事情。错误和例外,如果你有任何问题。 – Oded 2013-03-15 12:33:33

+0

RotateFlip有什么问题?这对你来说太慢了吗?你是否在循环中重复运行这些代码? – 2013-03-15 12:35:55

+0

不,目的不是翻转,我想知道在C#中的指针和不安全的代码更好,所以我试图翻转一个位图!并且我将来需要用于图像处理的指针:) – Mehran 2013-03-15 12:39:32

回答

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; 
+0

我一开始认为,但我认为这就是'p + = offset'的目的所在。也许他需要检查'offset'不是零。 – 2013-03-15 12:58:43

+0

那么“p + = offset”呢?它不是在改变y方向吗? – Mehran 2013-03-15 13:00:36

+1

@Dotrix是的,你的'抵消'是错误的。通过减去图像的宽度乘以每像素的字节数,可以有效地将'offset'减少到0或接近0. – Virtlink 2013-03-15 13:02:27