比较使用 Thread.Sleep 和 Timer 延迟执行

Question

我有一个方法应该延迟运行指定的时间。

我应该使用

Thread thread = new Thread(() => {
    Thread.Sleep(millisecond);
    action();
});
thread.IsBackground = true;
thread.Start();

或者

Timer timer = new Timer(o => action(), null, millisecond, -1);

我读过一些文章，介绍如何使用 Thread.Sleep 是糟糕的设计。 但我真的不明白为什么。

但是，对于使用Timer，Timer有一个dispose方法。 由于执行被延迟，我不知道如何处理Timer。 你有什么建议吗？

或者，如果您有延迟代码执行的替代建议，我们也将不胜感激。

Answer 1

一个区别是 System.Threading.Timer 在线程池线程上分派回调，而不是每次都创建一个新线程。 如果您需要在应用程序的生命周期中多次发生这种情况，这将节省创建和销毁一堆线程的开销（正如您引用的文章指出的那样，这是一个非常资源密集型的进程），因为它将只需重用池中的线程，如果您同时运行多个计时器，则意味着同时运行的线程会更少（也节省了大量资源）。

换句话说，Timer 将会更加高效。 它也可能更准确，因为 Thread.Sleep 只能保证至少等待您指定的时间（操作系统可能会使其休眠更长时间）。 当然，Timer 仍然不会完全准确，但其目的是尽可能接近指定时间触发回调，而这不一定是 Thread.Sleep 的目的。

至于销毁Timer，回调可以接受一个参数，因此您可以将Timer本身作为参数传递，并在回调中调用Dispose（尽管我没有没有尝试过这个——我想计时器可能在回调期间被锁定）。

编辑：不，我想你不能这样做，因为你必须在 Timer 构造函数本身中指定回调参数。

也许是这样的？ （同样，还没有真正尝试过）

class TimerState
{
    public Timer Timer;
}

...并启动计时器：

TimerState state = new TimerState();

lock (state)
{
    state.Timer = new Timer((callbackState) => {
        action();
        lock (callbackState) { callbackState.Timer.Dispose(); }
        }, state, millisecond, -1);
}

锁定应该防止计时器回调在设置 Timer 字段之前尝试释放计时器。

---

附录：正如评论者所指出的，如果 action() 对 UI 做了一些事情，那么使用 System.Windows.Forms.Timer 可能是一个更好的选择，因为它将在 UI 线程上运行回调。 但是，如果情况并非如此，并且取决于 Thread.Sleep 与 Threading.Timer，则 Threading.Timer 是去。

Answer 2

@miniscalope 不，不要使用 ThreadPool.RegisterWaitForSingleObject 而不是计时器，System.Threading.Timer 将在时间过去时将回调排队在线程池线程上执行，并且不需要等待句柄，等待单个对象将在线程调用回调之前，绑定一个线程池线程，等待事件发出信号或超时到期。

Answer 3

我对 System.Timer 的唯一不满是，大多数时候我发现它在轮询服务中使用了很长的延迟（小时、分钟），并且开发人员经常忘记在他们之前启动该事件启动计时器。 这意味着如果我启动应用程序或服务，我必须等到计时器过去（小时，分钟）才能真正执行。

当然，这不是计时器的问题，但我认为它经常被不当使用，因为它太容易被误用。

Answer 4

我记得实施了一种与埃里克类似的解决方案。
然而，这是一个有效的方法；）

class OneTimer
    {
        // Created by Roy Feintuch 2009
        // Basically we wrap a timer object in order to send itself as a context in order to dispose it after the cb invocation finished. This solves the problem of timer being GCed because going out of context
        public static void DoOneTime(ThreadStart cb, TimeSpan dueTime)
        {
            var td = new TimerDisposer();
            var timer = new Timer(myTdToKill =>
            {
                try
                {
                    cb();
                }
                catch (Exception ex)
                {
                    Trace.WriteLine(string.Format("[DoOneTime] Error occured while invoking delegate. {0}", ex), "[OneTimer]");
                }
                finally
                {
                    ((TimerDisposer)myTdToKill).InternalTimer.Dispose();
                }
            },
                        td, dueTime, TimeSpan.FromMilliseconds(-1));

            td.InternalTimer = timer;
        }
    }

    class TimerDisposer
    {
        public Timer InternalTimer { get; set; }
    }

Answer 5

我认为如果您确实想将应用程序暂停指定的时间，Thread.Sleep 就可以了。 我认为人们之所以说这是一个糟糕的设计，是因为在大多数情况下，人们实际上并不希望应用程序暂停。

例如，我正在开发一个 pop3 客户端，其中程序员使用 Thread.Sleep(1000) 来等待套接字检索邮件。 在这种情况下，最好将事件处理程序连接到套接字，并在套接字完成后继续执行程序。

Answer 6

使用线程池。 RegisterWaitForSingleObject 而不是计时器：

//Wait 5 seconds then print out to console. 
//You can replace AutoResetEvent with a Semaphore or EventWaitHandle if you want to execute the command on those events and/or the timeout
System.Threading.ThreadPool.RegisterWaitForSingleObject(new AutoResetEvent(false), (state, bTimeout) => Console.WriteLine(state), "This is my state variable", TimeSpan.FromSeconds(5), true);