C#　btn.enable=falseに設定した時にクリックする発生イベントについて

実現したいこと

超初心者の質問で申し訳ありません
ボタン操作（コントロール）不能状態（Enable=false）に設定した時にイベントを発生しないようにしたい
＊そもそも、ボタン操作（コントロール）不能状態（Enable=false）にしても見た目上はクリックできないだけでボタン上をクリックするとイベントは発生するのでは？

ご多忙とは思いますがこの現象の対応策をご存じの方
ご教示願います

開発中システムついて

FormA(設定画面)にて設定した時刻が時間になると自動的に処理Xがバックグラウンドワーカーを使用し実行し実行中はFormB(処理経過画面)を「ProgressChanged」で表示させ、Xが終了するとFormBが閉じるシステムを開発しています。

「処理実行中は処理経過中(タイマー処理経過バーを表示)であるFormBだけでなく処理設定FormAも画面上に表示を
残して欲しい」とユーザーの希望があります

発生している問題

FormB表示中(処理X実行中)にenable=falseで設定しているFormAのボタンをクリックすると
（実際はfalse設定にしているのでクリックできないが画面上に設置しているボタンをクリックする）
処理Xの実行が終了しFormBが閉じられます

FormBが閉じられてからFormAを使用する為にボタン制御を使用可能状態（Enable=true）に設定していますが
そのロジックを実行した瞬間に先ほど使用不能状態(Enable=false)時にクリックしたイベントが発生してしまいます

１：「FromA」（制御Form）始業時から終業時まで起動が基本
＊処理Xを実行させる時間設定画面

２：設定時間に到達した場合に「処理X」を起動
＊起動前にFormAにある制御ボタンはボタン操作（コントロール）不能状態（Enable=false）に設定してから
処理Xを起動（バックグラウンドワーカーを使用）

３：処理X起動中はFormBも表示される
＊PC画面上にFormAとFormBが表示されている

４：処理X起動中（FormB表示中）にFromA上にあるボタンをクリック
＊Enable=falseに設定しているので実際にはクリック操作はできないが
ボタン上をクリックする

５：処理Xが終了しボタン操作（コントロール）可能状態（Enable=true）に設定変更

６：ボタン操作（コントロール）可能状態（Enable=true）に設定変更した後FormBをclose

７：上記６を実行した瞬間（FormBをclose）に４で操作したイベントが発生

試したこと

1：ユーザーに処理X実行中はFormAを非表示にさせて欲しいと依頼
→「FormBを誤って閉じてしまった時にわからなくなるのでダメ」と速攻で却下

2：上記操作5と6を入れ替える
＊FormBをclseしてからコントロールフラグ変更
→結果は同じ

３：コントロールのイベントを一時的に無効にする
→操作不能状態には遷移しますが結果は同じ

＜FORMA＞

設定時刻になると下記別UI（UI-Y)に実装されている処理Xが起動させ別UI（UI-Y)上のFormBにが表示されます
この時点でFromAのボタンはenabel=false状態になっています

FormB

FormBが表示中（処理X実行中）にFormBの「終了ボタン」をクリックします
＊ボタンはコントロール無効状態に遷移しています

処理Xが終了しFormBがCloseすると
ボタンをクリックした下記メッセージBOXが表示されます

不明瞭な解説で申し訳ありませんが
お手数ですがご指摘願います

以上よろしくお願いいたします

補足情報

開発環境
Win10Pro（64）
VisualStudio2022にて開発

YAmaGNZ

2023/10/16 02:21

最初に表示しているFormAとボタンのEnable=falseを実行しているFormAのインスタンスが異なっていたりしませんか？ Enable=falseをどこでどのように行っているかをコードで示してください。

toshi_wan

2023/10/16 02:38

YAmaGNZサマご質問ありがとうございます。コードです。読みにくいコーディングとは思いますがよろしくお願いします。 //自動バックアップ処理 int return_cd = START_BACKUP(USB_EXE_PATH); //自動バックアップ処理START public int START_BACKUP(string FileName) { 　　　　//タイマー処理停止　　　　SET_TIMER1.Stop(); SET_TIMER2.Stop(); SET_TIMER3.Stop(); LBL_TIMER.Stop(); //コントロール無効　　　　BTN_END.Enabled = false; //存在ドライブを全て取得 string[] drives = Directory.GetLogicalDrives(); //実行EXE存在チェック。存在した場合実行 foreach (string drive in drives) { //取得したドライブ+EXEファイル名で検索　　 string serch_name = drive + FileName; //起動EXE名 if (File.Exists(@serch_name)) { Process P = Process.Start(@serch_name, "B");　　//ここで処理Xを起動　　　　　　//バックアップ処理終了まで処理を待つ P.WaitForExit(); 　　　　　　//コントロール有効 BTN_END.Enabled = true; } }

y_waiwai

2023/10/16 02:49

コードはこんなところに書かないで本文の方に追記しましょう

YAmaGNZ

2023/10/16 02:50

この処理はFormAで行われているのですか？また、この処理はUIスレッドで行われているのですか？

tamoto

2023/10/16 03:00

もしかして、ボタンを押してから、FormB 表示 -> 処理 X -> ボタン再有効化まで全部同期処理でやってたりしますかね。その場合、処理中に行われたクリックは全ての処理が完了するまで後回しにされるので、ボタンが再有効化された後にクリックした扱いになってたりするかもしれないですね。

TN8001

2023/10/16 09:19

> 実行中はFormB(処理経過画面)を「ProgressChanged」で表示させ、Xが終了するとFormBが閉じるこのような進捗ダイアログは、ShowDialog（モーダル表示）することが多いと思うのですが、できない都合があるんでしょうか？モーダルならそもそもFromAをさわれないので、「ボタン制御」する必要もありません。

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

回答5件

ベストアンサー

下にサンプルコードを書きます。
いまは、button1 の処理になっているわけで、button2, button3 のようにすれば解消します。

csharp
1private void button1_Click(object sender, EventArgs e) {
2    Button button = (Button)sender;
3    button.Enabled = false;
4    Execute();
5    button.Enabled = true;
6}
7
8private void button2_Click(object sender, EventArgs e) {
9    Button button = (Button)sender;
10    button.Enabled = false;
11    Execute();
12    // true の直前に Application.DoEvents()
13    Application.DoEvents(); 
14    button.Enabled = true;
15}
16
17//       ↓ async を付ける
18private async void button3_Click(object sender, EventArgs e) {
19    Button button = (Button)sender;
20    button.Enabled = false;
21    await Task.Run(Execute); // 非同期実行
22    button.Enabled = true;
23}
24
25private void Execute() {
26    for (int i = 0; i <= 100; i += 2) {
27        Thread.Sleep(100);
28        ShowProgress(i);
29    }
30}
31
32private void ShowProgress(int i) {
33    if (InvokeRequired) {
34        Invoke((Action<int>)ShowProgress, i);
35        return;
36    }
37    label1.Text = i.ToString();
38    label1.Refresh();
39}