C# タイマー処理を停止したい

Question

開発環境:visual studio 2017

現在、タイマー処理を利用して一定時間毎にループする処理を実装したいのですが、苦戦しています。
苦戦している点は、停止時にFormは消さずに処理のみ停止させたいです。
現状は下記で記載しています。停止ボタンを押すとアプリケーションごと終了させてしまっています。

希望としては、
①Formを消さずにループ処理のみ終了させたい
②いつでもボタンを押せば停止するようにしたい。
③再度、実行するときは最初からスタートしたい
です。

タイマーの種類も多く存在しており、いろいろなサイトを見ても理解出来ていません。
下記のソースを使ってこんな感じに記載すればよいと教えていただけると理解して今後に生かせると思っていますのでお手数をおかけ致しますが回答お願いいたします。

public partial class Form1 : Form
    {
        System.Windows.Forms.Timer timer1 = new System.Windows.Forms.Timer();
        System.Windows.Forms.Timer timer2 = new System.Windows.Forms.Timer();
            loop1(Main_loop);
            loop2(sabu_loop);
        public Form1()
        {
            InitializeComponent();
            button1.Enabled = true;
            button2.Enabled = false;
        }
        public void button1_Click(object sender, EventArgs e)
        {
            button1.Enabled = false;
            button2.Enabled = true;
            
         //タイマー開始
            timer1.Enabled = true;
            timer2.Enabled = true;
        }
        private void button2_Click(object sender, EventArgs e)
        {
            //タイマーの停止
            timer1.Enabled = false;
            timer2.Enabled = false;
            //ボタンを戻す
            button1.Enabled = true;
            button2.Enabled = false;
        }
      
        //3秒ごとに動くタイマー
        public void loop1(EventHandler eventHandler)
        {
            timer1.Tick += new EventHandler(eventHandler);
            timer1.Interval = 1000;
            timer1.Enabled = false;

        }
        //numericUpDown1 * 60秒 でループするタイマー
        public void loop2(EventHandler eventHandler)
        {
            int intVal = Decimal.ToInt32(numericUpDown1.Value);
            timer2.Tick += new EventHandler(eventHandler);
            timer2.Interval = intVal * 60000;
            timer2.Enabled = false;
        }
        private void Main_loop(object sender, EventArgs e)
        {
            //メイン処理
        }
        void sabu_loop(object sender, EventArgs e)
        {
            //サブ処理
        }

    }
}

Answer 1

//loopの中ではなく、クラス内で使用できるスコープで宣言する
System.Windows.Forms.Timer timer = new System.Windows.Forms.Timer();

        private void button2_Click(object sender, EventArgs e)
        {
            //タイマーの停止
            timer.Enabled = false;
            //ボタンを戻す
            button1.Enabled = true;
            button2.Enabled = false;

        }

でタイマーの停止はできるかと
再開に関してはMain_loop内の処理次第でしょう。
フラグなり設けるか、して最初から行えばいいでしょう。

また、loop内でTimerの生成、イベントハンドラを設定するのではなく、コンストラクタで行ったほうがよろしいかと
loop内ではIntervalとEnabledの設定だけにしたほうが良いと思います。

追記

        public Form1()
        {
            InitializeComponent();
            button1.Enabled = true;
            button2.Enabled = false;
            //イベントハンドラの設定
            timer1.Tick += new EventHandler(Main_loop);
            timer2.Tick += new EventHandler(sabu_loop);
        }

でloop1内などでは、イベントハンドラを設定する必要はありません。

というか、使うタイマーがWindows.Forms.Timerなのであれば、フォームデザイナ上で配置してイベント登録すれば、このようなことにならないかと思います。

Comments

[warks1]

回答ありがとうございます。
質問文のソースも変更しました。
Formが消えずに停止するようになったのですが、停止してから再開した際に2回動くようになってしまいました。何かおかしい点等分かれば教えていただけないでしょうか。

[YAmaGNZ]

回答にイベントハンドラをloop内で設定しないように書いたのはこの為です。
イベントハンドラが設定されているところに
timer1.Tick += new EventHandler(eventHandler);
ともう一度設定した場合、上書きされるのではなく、追加されます。
なので、イベントが発生した時に登録されているイベントハンドラが順次呼ばれますので、イベントハンドラが複数回呼ばれることになります。

[warks1]

無知で申し訳ありません。
質問文のソースを再度変更しました。
このような解釈で問題ないでしょうか。

[warks1]

追記ありがとうございます。
質問文のソースを追記していただいたように変えたところ、ループが何度も瞬間で発生しました。。
ご丁寧に説明いただいているのになかなか理解が追いつかず申し訳ありません。
もう少し、回答いただいた内容で少しずつ理解していこうかと思います。

[YAmaGNZ]

Intervalはボタンが押された時に設定してください。