即使我正在顺序处理，我也应该阻止

我有一个Windows服务，定期需要做一些工作。 所以我设置了一个System.Timers.Timer来做到这一点。 让我们假设处理时间可能大于计时器间隔。 让我们假设如果发生这种情况会是一件非常糟糕的事情。

为了避免这种情况，我将Timer上的AutoReset设置为false，然后在我的进程中调用start。

public partial class Service : ServiceBase{ System.Timers.Timer timer; public Service() { timer = new System.Timers.Timer(); //When autoreset is True there are reentrancy problme timer.AutoReset = false; timer.Elapsed += new System.Timers.ElapsedEventHandler(DoStuff); } protected override void OnStart(string[] args) { timer.Interval = 1; timer.Start(); } private void DoStuff(object sender, System.Timers.ElapsedEventArgs e) { Collection stuff = GetData(); LastChecked = DateTime.Now; foreach (Object item in stuff) { item.Dosomthing(); //Do somthing should only be called once } TimeSpan ts = DateTime.Now.Subtract(LastChecked); TimeSpan MaxWaitTime = TimeSpan.FromMinutes(5); if (MaxWaitTime.Subtract(ts).CompareTo(TimeSpan.Zero) > -1) timer.Interval = MaxWaitTime.Subtract(ts).TotalMilliseconds; else timer.Interval = 1; timer.Start(); } 

目前代码没有阻止，因为我知道由于AutoReset = false，它被顺序处理。 但我可以做到这一点

 lock(myLock) { Collection stuff = GetData(); LastChecked = DateTime.Now; foreach (Object item in stuff) { item.Dosomthing(); //Do somthing should only be called once } } 

编辑：澄清我的问题

我把服务设计成单线程，所以我不需要锁。 如果我添加锁定，我仍然在我的性能预算范围内，因此性能不是理由。

基本上我正在权衡两面，我正在尝试理清Right Thing™是什么。 在“无锁”方面，我依赖于对我的代码正确性的设计。 在“锁定”方面，我将添加不必要的代码。

哪个更好？

Answer 1

我要么全力以赴保持线程安全，要么根本没有线程安全，只要在文档中非常清楚地写出该类不是线程安全的。

最糟糕的事情是几年后回来，而不记得一切是否安全。 然后你最终浪费了大量时间来调查你的代码，或者更糟糕的是，你最终会误导自己。

Answer 2

如果多个线程将使用这些对象，则只需要lock 。 在您的情况下，您的设计可以防止这种情况发生，因此无需锁定。

在这种情况下，添加锁定的唯一真正好处是可以防止在以后更改调度算法时出现问题。

Answer 3

就像其他人所说的那样，如果是单线程，则无需锁定。 此外，您可以一起跳过计时器：

  TimeSpan maxInterval = new TimeSpan(0, 10, 0); while(true) { DateTime startTime = DateTime.UtcNow; //Do lots and lots of work TimeSpan ts = DateTime.UtcNow - startTime; ts = (ts > maxInterval ? new TimeSpan(0) : maxInterval-ts); Thread.Sleep(ts); }