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

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

ただいまの
回答率

89.65%

C# Windowsフォームのボタンイベント制御について

受付中

回答 3

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 464

f_tonakai

score 7

前提・実現したいこと

C#でWindowsフォームの開発をしています。
今やりたいことは
1.フォーム上にボタンが2つ [button_A]と[button_B] がある。
2.[button_A]を押すと、処理が走る(例えば、DB接続やファイル書込)
3.(2)の処理中に[button_A]と[button_B]のクリックイイベントを動かないようにする。
ただし、ボタン非活性(enableプロパティ=true)は行わない。

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

[button_A]の実行中に[button_A]や[button_B]をクリックすると、
[button_A]の処理後にイベントが実行されてしまう。(イベントがメモリ上に残ってしまう)

試したこと

イベント破棄をするために、
this.button1.Click += new System.EventHandler(this.button1_Click); 
を実装したが、うまくいかない

// Form1.cs

namespace TestSample {
    public partial class Form1 : Form {

        private int isProcessing;

        public Form1() {
            isProcessing = 0;
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e) {

            if(isProcessing == 1) {
                return;
            }

            isProcessing = 1;
            this.button1.Click -= new System.EventHandler(this.button1_Click);
            this.button2.Click -= new System.EventHandler(this.button2_Click);

            // テキストに数字を出力(テスト用で簡易なロジック実装)
            DataOutPut dataOutPut = new DataOutPut("[start]", "[end]");
            dataOutPut.OutPutFile();

            isProcessing = 0;
            Application.DoEvents();
            this.button1.Click += new System.EventHandler(this.button1_Click);
            this.button2.Click += new System.EventHandler(this.button2_Click);

            return;

        }

        private void button2_Click(object sender, EventArgs e) {
            this.Close();
            return;
        }

        private void Form1_Closing(object sender, FormClosingEventArgs e) {
            if (isProcessing == 1) {
                Console.WriteLine("★ -- No Form1_Closing() -- ★");
                e.Cancel = true;
            }
        }
    }
}
// DataOutPut.cs

namespace TestSample {
    class DataOutPut {

        private string startCode;
        private string endCode;

        public DataOutPut(string startCode, string endCode) {
            this.startCode = startCode;
            this.endCode = endCode;
        }

        // テキストファイルに出力
        public int OutPutFile() {

            string filePath = @"C:\Users\fujiwarak\Desktop\work\";
             string fileName = "test_";
            string dt = (DateTime.Now).ToString("yyyyMMddHHmmss");
            Encoding enc = Encoding.GetEncoding("Shift_JIS");

            int index = 0;
            while (index < 3) {
                string filePathName = filePath + fileName + index.ToString() + dt + ".txt";
                using (StreamWriter writer = new StreamWriter(filePathName, true)) {
                    writer.WriteLine(startCode);
                    int loop = 0;
                    int number = 0;
                    while (loop <= 10) {
                        System.Threading.Thread.Sleep(1000);
                        writer.WriteLine(number.ToString());
                        number++;
                        loop++;
                    }
                    writer.WriteLine(endCode);
                }
                index++;
            }
            return 0;
        }
    }
}
  • 気になる質問をクリップする

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • Zuishin

    2019/06/06 06:53 編集

    OutPutFile メソッドのループの中に Application.DoEvents を入れたらおそらく思う通りになると思いますが、Enabled でいいじゃないですか。何ですかその押せるのに押したらいけないという変な仕様?

    キャンセル

  • f_tonakai

    2019/06/08 17:45

    OutPutFile メソッド側にいれてやってみます。
    ありがとうございます。

    キャンセル

  • Zuishin

    2019/06/08 17:48

    Enabled でいいじゃないですか。

    キャンセル

回答 3

0

そのコードを提示しましょう
Aを押されてたらBをパスするという処理を書けば、動くことはありません。


this.button1.Click += new System.EventHandler(this.button1_Click); 

+=というのはイベントを追加するだけです。


イベントが残る、んじゃなくて、メッセージがメッセージキューに溜まってるだけなんで、
フラグを解除する前にメッセージキューを処理してしまいましょう

        bool flg;

        private void button3_Click(object sender, EventArgs e)
        {
            if (flg) return;

            flg = true;  // try finally するほうがいいでしょうな
            Debug.WriteLine("button3");
            Thread.Sleep(2000);
            Application.DoEvents();  // メッセージキューを処理
            flg = false;
        }

        private void button4_Click(object sender, EventArgs e)
        {
            if (flg) return;

            Debug.WriteLine("button4");
            Thread.Sleep(2000);
        }

んで、フォームの☓をどうにかしたいなら、FormClosingイベントを処理すればいいです

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/06/05 23:06 編集

    ありがとうございます

    追加なんですが
    [button_A]も[画面右上の×ボタン]も動かないようにしたいです
    [button_A]の連打はできないようにしたいです

    キャンセル

  • 2019/06/05 23:09

    その処理中になんかフラグを立てるようにし、それらの処理をしようとするときにそのフラグを判断し、以降の処理をパスするようにコードを書けばいいです

    キャンセル

  • 2019/06/08 17:47

    ソースまでありがとうございます。
    参考にさせて頂きます。

    キャンセル

0

            this.button1.Click -= new System.EventHandler(this.button1_Click);
            this.button2.Click -= new System.EventHandler(this.button2_Click);

            this.button1.Click += new System.EventHandler(this.button1_Click);
            this.button2.Click += new System.EventHandler(this.button2_Click);

ではなく

            this.button1.Click -= this.button1_Click;
            this.button2.Click -= this.button2_Click;

            this.button1.Click += this.button1_Click;
            this.button2.Click += this.button2_Click;

とそれぞれ該当箇所書き換えてみてください。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/06/06 07:13

    同じだと思います。後者は糖衣構文なので。

    キャンセル

  • 2019/06/06 17:01

    あ、そうか。ご指摘ありがとうございます。

    キャンセル

0

OutPutFileメソッドの終了時にtimerを立ち上げて
timer_tickイベントで
isProcessing = 0;
とすれば良いのではないでしょうか?
どれだけのイベントがメモリ上に残るのかわかりませんが、残ったイベントを逃がせると思います。

あと、isProcessingはboolでもいい気はしますが・・・。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

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

  • ただいまの回答率 89.65%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる