只需停止异步方法答案

【问题标题】：simply stop an async method只需停止异步方法
【发布时间】：2013-03-14 22:41:02
【问题描述】：

我有这个方法可以在用户点击屏幕时播放声音，并且我希望它在用户再次点击屏幕时停止播放。但问题是“DoSomething()”方法不会停止，它会一直运行直到完成。

bool keepdoing = true;

private async void ScreenTap(object sender, System.Windows.Input.GestureEventArgs e)
    {
        keepdoing = !keepdoing;
        if (!playing) { DoSomething(); }
    }

private async void DoSomething() 
    {
        playing = true;
        for (int i = 0; keepdoing ; count++)
        {
            await doingsomething(text);
        }
        playing = false;
    }

任何帮助将不胜感激。
谢谢:)

【问题讨论】：

尝试将keepdoing 声明为volatile bool keepdoing = true; 但如果doingsomething 的返回时间过长，用户可以按两次屏幕，从而将keepdoing 切换为false 并返回true。
做某事不需要很长时间，但由于循环，Dsomething() 需要时间。 & 我很遗憾地说，但 volatile 不起作用。
易失性不是问题，因为等待之后的代码总是被分派回UI线程......这不是同步问题。

标签： c# windows-phone async-await

【解决方案1】：

这就是CancellationToken 的用途。

CancellationTokenSource cts;

private async void ScreenTap(object sender, System.Windows.Input.GestureEventArgs e)
{
  if (cts == null)
  {
    cts = new CancellationTokenSource();
    try
    {
      await DoSomethingAsync(cts.Token);
    }
    catch (OperationCanceledException)
    {
    }
    finally
    {
      cts = null;
    }
  }
  else
  {
    cts.Cancel();
    cts = null;
  }
}

private async Task DoSomethingAsync(CancellationToken token) 
{
  playing = true;
  for (int i = 0; ; count++)
  {
    token.ThrowIfCancellationRequested();
    await doingsomethingAsync(text, token);
  }
  playing = false;
}

【讨论】：

是否应该有一个finally 块将cts 设置为null？完成（未取消）运行后，需要轻按 2 次才能运行 DoSomethingAsync。
@Gusdor：好收获！我在修复中进行了编辑。这不是很好 - CTS 可以被重用，而不仅仅是 GCed，但它传达了一般意义。

【解决方案2】：

在不引发异常的情况下使用 CancellationToken 的另一种方法是声明/初始化 CancellationTokenSource cts 并将 cts.Token 传递给 DoSomething，如上面 Stephen Cleary 的回答。

private async void DoSomething(CancellationToken token) 
{
    playing = true;
    for (int i = 0; keepdoing ; count++)
    {
        if(token.IsCancellationRequested)
        {
         // Do whatever needs to be done when user cancels or set return value
         return;
        }
        await doingsomething(text);
    }
    playing = false;
}

【讨论】：

好建议，因为在我的情况下它不会是一个例外。