【发布时间】:2011-10-11 15:46:43
【问题描述】:
Task Parallel Library 非常棒,在过去的几个月里我经常使用它。然而,有一些事情让我很困扰:TaskScheduler.Current 是默认任务调度程序,而不是TaskScheduler.Default。这在文档和示例中乍一看绝对不明显。
Current 可能会导致细微的错误,因为它的行为会根据您是否在另一个任务中而发生变化。哪个不容易确定。
假设我正在编写一个异步方法库,使用基于事件的标准异步模式在原始同步上下文中发出信号完成信号,就像 XxxAsync 方法在 .NET Framework 中所做的一样(例如DownloadFileAsync)。我决定使用任务并行库来实现,因为使用以下代码很容易实现此行为:
public class MyLibrary
{
public event EventHandler SomeOperationCompleted;
private void OnSomeOperationCompleted()
{
SomeOperationCompleted?.Invoke(this, EventArgs.Empty);
}
public void DoSomeOperationAsync()
{
Task.Factory.StartNew(() =>
{
Thread.Sleep(1000); // simulate a long operation
}, CancellationToken.None, TaskCreationOptions.None, TaskScheduler.Default)
.ContinueWith(t =>
{
OnSomeOperationCompleted(); // trigger the event
}, TaskScheduler.FromCurrentSynchronizationContext());
}
}
到目前为止,一切正常。现在,让我们在 WPF 或 WinForms 应用程序中单击按钮调用此库:
private void Button_OnClick(object sender, EventArgs args)
{
var myLibrary = new MyLibrary();
myLibrary.SomeOperationCompleted += (s, e) => DoSomethingElse();
myLibrary.DoSomeOperationAsync(); // call that triggers the event asynchronously
}
private void DoSomethingElse() // the event handler
{
//...
Task.Factory.StartNew(() => Thread.Sleep(5000)); // simulate a long operation
//...
}
这里,编写库调用的人选择在操作完成时启动一个新的Task。没有什么不寻常的。他或她遵循网络上随处可见的示例,只需使用Task.Factory.StartNew 而不指定TaskScheduler(并且在第二个参数处指定它并不容易重载)。 DoSomethingElse 方法在单独调用时工作正常,但一旦被事件调用,UI 就会冻结,因为 TaskFactory.Current 将重用我的库延续中的同步上下文任务调度程序。
找出这一点可能需要一些时间,尤其是当第二个任务调用隐藏在某个复杂的调用堆栈中时。当然,一旦你知道一切是如何工作的,这里的修复很简单:总是为你希望在线程池上运行的任何操作指定TaskScheduler.Default。但是,也许第二个任务是由另一个外部库启动的,不知道这种行为,并且在没有特定调度程序的情况下天真地使用 StartNew。我预计这种情况会很常见。
在思考之后,我无法理解编写 TPL 的团队选择使用 TaskScheduler.Current 而不是 TaskScheduler.Default 作为默认值:
- 一点都不明显,
Default不是默认的!而且文档严重缺乏。 -
Current使用的真正任务调度程序取决于调用堆栈!这种行为很难保持不变量。 - 使用
StartNew指定任务调度程序很麻烦,因为您必须先指定任务创建选项和取消令牌,导致行长、可读性差。这可以通过编写扩展方法或创建使用Default的TaskFactory来缓解。 - 捕获调用堆栈会产生额外的性能成本。
- 当我真的希望一个任务依赖于另一个正在运行的父任务时,我更愿意明确指定它以简化代码阅读,而不是依赖调用堆栈魔法。
我知道这个问题可能听起来很主观,但我找不到一个好的客观论据来解释为什么会出现这种行为。我确定我在这里遗漏了一些东西:这就是我转向你的原因。
【问题讨论】:
-
我正在努力完全按照您的示例进行操作,但是假设将在 UI 上下文中调用它,这难道不是消费代码 (
DoSomethingElse) 中的错误吗? (如果这就是你想要表达的意思——它不是在 UI 上下文中创建任务) -
恰恰相反:
DoSomethingElse可以在此处的任何上下文中运行,但在这种特定情况下,它创建的任务将在父任务的上下文中运行,它本身在 UI 线程上运行,没有知道它。如果使用Default任务调度程序,则没有问题。我指定它没有任何问题,但我不控制每个第三方库,并不总是意识到这一事实。我不明白为什么Current是所有那些潜在危险的变化上下文的默认值。不过这个问题可能太有争议了。 -
在 .NET 4.5 中,现在有 Task.Run,其中 TaskScheduler.Default 是默认的 TaskScheduler:blogs.msdn.com/b/pfxteam/archive/2011/10/24/10229468.aspx
-
您是否考虑过显式调用 UI 线程而不是使用调度程序进行调用?对我来说,这似乎是灾难的秘诀。不过我同意你的看法,这在 TPL 团队方面相当缺乏逻辑。
标签: c# .net task-parallel-library conceptual synchronizationcontext