每天在特定时间执行方法的代码C#（Windows服务）失败

Question

我每天早上 5 点都有这段代码来执行 windows 服务的方法：

编辑：

MyService ws = new MyService ();

protected override void OnStart(string[] args)
{
    if (serviceHost != null)
    {
        serviceHost.Close();
    }

    serviceHost = new ServiceHost(typeof(MyService));

    serviceHost.Open();
    double TimeOfExecution = 5;

    DateTime now = DateTime.Now;
    DateTime today5am = now.Date.AddHours(TimeOfExecution);
    DateTime next5am = now <= today5am ? today5am : today5am.AddDays(1);

    System.Threading.TimerCallback callback = new System.Threading.TimerCallback(ws.MethodToExecute());

    var timer1 = new System.Threading.Timer(callback, null, next5am - DateTime.Now, TimeSpan.FromHours(24));

 }

我预计服务会在下一个 5 点执行，之后每 24 小时执行一次。

方法MethodToExecute() 确实在当天 5 点（或其他指定时间）执行，但第二天它未能执行。此外，似乎我是否第一次执行它并不重要，但似乎服务在一段时间后进入睡眠状态并且不执行，所以如果 5 点到达不是当天，而是第二天，它会执行失败。

有谁知道可能出了什么问题？

Answer 1

GC 将收集您的 timer，因为在 OnStart 方法之后您没有任何对它的引用。

你只是把它当作一个局部变量。我希望您知道，一旦 JIT 表明它们不再在代码中使用，局部变量就有资格进行垃圾回收。

修复：只需将 timer 存储在实例变量中，就完成了。

private System.Threading.Timer my5AmTimer = null;

protected override void OnStart(string[] args)
{
    //All other code..

   this.my5AmTimer = new System.Threading.Timer(callback, null, next5am - DateTime.Now, TimeSpan.FromHours(24));
}

Answer 2

var timer1 = new System.Threading.Timer(...);

计时器是棘手的对象。当它们不再被引用时，它们就像任何 .NET 对象一样受到垃圾回收的影响。该语句的语法足以知道何时发生这种情况，您对var 的使用表明“timer1”是方法的局部变量。换句话说，在您的方法返回后，您创建的 Timer 对象没有任何引用。

那么，当没有 引用时，究竟是什么让 Timer 保持活跃和滴答作响？你发现：什么都没有。您的计时器是否会真正滴答作响是一个废话。如果您的程序继续运行做其他事情，那么它会触发 gen#0 集合，并且计时器将消失。它永远不会滴答作响。如果你的程序没有，那么它可以存活到凌晨 5 点。

您必须将对象引用存储在一个变量中，以确保它存在足够长的时间。那应该是一个静态变量。

或者使用 System.Timers.Timer。它的作用很像 System.Threading.Timer，但具有更好的生存技能。只要它具有 Elapsed 事件处理程序并且已启用，它就可以保证保持活动状态。

Answer 3

请read the fine manual:

注意

只要您使用 Timer，就必须保留对它的引用。与任何托管对象一样，当没有对 Timer 的引用时，它会受到垃圾回收的影响。 Timer 仍然处于活动状态这一事实并不能阻止它被收集。

保留对它的引用：

MyService ws = new MyService ();

System.Threading.Timer timer;

protected override void OnStart(string[] args)
{
    ...

    if (timer1 != null)
        timer1.Dispose();

    timer1 = new System.Threading.Timer(callback, null,
        next5am - DateTime.Now, TimeSpan.FromHours(24));
 }

Answer 4

另一个选项可能是告诉 GarbageCollector 保持局部变量处于活动状态：

GC.KeepAlive(timer1);