c#:2つのUserControl間でデータをやり取りしたい

Question

c#で2つのUserControl間でデータをやり取りしたいのですがうまくいきません。
具体的にはUserControl3でテキストボックスにデータを入力した後、UserControl3のボタンを押すとUserControl4が表示されUserControl3は非表示にします。この時にUserControl3で入力したデータをUserControl4のラベルに表示させたいのです。

試してみたこと:
以下のようなプログラムを試してみましたがラベルには何も表示されていません。
一応,下図のように初期値としてlabel2とlabel3があるのですがそれが消えてしまうということは。実際に代入されているのが空の文字列になっているのだと思います。

//Form1.cs(一部)
ctr1 = new UserControl1();
ctr2 = new UserControl2();
ctr3 = new UserControl3();
ctr4 = new UserControl4();

//UserControl3.cs(一部)
private void button1_Click(object sender, EventArgs e)
        {
            Form1.ctr4.Visible = true;
            Form1.ctr3.Visible = false;
            Form1.ctr4.label2.Text = textBox1.Text;
            Form1.ctr4.label3.Text = textBox2.Text;
        }

UerControl4.cs(デザイン)

Answer 1

まず、各ユーザーコントロールが他のユーザーコントロールにアクセスする設計がよろしくないかと思います。
各コントロールは何かして欲しいタイミングでイベントを起こし、そのユーザーコントロールが使用されているフォームにてイベントを受け取り、他のコントロールへアクションを起こすという形がいいのではないかと思います。

usercontrol3

public event EventHandler OKClick;
public string InputString1
{
    get
    {
        return textBox1.Text;
    }
}
public string InputString2
{
    get
    {
        return textBox2.Text;
    }
}

private void button1_Click(object sender, EventArgs e)
{
    if (OKClick != null)
    {
        OKClick(this, e);
    }
}

usercontrol4

public string Message1
{
    set
    {
        label2.Text = value;
    }
}
public string Message2
{
    set
    {
        label3.Text = value;
    }
}

Form1

private void usercontrol3_OKClick(object sender, EventArgs e)
{
    ctr3.Visible = false;
    ctr4.Visible = true;
    ctr4.Message1 = ctr3.InputString1;
    ctr4.Message2 = ctr3.InputString2;

}

みたいな感じで

Comments

[dodome_iro]

ご回答ありがとうございます。しかし、全くの初心者なものでうまくいきません。差し支えなければもう少しお付き合い願いませんか？m(-_-)m
うまくいかない具体的内容としては、前回教えていただいた修正を加えたのですがUserControl4に遷移しなくなりました。調べてみたところOKCick == null となっているようでOKClick(this, e);が機能していません。また、button1_Click自身が機能していることは確認済みです。
PS.あまりにも初歩的なことで躓きすぎているので無視していただいても一向にかまいません

[YAmaGNZ]

UserControl3のイベントのイベントハンドラを登録していないのでしょう。
FormのLoadイベント時などに
ctr3.OKClick += new EventHandler(usercontrol3_OKClick);
という感じでイベントハンドラの追加を行ってください。

Answer 2

現在起きている問題については今掲載されているソースからは判断できない気がします。
とりあえずそれはどうでもいいです（え？いいの？）。

ともかく「ある UserControl に同 Form 上にある別の UserControl を操作するイベントを直接実装する」というやり方は一般的なセオリーから外れた実装のように思います。
（そんな密結合なものを個別の UserControl として実装する意味はなんでしょうか？）
私には未来のソースコードに私には見えます。人類にはちょっと早すぎる可能性があります（言い過ぎー！）。

一般的には Form 内の要素間の作用は Form の実装として解決するでしょう。
やり方はいくつかありますが一例を挙げるならサンプルの button1_Click のイベントは UserControl ではなく Form に書くというのが一般的なように思います。
そうすると自然と問題も解決しやすくなるような気がします（たぶん）。

Comments

[toydev]

これ書いている間に既にほかの回答がついてましたねー。YAmaGNZ さんと言っていることは一緒です！やっぱりそう感じますよね。よかった（ﾎｯ）。