Question

我正在以正确/最好的方式完成此任务吗？

我有一个带定时器的窗口。每次计时器滴答时，我都会调用下面显示的RunTask方法。在RunTask内，我致电DoTheThing。 DoTheThing可能需要一段时间才能运行，并且可能会失败（它是数据库更新）。我想确保在任何时候，我只有一个DoTheThing突出。我还想确保我没有一堆RunTask个实例排队并等待运行RunTask的{{1}}实例发布锁定。

DoTheThing

由于程序的体系结构，我宁愿将所有线程同步放在此方法中，而不是启用和禁用计时器。

Answer 1

通过略微区别地思考问题，它变得容易多了。不是每隔x秒触发一次计时器，为什么不在调用之间等待x秒？

现在你可以运行一个异步循环来完成预定的工作，并为自己节省一些痛苦的同步工作。

async Task RunActionPeriodicallyAsync(Action action, 
                           TimeSpan ts, 
                           CancellationToken token = default(CancellationToken))
{
    while(!token.IsCancellationRequested)
    {
        action();
        await Task.Delay(ts, token);
        //or alternatively (see comment below)
        //var delayTask = Task.Delay(ts, token);
        //action();
        //await delayTask;
    }
}

现在，只需拨打RunActionPeriodicallyAsync一次，对其操作的调用就不会重叠。

RunActionPeriodicallyAsync(() => DoSomething(), TimeSpan.FromSeconds(10))

您可以重载此操作以采取异步＆＃34;操作＆＃34; ...实际上是Func<Task> ...

async Task RunActionPeriodicallyAsync(Func<CancellationToken, Task> actionAsync, 
                           TimeSpan ts, 
                           CancellationToken token = default(CancellationToken))
{
    while(!token.IsCancellationRequested)
    {
        await actionAsync(token);
        await Task.Delay(ts, token);
        //or alternatively (see comment below)
        //await Task.WhenAll(actionAsync(token), Task.Delay(ts, token))
    }
}

并使用它：

RunActionPeriodicallyAsync(async cancTok => await DoSomethingAsync(cancTok), 
                           TimeSpan.FromSeconds(10))

Answer 2

如果您担心锁定过多，可以执行以下操作。如果一个任务完成而另一个任务刚好在检查（标记），你可能会错过一次运行，但是你摆脱了一些锁定，你只需要在设置isTaskRunnung = true时锁定。另外，您需要将方法标记为异步，以便等待任务。

public async Task RunTask()
{
    bool canRunTask = true;

    // Check if another instance of this method is currently executing.  If so, do not execute the rest of this method
    if (this.isTaskRunning)
    {                                       // <-- ___MARK___
        canRunTask = false;
    }
    else
    {
        lock (this.runTaskLock)
        {
            if (this.isTaskRunning)
            {
                canRunTask = false;
            }
            else
            {
                    this.isTaskRunning = true;
            }
        }
    }

    // Call DoTheThing if another instance is not currently outstanding
    if (canRunTask)
    {
        try
        {
            await Task.Run(() => DoTheThing());
        }
        catch (Exception ex)
        {
            // Handle the exception
        }
        finally
        {
            this.isTaskRunning = false;
        }
    }
}

正确的方法来进行任务同步？

2 个答案: