Question

假设我们将此事件附加到计时器事件处理程序。

private void TimerTick(object sender, ElapsedEventArgs e)
{
    if(_gaurd) return;
    lock (this) // lock per instance
    {
        _gaurd = true;
        if (!_timer.Enabled) return;

        OnTick(); // somewhere inside here timer may pause it self.

        _gaurd = false;
    }
}

现在有两件事可以暂停这个计时器。一个是来自UI线程的用户请求，第二个是可以自行暂停的计时器。

如果计时器暂停，我们可以保证暂停将在我们继续之前完成。

timer.Stop();
OnPause(); // timer must be paused because OnPause() is not thread safe.

但是，如果用户请求定时器暂停，请求来自另一个线程，我们无法保证计时器是否完全暂停。

timer.Stop();
OnPause(); // timer event may be still inside OnTick() and may conflict with OnPause()

----------------------------------------------- -------------------------------------------------- ------

所以我正在寻找一种方法来使这个线程安全。这是我到目前为止所尝试过的，但我不确定这是否适用于所有情况。

它看起来很好，但想确保如果有什么我不知道的。或者可能知道是否有更好的方法来使这个过程线程安全。

我试图将用户请求与计时器的内部工作分开。因此，我的计时器有两个暂停方法。

public class Timer
{
    internal void InternalStop() // must be called by timer itself.
    {
        timer.Pause(); // causes no problem
    }

    public void Stop() // user request must come here. (if timer call this deadlock happens)
    {
        InternalStop();
        lock (this) // reference of timer
        {
            // do nothing and wait for OnTick().
        }
    }
}

这不是实际代码，但行为是相同的。它应该说明这个类不是线程安全的。：

public class WorkingArea
{
    private List<Worker> _workers;

    public void OnTick()
    {
        foreach(var worker in _workers)
        {
            worker.Start();
        }

        if(_workers.TrueForAll(w => w.Ends))
        {
            PauseTimer();
        }
    }

    public void OnPause() // after timer paused
    {
        foreach(var Worker in _workers)
        {
            worker.Stop();
        }
    }
}

Answer 1

我的计时器已经是线程安全的。

这完全是因为我不了解Re-entrant locks

因此，如果来自另一个线程的用户请求暂停计时器，lock将正常工作并将阻止，直到计时器完全暂停。

如果计时器内部暂停，它不会锁定。因为它在同一个线程中获得了锁。

public class Timer
{
    private timer = new System.Timers.Timer();

    private bool _guard = false;

    // stops the timer and waits until OnTick returns and lock releases.
    // timer can safely pause it self within OnTick.
    // if user request to pause from another thread, full pause is ensured
    public void Stop()       
    {
        timer.Pause();
        lock (this) // reference of timer. it wont dead lock
        {
            // do nothing and wait for OnTick().
        }
    }

    private void TimerTick(object sender, ElapsedEventArgs e)
    {
        if(_gaurd) return;
        lock (this) // lock per instance
        {
            _gaurd = true;
            if (!_timer.Enabled) return;

            OnTick(); // somewhere inside here timer may pause it self.

            _gaurd = false;
        }
    }
}

如何确保计时器完全暂停？

----------------------------------------------- -------------------------------------------------- ------

1 个答案: