メインフォームからユーザコントロールを操作できない

前提・実現したいこと

C#でWindows上で動作するツールのプロト版を作成しようとしています。
メインフォームと複数のユーザコントロールを用いたツールで、
メインフォームに共通的なボタンを実装し、その結果を出力するための画面がユーザコントロールになります。
とっかかりとして、メインフォームのボタンクリックでユーザコントロールのテキストに文字列を表示させようとしていますが、テキストが表示されません。
ユーザコントロール上にテキストを表示させるメソッドはコールされていることを確認しており、Step実行することで、テキストボックスへ値が代入されていることも確認できています。
同様のボタンをユーザコントロール側に実装すると、期待通りテキストが表示されます。

発生している問題・エラーメッセージ

メインフォームのボタンクリックでユーザコントロールのテキストに文字列を表示させることができない。エラーの発生はなし。期待するメソッドも全てコールされている。

該当のソースコード

実際のツールのキャプチャを記載します。
Formのテキストは更新されますが、ユーザコントロール上のテキストは更新されません。（ｔｅｓｔｃメソッドはコールされている）

C#
1//■メインフォーム
2    public partial class Form1 : Form
3    {
4        //private static UserControl1 _uc1;
5
6        public Form1()
7        {
8            InitializeComponent();
9        }
10
11        public void button1_Click(object sender, EventArgs e)
12        {
13            //UserControl1.Instance.Show();
14
15            textBox1.Text = "abc";
16            UserControl1.Instance.testc();
17
18        }
19    }
20
21//■ユーザコントロール
22    public partial class UserControl1 : UserControl
23    {
24        private static UserControl1 _uc1;
25
26        public UserControl1()
27        {
28            InitializeComponent();
29        }
30
31
32        //ただ一つのフォームにアクセスするためのプロパティ
33        public static UserControl1 Instance
34        {
35            get
36            {
37                //_instanceがnullまたは破棄されているときは、
38                //新しくインスタンスを作成する
39                if (_uc1 == null || _uc1.IsDisposed)
40                {
41                    _uc1 = new UserControl1();
42                }
43                return _uc1;
44            }
45        }
46
47
48//メインフォームからこの関数を呼んでも、テキストが更新されない
49
50        public void testc()
51        {
52            textBox1.Text = "cde";
53        }
54
55//ユーザコントロール（自分）からこの関数を呼べば、テキストが更新される。
56        public void button1_Click(object sender, EventArgs e)
57        {
58            textBox1.Text = "def";
59        }
60
61
62    }
63       
64
65### 試したこと
66
67同様のボタンをユーザコントロール側に実装すると、期待通りテキストが表示されます。
68
69### 補足情報（FW/ツールのバージョンなど）
70
71ここにより詳細な情報を記載してください。

Zuishin

2020/05/07 08:09 編集

コンパイルエラーにならないのが不思議です。textBox1 が見えないからでしょう。コンパイルできて回答者の手元で再現できる最小のコードを新しく作成して掲載すれば解決すると思います。

testUser2022

2020/05/07 08:24

すみません、初の投稿でスレッドの編集方法がわからず、時間がかかってしまいました。コードを更新しました。

Zuishin

2020/05/07 08:44 編集

なぜシングルトンなのかわかりませんが、デザイナで作った時点では _uc1 が null だからですね。YamGNZ さんの意見の通り、シングルトンを使うのをやめればいいと思います。どうしてもこれで行くならコンストラクタで _uc1 に this を入れることでうまくいきそうではありますが、冗長かつ他のバグが入りそうという点でお勧めしません。

testUser2022

2020/05/07 09:07

ありがとうございます！試してみます！

testUser2022

2020/05/07 14:48

無事解決しました！ありがとうございました！

行動規範の内容に同意します

回答1件

ベストアンサー

Instanceを呼んだ時に新しいUserControl1が作成され、そのコントロールのtextBox1のTextが変更されています。

なぜstaticメソッドでインスタンスを管理しようとしているのか分かりませんが
デザイナでUserControl1を配置しているのではないですか？
そのインスタンス(仮にusercontrol1とします)で

C#
1usercontrol1.testc();

とするのではダメなのでしょうか？

追記

デザイナで配置したら

このような感じだと思います。
そして、下記のように配置した名前でアクセスできるはずです。

フォーム側

C#
1private void button1_Click(object sender, EventArgs e)
2{
3    userControl11.testc();
4}

UserControl1側

C#
1public partial class UserControl1 : UserControl
2{
3    public UserControl1()
4    {
5        InitializeComponent();
6    }
7
8    public void testc()
9    {
10        textBox1.Text = "cde";
11    }
12}

投稿2020/05/07 08:10

編集2020/05/07 09:51

YAmaGNZ

総合スコア10489

testUser2022

2020/05/07 08:27

C#の経験が浅く、申し訳ないのですが、、、 usercontrol1.testc(); にするとコンパイルが通りません。UserControlのインスタンスを取得するために必要と思い、Instanceのプロパティを実装しています。

YAmaGNZ

2020/05/07 08:29

UserControl1はデザイナで配置しているのですか？

testUser2022

2020/05/07 08:31

はい。デザイナで配置しております。

YAmaGNZ

2020/05/07 08:38

そうであれば、その配置したUserControl1の名前を使用してください。私が提示したものはあくまで「例」です。また、質問文に「複数のユーザコントロールを用いた」とありますが、それは現在作成しているUserControl1を複数使用するということですか？

testUser2022

2020/05/07 08:42

＞そうであれば、その配置したUserControl1の名前を使用してください。ご指摘ありがとうございます。承知しました。＞また、質問文に「複数のユーザコントロールを用いた」とありますが、それは現在作成しているUserControl1を複数使用するということですか？混乱を招く書き方で申し訳ありません。その通りです。現状はユーザコントロールは一つしかありません。

YAmaGNZ

2020/05/07 09:10

現在作成されているUserContorl1を複数使うとのことですが、そうなるとますますstaticメソッドでインスタンスを管理する必要がないように思えます。

testUser2022

2020/05/07 09:35

何度もすみません、うまくいっておりません。ユーザーコントロールのインスタンスをメインフォームへ伝える手段が理解できていません。すみませんが、どのような方式で実装するべきか、御指南いただけますでしょうか。

testUser2022

2020/05/07 09:38

ちなみにですが、staticにしていた理由は、ユーザーコントロールのインスタンスをメインフォームへ伝える手段を色々ググってヒットした論理を流用しておりました。staticである必要性は全くありません。

YAmaGNZ

2020/05/07 09:42

デザイナで配置しているのであれば、その名前ですでにインスタンスが作成されています。ですので、デザイナで配置した名前がuserControl1だったとしたらuserControl1.testc();でアクセスできるはずです。 staticメソッドのInstanceを呼ぶ必要はありません。

退会済みユーザー

2020/05/07 09:44

・使用するユーザーコントロールの数が確定している →普通にフォームデザイナで配置し、コントロール名（Name）でアクセスしてください・使用するユーザーコントロールの数が動的に変わる →ユーザーコントロールをnewで作成し、フォームにControls.Addで追加してください

testUser2022

2020/05/07 10:03

皆さん、ありがとうございます。完全な答えまで出していただき申し訳ないのですが、 userControl11.testc();の箇所でエラーになってしまっております。”静的でないフィールド、メソッド、またはプロパティuserControl11.testc();でオブジェクト参照が必要です”というエラーです。デザイナ配置時のミスかもしれません。一度作り直してみます。

testUser2022

2020/05/07 10:42

ソリューションから作り直したのですが、現象変わらずです。 .NET 4.5ですが、古いでしょうか。。もう少しプロパティ等を見直してみます。

testUser2022

2020/05/07 14:29

すみません、解決方法がまだわかりません。 radianさんの＞・使用するユーザーコントロールの数が確定している＞→普通にフォームデザイナで配置し、コントロール名（Name）でアクセスしてくださいの方式で実現したいのですが、コントロール名でアクセスしようとすると、 [オブジェクト参照が必要です]のエラーになってしまいます。

testUser2022

2020/05/07 14:31

何度もすみません、解決しましたm(__)m

testUser2022

2020/05/07 14:34

ユーザーコントロール単品のコントロール名（Nama）でアクセスしようとしていました。正しくは、”Form上に配置したユーザーコントロールのコントロール名（Name）”だったのですね。皆様、ご協力ありがとうございました！！

行動規範の内容に同意します