[C#]異なるダイアログで部品を共有したいのですが

Question

メモ帳のようなアプリを作っているのですが、検索ダイアログと置換ダイアログで共通する部分があり、（検索文字列のテキストボックス、大文字小文字区別のためのチェックボックスなど）共有化したいと考えました。そこで、自作のダイアログクラスに静的な変数として各種のコントロールを定義して、インスタンス生成後、各自で必要な部品を追加するようにしました。
以下のコードですとコンパイルはうまくいくのですが、検索ダイアログに何も表示されず、置換ダイアログだけにコントロールが配置されてしまいます。staticを使うやり方が間違っているのでしょうか？解決策をご教示ください。

using System;
using System.Drawing;
using System.Windows.Forms;
namespace notepad{
class Program{
    [STAThread]
    static void Main(){
        Application.EnableVisualStyles();
        Application.Run(new MainForm());
    }
}
class MainForm : Form{
    MainMenu mainMenu;
    MenuItem find, replace;
	Form findDialog, replaceDialog;
	RichTextBox textBox;
    public MainForm(){
		this.Size = new Size(600, 400);
		mainMenu = new MainMenu();
	    find = new MenuItem(){Text="検索(&F)...", Shortcut  =  Shortcut.CtrlF,};
	    find.Click += (sender,e) => {findDialog.Show();};
		replace = new MenuItem(){Text="置換(&H)...", Shortcut  =  Shortcut.CtrlH,};
		replace.Click += (sender,e) => {replaceDialog.Show();};
	    textBox = new RichTextBox(){Dock = DockStyle.Fill,};
	    mainMenu.MenuItems.AddRange(new MenuItem[]{find,replace});
		findDialog = new EditDialog(){Text="検索"};
		findDialog.Controls.AddRange(new Control[]{
		EditDialog.targetLabel, EditDialog.targetTextBox,});
	    replaceDialog = new EditDialog(){Text="置換"};
		replaceDialog .Controls.AddRange(new Control[]{
		EditDialog.targetLabel, EditDialog.targetTextBox,});
	    this.Menu=mainMenu;
	    this.Controls.AddRange(new Control[]{textBox,});
    }
}
class EditDialog : Form{
  static public Label targetLabel;
  static public TextBox targetTextBox;
  static EditDialog(){
		EditDialog.targetLabel = new Label(){
		 Text = "検索する文字列(&N)",
	  	AutoSize = true,
	  	Location = new Point(20, 30),
		}；
		EditDialog.targetTextBox = new TextBox(){
			Location = new Point(150, 30),
			Size = new Size(100, 20),
		};
	}
	public EditDialog(){
		this.Size=new Size(400, 150);
		this.Closing+=(sender,e) => {this.Hide();e.Cancel=true;};
	}
}
}

Answer 1

最近似たようなことをやりましたが、共通部分の部品のみ持っているフォームを基底クラスとしてそれを継承して別々のフォームを作る、という方法をとりました。その際にリンク先の@ITの記事を参考にしました。こういう解決方法はどうでしょうか。

Comments

[negitronics]

staticをやめて継承を利用したらうまくいきました。確かにこちらのほうがシンプルですね。

Answer 2

こういう実装はしたことがないため憶測になってしまうのですが、インスタンスが共有化されるためにもう一方に貼り付けると元の場所から剥がれ落ちる、みたいなことかと。
あくまでインスタンスは一つなので。

staticは扱いが難しい気がしているので、個人的にあまり使わない方がいいと思っています。
使うときはSingletonパターンくらいですかね…。

staticメソッドは別です。割と使います。
拡張メソッドもstaticクラスのstaticメソッドですしね。

using System;
using System.Drawing;
using System.Windows.Forms;

namespace notepad
{
    class Program
    {
        [STAThread]
        static void Main()
        {
            Application.EnableVisualStyles();
            Application.Run(new MainForm());
        }
    }
    class MainForm : Form
    {
        MainMenu mainMenu;
        MenuItem find, replace;
        EditDialog findDialog, replaceDialog;
        RichTextBox textBox;

        public MainForm()
        {
            this.Size = new Size(600, 400);

            find = new MenuItem() { 
                Text = "検索(&F)...",
                Shortcut = Shortcut.CtrlF, 
            };
            find.Click += (sender, e) => findDialog.Show();

            replace = new MenuItem() { 
                Text = "置換(&H)...", 
                Shortcut = Shortcut.CtrlH,
            };
            replace.Click += (sender, e) => replaceDialog.Show();

            textBox = new RichTextBox() {
                Dock = DockStyle.Fill,
            };

            mainMenu = new MainMenu();
            mainMenu.MenuItems.AddRange(new MenuItem[] { find, replace });

            findDialog = new EditDialog(caption:"検索");
            replaceDialog = new EditDialog(caption:"置換");
            this.Menu = mainMenu;
            this.Controls.AddRange(new Control[] { textBox, });
        }
    }

    class EditDialog : Form
    {
        public Label targetLabel;
        public TextBox targetTextBox;

        public EditDialog(string caption)
        {
            this.targetLabel = new Label()
            {
                Text = "検索する文字列(&N)",
                AutoSize = true,
                Location = new Point(20, 30),
            };
            this.targetTextBox = new TextBox()
            {
                Location = new Point(150, 30),
                Size = new Size(100, 20),
            };

            this.Text = caption;
            this.Size = new Size(400, 150);
            this.Controls.AddRange(new Control[] { 
                targetLabel,
                targetTextBox,
            });
        }
    }
}

すみません。自分にとって見やすいように勝手にいろいろ書き換えました。
diffってください。

Comments

[negitronics]

インスタンスが剥がれ落ちる、という考えは思いつきませんでしたが、確かにそのようなことが起きているようです。ダイアログクラスをインスタンス化する順番を変えて、検索ダイアログを後からインスタンス化すると、検索ダイアログ側にコントロールが表示されるようになりました。今回はstaticをやめてみますが、個人的にこの挙動の根拠を調べたいと思います。ありがとうございました。