timerでフラグを立てて次のイベントに割り込みをかけたい（C#）

Question

ボタンを用意（今回は３つ）して、ボタン１を押すと10秒後にtimerでフラグをたて
フラグを立てた後どのボタンを押しても

MessageBox.Show("10秒立ったので処理を終了します");
this.Close();

を次に行われるイベントに割り込みをかけたいのですがフラグを立てた後割り込みイベントをどのように立てたらよいのでしょうか？

using System;
using System.Windows.Forms;

namespace WindowsFormsApp
{
    public partial class Form1 : Form
    {
        bool b = false;

        public Form1()
        {
            InitializeComponent();
        }

        private void Button1_Click(object sender, EventArgs e)
        {
            MessageBox.Show("ボタン1");
            timer1.Interval = 10000;
            timer1.Start();
        }

        private void Button2_Click(object sender, EventArgs e)
        {
            MessageBox.Show("ボタン2");
        }

        private void Button3_Click(object sender, EventArgs e)
        {
            MessageBox.Show("ボタン3");
        }
        
        private void Timer1_Tick(object sender, EventArgs e)
        {
            b = true;
            timer1.Stop();
        }


    }
}

使用環境
visualstudio2019　
WindowsForms
.NET framework4.7.2

Answer 1

イベント削除の覚書として。

using System;
using System.Windows.Forms;

namespace WindowsFormsApp1
{
	public partial class Form1 : Form
	{
		public Form1()
		{
			InitializeComponent();

			var class1 = Class1.Instance;

			// シングルトンなのでどのフォームでもボタンを追加できる。
			class1.Buttons.AddRange(new[] {
				button1,
				button2,
				button3,
			});

			// タイマーをミリ秒で設定
			class1.Timer.Interval = 10 * 1000;

			// タイマー終了後にボタンをクリックした時の動作
			class1.Click += (sender, e) =>
			{
				MessageBox.Show("10 秒経ったので元のイベントを破棄しました。");
			};
		}

		private void button1_Click(object sender, EventArgs e)
		{
			Class1.Instance.Timer.Start();
		}

		private void button2_Click(object sender, EventArgs e)
		{
			MessageBox.Show(nameof(button2));
		}

		private void button3_Click(object sender, EventArgs e)
		{
			MessageBox.Show(nameof(button3));
		}
	}
}

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Reflection;
using System.Windows.Forms;

namespace WindowsFormsApp1
{
	public class Class1
	{
		private Class1()
		{
			Timer.Tick += (s, e) =>
			{
				var getEvents = typeof(Control)
					.GetMethod("get_Events", BindingFlags.Instance | BindingFlags.NonPublic);
				var eventClick = typeof(Control)
					.GetField("EventClick", BindingFlags.Static | BindingFlags.NonPublic);
				foreach (var button in Buttons)
				{
					RemoveOldClickEvents(button, getEvents, eventClick);
					button.Click += Click;
				}
				Timer.Stop();
			};
		}

		private void RemoveOldClickEvents(Button button, MethodInfo get_Events, FieldInfo eventId)
		{
			var events = (EventHandlerList)get_Events.Invoke(button, Array.Empty<object>());
			var handlers = events[eventId.GetValue(button)]?.GetInvocationList();
			if (handlers == null) return;
			foreach (EventHandler handler in handlers)
			{
				button.Click -= handler;
			}
		}

		public static Class1 Instance { get; }
			= new Class1();

		public List<Button> Buttons { get; }
			= new List<Button>();

		public Timer Timer { get; }
			= new Timer();

		public event EventHandler Click;
	}
}

Answer 2

ちょっと今スマホなのでテストする環境がないのですが加筆・修正しました。

if(b){
    MessageBox.Show("10秒立ったので処理を終了します");
    this.Close();
}

を何度も書くのはいくら何でもナンセンスなので
Timerとflagと上記処理をまとめて1つのクラスにして、イベントで呼び出される各メソッドがそのインスタンスのCloseチェックメソッドを呼び出す形にすれば綺麗になると思います。

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace WindowsFormsApp
{
    public partial class Form1 : Form
    {
        CloseTimer closeTimer;
        public Form1()
        {
            InitializeComponent();

            closeTimer = new CloseTimer(this, 10000, "10秒立ったので処理を終了します");
        }

        private void Button1_Click(object sender, EventArgs e)
        {
            closeTimer.Start();
            MessageBox.Show("ボタン1");
        }

        private void Button2_Click(object sender, EventArgs e)
        {
            if (closeTimer.CloseIfCan())
                return;
            MessageBox.Show("ボタン2");
        }

        private void button3_Click(object sender, EventArgs e)
        {
            if (closeTimer.CloseIfCan())
                return;
            MessageBox.Show("ボタン3");
        }
    }

    public class CloseTimer : IDisposable
    {
        bool canClose;
        Timer timer;
        string message;
        Form target;

        public CloseTimer(Form target, int delay, string message)
        {
            this.target = target;

            timer = new Timer();
            timer.Interval = delay;
            timer.Tick += (object sender, EventArgs e) =>
            {
                timer.Stop();
                canClose = true;
            };

            this.message = message;
        }

        public void Start()
        {
            if (canClose)
                return;
            timer.Start();
        }

        public bool CloseIfCan()
        {
            if (canClose)
            {
                MessageBox.Show(message);
                target.Close();
                return true;
            }
            return false;
        }

        public void Dispose()
        {
            timer.Dispose();
        }
    }
}

これで、遅延時間を変えたときに全部のボタンクリックイベントを修正しなくてもよくなりました。
例外チェックなど細かな部分は省いているので必要に応じて修正してください。
実際に書くときは別のファイルに書いた方が良いかと思います。

今回Timerを使ってますが、周期的に動作させるわけではないのでTask.Delayでも良いかもしれませんね。

Answer 3

割り込みイベントをどのように立てたらよいのでしょうか？

Button2_ClickとButton3_Clickの中でbの状態による分岐を作りfalseなら従来通りの処理を、trueなら

MessageBox.Show("10秒立ったので処理を終了します");
this.Close();

を呼ぶよう作れば良いように思います。

        private void Button2_Click(object sender, EventArgs e)
        {
            if(b){
                MessageBox.Show("10秒立ったので処理を終了します");
                this.Close();
            }else{
                MessageBox.Show("ボタン2");
            }
        }

また、もしTimerが有効な時のみ従来の処理をしたい場合は、booleanを使う代わりにEnabledプロパティを使い分岐を作ることができます。その場合はタイマーを開始する前と停止した後の振る舞いが工夫をしないと同じになるので気を付けてください。

Comments

[y.u]

ちなみにこれ大量のイベントやフォームがあった場合も全部に
bの分岐を作る必要があるのでしょうか？

別フォームやイベントがたくさんある中でタイマーでフラグが経った場合はどうなるのでしょうか？

一応メインフォーム以外を全部閉じるという処理
//特定フォーム以外（メイン以外のすべてを閉じる）
foreach (Form frm in Application.OpenForms)
{
if (frm.Name != "Main")
{
frm.Close();
}

}
は用意できたのですが

[BluOxy]

> ちなみにこれ大量のイベントやフォームがあった場合も全部にbの分岐を作る必要があるのでしょうか？

複数種類のイベントやフォームがあり、それ等でbを見る必要がでてくればその個数分の分岐を作る必要があります。

> 別フォームやイベントがたくさんある中でタイマーでフラグが経った場合はどうなるのでしょうか？

「タイマーでフラグが経った」というのは何を示しているのでしょうか。
bのことなのか、私が紹介したEnabledプロパティの話なのかが分かりません。

[BluOxy]

いずれにしろ、そのフォーム(Form1)で持っているタイマーの状態を、別のフォームやイベントが参照していれば何かしらの影響はあると思いますが、実際にどう作用するかはそのアプリケーションの中身によります。

[BluOxy]

一見Button2_ClickとButton3_Clickを見比べると同じ処理に見えますが、それ等2つのボタンの仕様が同じかどうかは明記されていないので、「まとめて」とはあえて突っ込みませんでした。

両方共、同じ処理ならまとめても問題ありませんし、それぞれのボタンの役割が分かれているのなら、イベントの定義も分かれているままで良いと思います。

複数bを定義する必要があるかどうかは、まずはボタンが同じ役割を持つかどうかの精査をしてからです。

Answer 4

b の分岐をたくさん作るのが嫌なら、ボタンクリックのハンドラ関数を全部まとめて一つにしてしまえばよろしい。
これならbの分岐は一つで済みます。で、クリック処理はボタン名で判断させればよろしい