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パラメータでした! 助けてくれたみんなありがとう

4

1 に答える 1

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;
于 2013-03-15T12:57:20.277 に答える