質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.51%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Q&A

解決済

1回答

4843閲覧

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

QUser

総合スコア1

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

0グッド

0クリップ

投稿2020/05/07 08:01

編集2020/05/07 08:35

前提・実現したいこと

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

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

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

該当のソースコード

実際のツールのキャプチャを記載します。
Formのテキストは更新されますが、ユーザコントロール上のテキストは更新されません。(testcメソッドはコールされている)
イメージ説明

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ここにより詳細な情報を記載してください。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Zuishin

2020/05/07 08:09 編集

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

2020/05/07 08:24

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

2020/05/07 08:44 編集

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

2020/05/07 09:07

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

2020/05/07 14:48

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

回答1

0

ベストアンサー

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

総合スコア10222

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

QUser

2020/05/07 08:27

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

2020/05/07 08:29

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

2020/05/07 08:31

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

2020/05/07 08:38

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

2020/05/07 08:42

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

2020/05/07 09:10

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

2020/05/07 09:35

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

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で追加してください
QUser

2020/05/07 10:03

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

2020/05/07 10:42

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

2020/05/07 14:29

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

2020/05/07 14:31

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

2020/05/07 14:34

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.51%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問