特定コンテナ？内のコントロールを全て削除したい

Question

c#でホワイトボードのような物を作成しました。
VisualBasicPowerPackのOvelShapeでクリックした部分に決まったサイズの円を描く事をしています。

そのOvelShapeを削除する（ホワイトボードをクリアするようなイメージ）作業があるのですが、
例えばpicturebox（pic1）にOvalShapeで円を描く場合、実際にはキャンバスとしてShapeContainerが置かれ、
その上？にOvalShape（円）が描かれる事は理解したので、下記のようなコードを書いてみました。

foreach (Control c in pic1.Controls)
{
  if (c.GetType().Name == "ShapeContainer")
  {
    c.Dispose();
  }
}

ShapeContainerが厳密にはコントロールでは無いためかRemoveがないのでDisposeで対応してます。
（ここら辺が原因でしょうか？？？）

pic1に5個のOvalShapeを置いてpic1.Controls.Countをしてみると5個コントロールがあることが取得できます。

しかし実際にコードを動かすと5個削除できずに3個できたりしています。
実際にはクリックで上記部分のコードが走るので何度か削除ボタンをクリックすると最終的には消えます。

なぜ上記コードで一回では消えないのでしょうか？？

参考になる事があればお願いします！！

Answer 1

次のコードでフォームに描画された OvalShape を削除できました。

            var canvas = (ShapeContainer)this.Controls["ShapeContainer"];

            // ↓InvalidOperationException が発生
            //foreach (Shape s in canvas.Shapes)
            //{
            //    canvas.Shapes.Remove(s);
            //}

            // 一旦 List に入れてから削除
            var shapes = new List<Shape>();
            foreach (Shape s in canvas.Shapes)
            {
                shapes.Add(s);
            }
            foreach(Shape s in shapes)
            {
                canvas.Shapes.Remove(s);
            }

Comments

[YAmaGNZ]

質問者さんのOvalShapeの追加コードがないので詳細が不明ですが、１つのShapeContainerにShapeを追加していく処理になっていないのではないかと思います。
なので、ControlsのループでShapeContainerを複数消すような処理になっていると想像しました。

Answer 2

Dispose しても即座に無くなるわけではありませんし、参照残っていれば消えません。
ShapeContainer は Form の components に入ってませんか？
そこからなら、Remove出来ると思います。

Answer 3

多分、foreachのループ内でControlsの中身が変化することが原因と思われます。

pic1.Controls[0] = ShapeContainer1
pic1.Controls[1] = ShapeContainer2
pic1.Controls[2] = ShapeContainer3

とあった時に
foreachの最初でpic1.Controls[0]が取得され、ShapeContainer1が消されます。
この時
pic1.Controls[0] = ShapeContainer2
pic1.Controls[1] = ShapeContainer3
とControlsが変化するのですが、次のループでpic1.Controls[1]が取得されShapeContainer3が消されることになります。

for (int i = pic1.Controls.Count - 1 ; i >=0 ; i--)
{
  if (pic1.Controls[i].GetType().Name == "ShapeContainer")
  {
    pic1.Controls[i].Dispose();
  }
}

と配列を逆から回して削除していってはどうでしょう？

Comments

[landy77]

あ！なるほど・・・
深く考えてませんでしたが回してる最中で対象が変化しちゃうんですね・・・・

コードを修正したところ想定通りに動作しました

ありがとうございました！