10

我正在玩弄 Windows 8 项目的异步服务,并且该服务有一些异步调用,一次只能调用一次。

 public async Task CallThisOnlyOnce()
 {
      PropagateSomeEvents();

      await SomeOtherMethod();

      PropagateDifferentEvents();
 }

由于您无法将异步调用封装在锁定语句中,因此我想到了使用该AsyncLock模式,但我认为我不妨尝试这样的事情:

 private Task _callThisOnlyOnce;
 public Task CallThisOnlyOnce()
 {
      if(_callThisOnlyOnce != null && _callThisOnlyOnce.IsCompleted)
         _callThisOnlyOnce = null;

      if(_callThisOnlyOnce == null)
         _callThisOnlyOnce = CallThisOnlyOnceAsync();

      return _callThisOnlyOnce;
 }

 private async Task CallThisOnlyOnceAsync()
 {
      PropagateSomeEvents();

      await SomeOtherMethod();

      PropagateDifferentEvents();
 }

因此,您最终CallThisOnlyOnceAsync只会同时执行一次调用,并且多个等待者会挂接到同一个任务上。

这是一种“有效”的做法,还是这种方法有一些缺点?

4

2 回答 2

7

一个任务可以有多个等待者。但是,正如 Damien 指出的那样,您提出的代码存在严重的竞争条件。

如果您希望每次调用方法时都执行代码(但不是同时),请使用AsyncLock. 如果您希望代码只执行一次,请使用AsyncLazy.

您提出的解决方案尝试组合多个调用,如果代码尚未运行,则再次执行该代码。这更加棘手,解决方案在很大程度上取决于您需要的确切语义。这是一个选项:

private AsyncLock mutex = new AsyncLock();
private Task executing;

public async Task CallThisOnlyOnceAsync()
{
  Task action = null;
  using (await mutex.LockAsync())
  {
    if (executing == null)
      executing = DoCallThisOnlyOnceAsync();
    action = executing;
  }

  await action;
}

private async Task DoCallThisOnlyOnceAsync()
{
  PropagateSomeEvents();

  await SomeOtherMethod();

  PropagateDifferentEvents();

  using (await mutex.LockAsync())
  {
    executing = null;
  }
}

也可以使用 来执行此操作Interlocked,但是该代码会变得很难看。

PS 我的AsyncEx 库AsyncLock中有,AsyncLazy和其他准备好的async原语。

于 2012-11-23T16:50:25.897 回答
4

如果可能涉及多个线程,则此代码看起来非常“活泼”。

一个例子(我相信还有更多)。假设_callThisOnlyOnce当前是null

Thread 1                                                          Thread 2

public Task CallThisOnlyOnce()
{
  if(_callThisOnlyOnce != null && _callThisOnlyOnce.IsCompleted)
     _callThisOnlyOnce = null;

  if(_callThisOnlyOnce == null)
                                                                   public Task CallThisOnlyOnce()
                                                                   {
                                                                     if(_callThisOnlyOnce != null && _callThisOnlyOnce.IsCompleted)
                                                                        _callThisOnlyOnce = null;

                                                                     if(_callThisOnlyOnce == null)
                                                                        _callThisOnlyOnce = CallThisOnlyOnceAsync();

                                                                     return _callThisOnlyOnce;
                                                                   }
     _callThisOnlyOnce = CallThisOnlyOnceAsync();

  return _callThisOnlyOnce;
}

您现在有 2 个呼叫同时运行。

至于多个等待者,是的,您可以这样做。我确信我在某处看到了来自 MS 的示例代码,其中显示了优化,例如结果Task.FromResult(0)存储在静态成员中,并在函数想要返回零时返回。

但是,我未能成功找到此代码示例。

于 2012-11-23T13:39:46.760 回答