System.Timers.TimerのEnableがFalseになってしまい、Stop()が利かない

Question

VisualStudio2022、.NET Framework4.8、C#
にてSystem.Timers.Timerを利用し1秒毎の定期処理を実装しています

Start()実行後初回は即時実行したいため以下のように実装しました。
１．Start()呼び出し前にAutoResetをfalse、Intervalを1とする
２．初回のTimer呼び出しの際にAutoResetをtrue、Intervalを1000に設定する

        private static System.Timers.Timer T;
        static void Main(string[] args)
        {
            T = new System.Timers.Timer();
            T.AutoReset = false;
            T.Elapsed += Timer_Elapsed;
            T.Interval = 1;
            T.Start();
            Console.WriteLine(string.Format("Enabled:{0},AutoReset:{1},Interval:{2}", T.Enabled, T.AutoReset, T.Interval));
            Console.ReadKey();
            T.Stop();
            Console.ReadKey();
        }
        private static bool Initialized = false;
        private static void Timer_Elapsed(object sender, System.Timers.ElapsedEventArgs e)
        {
            Console.WriteLine(string.Format("Enabled:{0},AutoReset:{1},Interval:{2}", T.Enabled, T.AutoReset, T.Interval));
            if (!Initialized)
            {
                Initialized = true;
                T.AutoReset = true;
                T.Interval = 1000;
            }
            // 処理
        }

処理の実行タイミングは望んだとおりに動作しているのですが、なぜかStop()でタイマーが停止しません

わかっていることは
Start()した直後はTimer.Enableがtrueなのですが、Timerの処理が呼び出されたときにはfalseとなっており、それが原因でおそらくはStop()が利かないものと思っています。
もしかしたら初回の呼び出しでIntervalを設定した際再度Start()を呼び出さないとならないかとも思いましたが、実際には今のコードでTimerの処理自体は定期的に呼び出されています。（Enableがfalseなのに呼び出しが行われているのもおかしい気はしますが…）

初回は即時実行するタイマー処理を実装し、Stop()で処理を止められるようにしたいのですがどうしたらよいでしょうか？

Comments

[退会済みユーザー]

Timer 自体がメインスレッドとは別のスレッドで動いているからでは？　

ちなみに、Microsoft ドキュメントには "Elapsed イベントは ThreadPool スレッドで発生するため、1 つのスレッドでイベント処理を実行する間に、Stop メソッドの呼び出しが別のスレッドで実行されることがあります。これにより、Stop メソッドを呼び出した後で、Elapsed イベントが発生することがあります" と書いてあります。

単なる思い付きですのでハズレでしたらすみません。

[ikarimame]

＞Timer 自体がメインスレッドとは別のスレッドで動いているからでは？

Timer自体がメインスレッドとは別のスレッドで動いているためにメインスレッドからStop()を呼び出しても機能しない、という事でしょうか？

＞ちなみに、Microsoft ドキュメントには "Elapsed イベントは ThreadPool スレッドで発生するため、1 つのスレッドでイベント処理を実行する間に、Stop メソッドの呼び出しが別のスレッドで実行されることがあります。これにより、Stop メソッドを呼び出した後で、Elapsed イベントが発生することがあります" と書いてあります。

これはタイミングにStop()を呼び出した後１度Elapsedが発生するという事なのかなと思っております、今回の件はStop()を呼び出した後もElapsedが発生し続けてしまいます。

[退会済みユーザー]

すみません、上のコメントはハズレだったようです。

Timer を 2 つ使うのはいかがですか？　後で回答欄にその案を書いておきます。

Answer 1

System.Timers.Timerの不具合っぽいですね。

https://referencesource.microsoft.com/#System/services/timers/system/timers/Timer.cs

ソースを見ると、AutoReset プロパティを True にしたとき、Enabled プロパティを無視してタイマーを有効にしちゃってます。

T.Start();

を入れればOKかと。

ただ、フラグチェックを入れるのも微妙な気がするので

static void Main(string[] args) {
    var T = new System.Timers.Timer();
    T.Elapsed += Timer_Elapsed;
    T.Interval = 1000;
    TimerProc(DateTime.Now);
    T.Start();
    Console.ReadKey();
    T.Stop();
    Console.ReadKey();
}

private static void Timer_Elapsed(object sender, System.Timers.ElapsedEventArgs e) {
    TimerProc(e.SignalTime);
}

private static void TimerProc(DateTime SignalTime) {
    // 処理
}

のようにしとくといいんじゃないでしょうか。

Comments

[TN8001]

AutoResetをいじらなければ（trueのままで）いいんじゃないですかね？

へたにいじるとドツボにはまってしまうようです^^;
[The curious case of System.Timers.Timer | I Came, I Learned, I Blogged](https://evolpin.wordpress.com/2014/04/25/the-curious-case-of-system-timers-timer/)

> System.Timers.Timerの不具合っぽいですね。

「そういう仕様です」ということですねｗ

[KOZ6.0]

＞「そういう仕様です」ということですねｗ
ですね！ｗ

[ikarimame]

ソースの提示ありがとうございます、眺めてみたところ
AutoReset=falseの場合MyTimerCallbackメソッドの中でenableフィールドをfalseにしているが
Enableプロパティを変更したわけではないのでEnableプロパティのsetに実装されているtimerの開放処理が実行されない。
そのためAutoResetをtrueにした際にタイマーが再起動し繰り返しElapsedが発生するようになるが、Enableはfalseという状態が発生してしまうという事のようですね
Stopが動作しないのはEnableがfalseだからのようです……

今回はさほど重い処理を実行するわけではないため、KOZ6.0様に提案していただいた初回はElapsedの呼び出しではなく自前で呼び出すという方向で実装しようと思います。

＞「そういう仕様です」ということですねｗ
これは不具合ではないため修正されることはないだろうという意味でしょうか？
AutoResetをいじるとEnableが変わらないままタイマーが再起動してしまうのは気持ち悪いですね

[KOZ6.0]

＞「そういう仕様です」ということですねｗ
いや、これは冗談ですｗ
要望を出せば修正してくれるかもしれません。

[ikarimame]

承知いたしました、空気が読めずに申し訳ありません
本件.NET Frameworkではなく.NETですが不具合報告はされているようです、しかし.NETでも未だ修正されていないようです
https://github.com/dotnet/runtime/issues/33119

[KOZ6.0]

あれま、上がってましたか。
未解決のものが５Kとかあるから後回しになっちゃってるんでしょうね。

[TN8001]

リロードしたら既にコメントついてたｗ

>> 「そういう仕様です」ということですねｗ
> いや、これは冗談ですｗ
> 要望を出せば修正してくれるかもしれません。

これですかね？
[System.Timers.Timer may activate repeating timer in a way in which Stop() would not stop it · Issue #33119 · dotnet/runtime](https://github.com/dotnet/runtime/issues/33119)

先送りされていますが、直す気はありそうに読めます（いいね等しておくと優先度が上がるかもしれません）
.NET（Core）だけなのか、.NET Frameworkも含んでいるのかはよくわかりません。

C#（.NET）は破壊的変更を相当嫌う印象を持っています。
なので（おかしいのはわかってるけど）今更変えたくないので仕様として押し通すのかと思っていましたが、最近はそうでもないのかもしれませんね^^

Answer 2

「初期化処理をTimer_Elapsed内で行わなければいけない」という縛りは必要ですか？ もしその縛りを外してよければ

1. Main側で初期化処理を実行
2. タイマーを起動し定期処理を実行
3. 任意タイミングでタイマーを終了

と書けます。

こうすればTimer_Elapsedは定期実行すべき処理のみに専念できるかと思います。

Comments

[ikarimame]

今回は初回の処理を即時実行し以降は１秒間隔で定期処理する処理を実装したかったため、Timer_Elapsedの初回呼び出しでIntervalを変更するという実装を行った次第です。

Answer 3

上の質問に対する私のコメントで、

Timer を 2 つ使うのはいかがですか？　後で回答欄にその案を書いておきます。

・・・と書いた件です。

以下のコードで、

using System;

namespace ConsoleAppTimer
{
    internal class Program
    {
        private static System.Timers.Timer T;
        private static System.Timers.Timer T2;

        static void Main(string[] args)
        {
            T = new System.Timers.Timer();
            T.AutoReset = false;
            T.Elapsed += T_Elapsed;
            T.Interval = 1;

            // 2022/12/7 17:58 変更　T スタートのタイミング            
            // ここで T をスタートすると T2 の初期化前に T がスタートしてしまう可能性があるので
            // T.Start();

            Console.WriteLine(string.Format("T1 / Enabled:{0},AutoReset:{1},Interval:{2}", 
                T.Enabled, T.AutoReset, T.Interval));

            T2 = new System.Timers.Timer();
            T2.AutoReset = true;
            T2.Elapsed += T2_Elapsed;
            T2.Interval = 1000;

            // T2 の初期化が完了してからスタート
            T.Start();

            Console.ReadKey();
            T2.Stop();
            Console.ReadKey();
        }        

        //private static bool Initialized = false;

        private static void T_Elapsed(object sender, System.Timers.ElapsedEventArgs e)
        {
            Console.WriteLine(string.Format("T1 / Enabled:{0},AutoReset:{1},Interval:{2}",
                T.Enabled, T.AutoReset, T.Interval));

            T2.Start();

            //if (!Initialized)
            //{
            //    Initialized = true;
            //    T.AutoReset = true;
            //    T.Interval = 1000;
            //}

        }

        private static void T2_Elapsed(object sender, System.Timers.ElapsedEventArgs e)
        {
            Console.WriteLine(string.Format("T2 / Enabled:{0},AutoReset:{1},Interval:{2}",
                T2.Enabled, T2.AutoReset, T2.Interval));
        }
    }
}

結果は以下のようになります。実行後、最初の Console.ReadKey(); で入力待ちのところに入力して、次の行の T2.Stop(); で Timer を止めたところです。

Comments

[KOZ6.0]

これだと T2 の初期化が完了する前に T2.Start が走ってしまう可能性があります。
T2 の初期化は T.Start() の前にやったほうが良いです。

[退会済みユーザー]

そうですね。書き直しておきます。

[ikarimame]

ありがとうございます、参考にさせていただきます。