C# と VB.NET の質問掲示板

開発環境
VS2005 C#

はじめまして。C#初心者です。

Windowsアプリケーションで画像編集ソフトを作っています。
画素にBitmapのGetPixel()を使ってアクセスしているのですが、
パソコンにパンチを入れたくなるくらい遅いのです。

なので、Imageクラスを派生して、Bitmap.LockBitsを使用してBitmapオブジェクト的なものを
実装したいのです。

そこで、Bitmapのメタデータと、クラスの継承が載っているサイトを参考に
public sealed class ExBitmap : Image
{
　　public ExBitmap(Image original) : base (){}
    //...以下、実装予定。
}

以上のようにコードを打ってみたのですが、
「型 'System.Drawing.Image' のコンストラクタが定義されていません。」
と、エラーが出てしまいます。

この場合、どのように実装すればいいのでしょうか？
お願いいたします。

>■No7290 (Hongliang さん) に返信
> Image クラスは、ビットマップイメージとベクタイメージを統一的に扱うためのクラスです。
> ですので、当然ながら LockBits メソッドを持っていません。ビットマップイメージじゃないと意味がないメソッドですからね。
> ですから、Image から直接派生して LockBits で操作するクラスってのは、設計がおかしいと言うことになります。
> 
> でもそういうのは派生させるよりユーティリティクラス的に外部から操作した方がいいと思うなぁ。
回答ありがとうございます。
実は、ttp://junki.lix.jp/csgr/005ColorDataAccess3.htmのサイトを参考に
/// <summary>高速ビットマップ機能(暫定)</summary>
public class ExBitmap : IDisposable
{
        private bool _flagDispose = false;
        private Bitmap _Bitmap;
        private BitmapData _BitmapData;
        private IntPtr _Pointer;
        private int _Stride;
        private int _DataSize;
        private byte[] _ColorData;

        public Bitmap GetBitmap() { return _Bitmap; }
        public ExBitmap(Bitmap bitmap) { _Bitmap = bitmap; Main(); }
        public ExBitmap(int width, int height) { _Bitmap = new Bitmap(width, height); Main(); }
        // TODO: これだと、Aチャンネルにアクセスできないらしい...? public ExBitmap(Image image) { _Bitmap =new Bitmap(image); }
        private void Main()
        {
                _BitmapData = _Bitmap.LockBits(new Rectangle(0, 0, _Bitmap.Width, _Bitmap.Height), ImageLockMode.ReadWrite, PixelFormat.Format32bppArgb);
                _Pointer = _BitmapData.Scan0;
                _Stride = _BitmapData.Stride;
                _DataSize = _Stride * _Bitmap.Height;
                _ColorData = new byte[_DataSize];
                // ARGBデータのメモリコピーを可能にします。
                System.Runtime.InteropServices.Marshal.Copy(_Pointer, _ColorData, 0, _DataSize);
        }

        public Color GetPixel(int x, int y)
        {
                int ix = (_Stride * y) + (x * 4);
                byte A = _ColorData[ix + 3];
                byte R = _ColorData[ix + 2];
                byte G = _ColorData[ix + 1];
                byte B = _ColorData[ix + 0];
                return Color.FromArgb(A, R, G, B);
        }

        public void SetPixel(int x, int y, Color col)
        {
                int ix = (_Stride * y) + (x * 4);
                _ColorData[ix + 3] = col.A;
                _ColorData[ix + 2] = col.R;
                _ColorData[ix + 1] = col.G;
                _ColorData[ix + 0] = col.B;
        }

        protected virtual void Dispose(bool flag)
        {
                if (!_flagDispose)
                {
                        if (flag)
                        {
                                // ARGBデータの転送を行います。
                                System.Runtime.InteropServices.Marshal.Copy(_ColorData, 0, _Pointer, _DataSize);
                                _Bitmap.UnlockBits(_BitmapData);
                        }
                        this._flagDispose = true;
                }
        }

        public void Dispose()
        {
                Dispose(true);
                GC.SuppressFinalize(this);
        }

        ~ExBitmap()
        {
                Dispose(false);
        }
}
を実装してみました。
これで、かなりの速度で画像データにアクセスできます。
しかし、SetPixel()でアクセスした画像を得るには、
GetBitmap()を通さないと画像データが受け取れないので、
Imageクラスから派生したBitmapオブジェクト的な感じの
実装をしたいと思い質問させていただきました。

上記のような実装ってできないのでしょうか？

Hongliang さん、回答ありがとうございます。

確かに、実装してて、何か不安になってくるクラスです。

記事を参考に実装↓
bool throwcheck;

public void GetBitmap()
{
    System.Runtime.InteropServices.Marshal.Copy(_ColorData, 0, _Pointer, _DataSize);
    _Bitmap.UnlockBits(_BitmapData);
    throwcheck = true;
    return _Bitmap; 
}
public void SetPixel(...)
{
    if (throwcheck) throw new Exception("データにアクセスできないよ");
    ...
}
こういうことでしょうか？
なんか違う感がある。。。


それと、質問の仕方がおかしかったです。
やりたかったこと(?)は、
Bitmapクラスの戻り値はBitmapです。(戻り値って言わないかも)
で、ExBitmapクラスの戻り値をBitmapにしたいのです。
/// <summary>色調反転</summary>
/// <param name="bmp">ソース</param>
/// <returns></returns>
public static Image ColorReverse(Bitmap bmp)
{
                                
        Color col;
        using (ImageAbility.ExBitmap exbmp1 = new ImageAbility.ExBitmap(bmp.Width, bmp.Height))
        using (ImageAbility.ExBitmap exbmp2 = new ImageAbility.ExBitmap(bmp))
        {
                for (int i = 0; i < bmp.Width; i++)
                {
                        for (int j = 0; j < bmp.Height; j++)
                        {
                                col = exbmp2.GetPixel(i, j);
                                exbmp1.SetPixel(i, j, Color.FromArgb(255 - col.R, 255 - col.G, 255 - col.B));
                        }
                }
                return exbmp1.GetBitmap();
                // ↑exbmp1.GetBitmap(); で画像データを得ているわけですが、
                // 通常、Bitmapで操作した場合、Bitmap bmp = new Bitmap(ほにゃらら);
                // return bmp;で画像を得られます。
                // これと同じように、return exbmp1; でBitmapが返ってくるようにしたいのです。
                // 自分の経験ではできなさそうですが、どうでしょうか？
        }
 }

#日本語ﾑﾂｶｼｲ...

渋木宏明(ひどり) さん、回答ありがとうございます。

ソースコード、ありがとうございます。
参考になりましたm(_ _)m。

> 「そう書ける」ことが、そんなに重要かなー？
やっぱりBitmapクラスに極力、互換を持たせないと
Bitmapから始めた自分には、ちょっとつらいかなぁって...
今の自分の課題としては、「そう書ける」ことがかなり重要です。

そこで、ググってたら「インデクサでGet,Set」なるものを発見しましたが、
this[] の[]の部分をのぞいた感じ？にすればできるのではと、以下
public Bitmap this()
{
     get
     {
            System.Runtime.InteropServices.Marshal.Copy(_ColorData, 0, _Pointer, _DataSize);
            _Bitmap.UnlockBits(_BitmapData);
            return _Bitmap;
     }
}
エラーでしたダメでした...

> てことから、この場合は「Bitmap.Lock() を一時的にロックしてピクセル操作をさせるためのヘルパクラス」を作った方が楽なんじゃないかなーと思います。
> 
> 今ちょっと暇なんで、この後ご飯食べて寝てしまわなかったらサンプルくらい書くかも (^^;

こんな感じ。

http://hidori.jp/downloads/junktest/TestBitmapBits-20070904.zip

使い方は

using (BitmapBits bits = new BitmapBits(this.bitmap))
{
    for (int y = 0; y < this.bitmap.Height; y++)
    {
        for (int x = 0; x < this.bitmap.Width; x++)
        {
            bits.SetPixel(x, y, color);
        }
    }
}

とか。