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

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

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

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

Windows Forms

Windows Forms(WinForms)はMicrosoft .NET フレームワークに含まれる視覚的なアプリケーションのプログラミングインターフェイス(API)です。WinFormsは管理されているコードの既存のWindowsのAPIをラップすることで元のMicrosoft Windowsのインターフェイスのエレメントにアクセスすることができます。

Q&A

解決済

4回答

2240閲覧

[C#][Windows Forms]コンボボックスの変更後の無駄なデータ取得をまとめたい

S.Oda

総合スコア13

C#

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

Windows Forms

Windows Forms(WinForms)はMicrosoft .NET フレームワークに含まれる視覚的なアプリケーションのプログラミングインターフェイス(API)です。WinFormsは管理されているコードの既存のWindowsのAPIをラップすることで元のMicrosoft Windowsのインターフェイスのエレメントにアクセスすることができます。

0グッド

0クリップ

投稿2019/04/22 05:30

前提・実現したいこと

初めて質問させていただきます。
質問に関して不備等ありましたら、申し訳ありません。

[環境: Windows10, Visual Studio 2017, .net framework 4.7.2]

現在、Windows Formでアプリケーションを作成しているのですが、
以下の処理は通常、どのように実装するべきなのか教えていただきたいです。

画面上、
学年の開始を指定するコンボボックス(cmbGradeFrom)
学年の終了を指定するコンボボックス(cmbGradeTo)、
クラスの開始を指定するコンボボックス(cmbClassFrom)
クラスの終了を指定するコンボボックス(cmbClassTo)
があります。

それぞれのコンボボックスは、選択した内容が変更されたタイミングで
イベントが[該当のソースコード]のように発生するようになっており、
setEachFeeList()内では、学年の開始-終了、クラスの開始-終了で絞り込みを行い
データ取得をして、データグリッドに対象データを表示しています。

それぞれのコンボボックスが変更したら、データ取得処理に入るようになっているのですが、
一つのコンボボックスの選択変更がほかのコンボボックスの変更に影響を与えているため、
一つのコンボボックスを変更しただけでも複数のイベントが発生してしまい、
複数回データ取得処理を行ってしまいます。

一つのコンボボックスを変更したら、ほかのコンボボックスの値変更をすべて終わったあとに
データ取得を1度だけ実行するような作りにしたいのですが、それは可能でしょうか。
以上、よろしくお願いいたします。

該当のソースコード

C#

1 2 /// 学年FROMが変更されたときの処理 3 protected void cmbGradeFrom_TextChanged(object sender, EventArgs e) 4 { 5 if (cmbGradeFrom.Items.Count == 0 || cmbGradeTo.Items.Count == 0) return; 6 7 //学年の開始が終了より大きい場合、終了と開始を一致させる 8 if (cmbGradeFrom.SelectedIndex > cmbGradeTo.SelectedIndex) 9 { 10 cmbGradeTo.SelectedIndex = cmbGradeFrom.SelectedIndex; 11 } 12 13 //学年の開始と終了が同じ場合、クラスの最小と最大をクラスの開始と終了に設定 14 if (cmbGradeTo.SelectedIndex == cmbGradeFrom.SelectedIndex) 15 { 16 setClassFromTo(); 17 } 18 else 19 { 20 setEachFeeList(); //画面表示データ取得&表示処理 21 } 22 } 23 24 /// 学年TOが変更されたときの処理 25 protected void cmbGradeTo_TextChanged(object sender, EventArgs e) 26 { 27 if (cmbGradeFrom.Items.Count == 0 || cmbGradeTo.Items.Count == 0) 28 return; 29 30 //学年の開始が終了より大きい場合、開始を終了に一致させる 31 if (cmbGradeFrom.SelectedIndex > cmbGradeTo.SelectedIndex) 32 { 33 cmbGradeFrom.SelectedIndex = cmbGradeTo.SelectedIndex; 34 } 35 36 //学年の開始と終了が同じ場合、クラスの最小と最大をクラスの開始と終了に設定 37 if (cmbGradeFrom.SelectedIndex == cmbGradeTo.SelectedIndex) 38 setClassFromTo(); 39 } 40 else 41 { 42 setEachFeeList(); //画面表示データ取得&表示処理 43 } 44 } 45 /// クラスFromが変更されたときの処理 46 protected void cmbClassFrom_TextChanged(object sender, EventArgs e) 47 { 48 if (cmbClassFrom.Items.Count == 0 || cmbClassTo.Items.Count == 0) return; 49 //クラスの開始が終了より大きい場合、終了を開始に一致させる 50 if (cmbClassFrom.SelectedIndex > cmbClassTo.SelectedIndex) 51 { 52 cmbClassTo.SelectedIndex = cmbClassFrom.SelectedIndex; 53 } 54 setEachFeeList(); 55 } 56 /// クラスToが変更されたときの処理 57 protected void cmbClassTo_TextChanged(object sender, EventArgs e) 58 { 59 if (cmbClassFrom.Items.Count == 0 || cmbClassTo.Items.Count == 0) return; 60 //クラスの開始が終了より大きい場合、開始を終了に一致させる 61 if (cmbClassFrom.SelectedIndex > cmbClassTo.SelectedIndex) 62 { 63 cmbClassFrom.SelectedIndex = cmbClassTo.SelectedIndex; 64 } 65 setEachFeeList(); 66 } 67

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

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

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

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

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

guest

回答4

0

ベストアンサー

TextChangedじゃなくてDropDownClosedを使うという手もありますが、
ドロップダウンを開いて閉じないとイベントが発生しない問題があります。
(キー操作で変更した場合に発生しない)

他にも、 Validatedイベントなんかも使えそうです。

投稿2019/04/22 08:39

k.matsuda

総合スコア293

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

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

S.Oda

2019/04/22 10:54

そうですね、確かに他のeventで試してませんでした。 確認してみます。ありがとうございます。
S.Oda

2019/04/23 04:38

結局DropDownClosedで実装することにしました。 ご回答いただきありがとうございました。
guest

0

各イベントハンドラでタイマー起動
タイマーでデータ取得処理を行いタイマー停止
というのはどうでしょう?

下記のような感じです。

C#

1 private void comboBox1_TextChanged(object sender, EventArgs e) 2 { 3 Console.WriteLine("comboBox1"); 4 comboBox2.Text += "1"; 5 timer1.Start(); 6 } 7 8 private void comboBox2_TextChanged(object sender, EventArgs e) 9 { 10 Console.WriteLine("comboBox2"); 11 timer1.Start(); 12 } 13 14 private void timer1_Tick(object sender, EventArgs e) 15 { 16 Console.WriteLine("SetData"); 17 timer1.Stop(); 18 } 19

投稿2019/04/22 06:30

編集2019/04/22 07:18
YAmaGNZ

総合スコア10222

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

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

S.Oda

2019/04/22 07:08

ちょっとイメージがわかないので調べてみます。 ありがとうございました。
S.Oda

2019/04/22 10:55

おー、わざわざサンプルコード記入していただきありがとうございます。 確認します。
guest

0

イベントを一時的に外してしまうのはどうでしょうか?

C#

1 /// 学年FROMが変更されたときの処理 2 protected void cmbGradeFrom_TextChanged(object sender, EventArgs e) 3 { 4 if (cmbGradeFrom.Items.Count == 0 || cmbGradeTo.Items.Count == 0) return; 5 6 //学年の開始が終了より大きい場合、終了と開始を一致させる 7 if (cmbGradeFrom.SelectedIndex > cmbGradeTo.SelectedIndex) 8 { 9 // イベントが呼ばれないように一時的に外す 10 cmbGradeTo.TextChanged -= cmbGradeTo_TextChanged; 11 12 cmbGradeTo.SelectedIndex = cmbGradeFrom.SelectedIndex; 13 14 // イベントを元に戻す 15 cmbGradeTo.TextChanged += cmbGradeTo_TextChanged; 16 } 17 18 //学年の開始と終了が同じ場合、クラスの最小と最大をクラスの開始と終了に設定 19 if (cmbGradeTo.SelectedIndex == cmbGradeFrom.SelectedIndex) 20 { 21 setClassFromTo(); 22 } 23 else 24 { 25 setEachFeeList(); //画面表示データ取得&表示処理 26 } 27 }

投稿2019/04/22 05:55

nskydiving

総合スコア6500

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

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

S.Oda

2019/04/22 06:22

イベントを一時的に外すほうが、おそらくフラグ管理よりはいいような気がします。(イベントを実行したくないという理由が一目でわかるため) ただ、これも処理が込み入りそうなので、ほかにいい方法はないかなーと考えていたところです。 助言いただきありがとうございます。
guest

0

データ取得処理

を行うべきか否かをフラグか何かで制御すればよいのではないでしょうか.
(各コンボボックスのイベントハンドラはフラグが立っていたら即returnする,的な)

投稿2019/04/22 05:52

fana

総合スコア11632

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

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

S.Oda

2019/04/22 06:20

そうですね、それは私も思いましたが、処理が煩雑になりそうでしたので、実装をためらっていました。 他にいい方法がなければそうしようと思います。 ありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問