フォーム上の背景に複数の図形の描画

Question

フォーム上に複数の四角を動的に配置するために、ボタン入力されるたびに増減、変形、移動を行いたいのですが
私の書いたコードでは初期化の位置が悪いようで毎回一つの枠しか表示されません。

数値入力のボタンや、枠のon/offの切り替えが行われるたびにDrawFrameを呼んで、
BitmapとGraphicsを毎回初期化しているのが原因だと思うのですが、
どのように変更すればよいでしょうか？

外に出すと参照云々でエラーが出てしまい、flagで一回だけ行うなど考えてみたのですが、
ほかに何かありませんでしょうか？

        /////描画//////
        private void DrawFrame(int TP, decimal x, decimal y, decimal w, decimal h)
        {
            int X = Decimal.ToInt32(x);
            int Y = Decimal.ToInt32(y);
            int W = Decimal.ToInt32(w);
            int H = Decimal.ToInt32(h);

            Bitmap FrameCanvas = new Bitmap(PictureBox1.Width, PictureBox1.Height);
            Graphics Frame = Graphics.FromImage(FrameCanvas);                      

            //////枠の作成//////
            Pen PenState = new Pen(Color.FromArgb(TP, R, G, B), 2);
            Frame.DrawRectangle(PenState, X, Y, W, H);
            PictureBox1.Image = FrameCanvas;

            Frame.Dispose();
            PenState.Dispose();
        }

Answer 1

複数の四角

の座標を保持しておき，描画時にはその全てを描画すれば良いのではないでしょうか．

Comments

[asam1005]

いつもありがとうございます！
全座標を描画する発想はありませんでした！

ですが、数が多くなると少し処理が重くなってしまったので
もう少しほかのやり方を探してみます。。。

[fana]

Frame.DrawRectangle(PenState, X, Y, W, H);
の部分を四角形の個数分ループさせるだけでそんなに重くなりますか？
（四角形の個数がとんでもなく多い？）

[asam1005]

１０こ２０こだと全く気にならなかったのですが、10000こでやってみたら少し重くなってしまって.....

今は背景を最初に一回読み込んでただ枠を書いているだけなのでこれでも問題ないのですが、
カメラで撮影している画面を背景に枠を入れ込んだ場合に大丈夫なのかがいまいちわからなくて......

と思ってたんですが杞憂だったみたいなので、全描画でやらせていただきます！！
本当にいつもありがとうございます！

Answer 2

毎回新しい画像を作っては破棄しているからだと思います。

            Bitmap FrameCanvas = new Bitmap(PictureBox1.Width, PictureBox1.Height);

この部分を次のように変えてみてください。

            Bitmap FrameCanvas = pictureBox1.Image as Bitmap ??
                new Bitmap(PictureBox1.Width, PictureBox1.Height);

Comments

[asam1005]

全部表示されるようにはなりましたが、移動や変形した際に前の枠も残ってしまって、画面が四角だらけになってしましました......

ちょっとas演算子についても勉強してみます。