C# 任务 ContinueWith 未按预期工作

Question

如何等到前一个方法完成后再继续执行？我以为这很容易，但事实并非如此。尽管我已经阅读了很多示例，但我一定是在做一些非常愚蠢的事情。在下面的代码中，我不能让 GetDocVM() 方法执行，直到 AddUserDocuments() 方法完成。为什么？因为 GetDocVM() 不会拉入刚刚添加的记录。我继承了这段代码并正在努力改进它。

ut.ModelJSON = await Task.Run(() => _userTransactionService.ConvertToModelJson(typeof(UserDocument).Name, "", transactionDocs)).ConfigureAwait(false);
var taskReturnsVoid = Task.Run(() => _genericUploadService.AddUserDocuments(ut, docs));
List<GenericUploadDocumentViewModel> viewModel = new List<GenericUploadDocumentViewModel>();
await taskReturnsVoid.ContinueWith((t) =>
           {
                 viewModel = GetDocVM();//I EXPECTED THIS TO WAIT TO BE EXECUTED
           });
return Json(viewModel, JsonRequestBehavior.AllowGet);  //GETTING HERE TOO SOON

Answer 1

我不羡慕你，因为这看起来是一个非常糟糕的代码库，只是这几行存在多个问题。

最大的问题之一是您不应该在 ASP.NET 上使用 Task.Run 运行 CPU 密集型工作。这是what Stephen Cleary writes about this：

ASP.NET 上的异步和等待都是关于 I/O 的。他们非常擅长读写文件、数据库记录和 REST API。但是，它们不适合 CPU 密集型任务。您可以通过等待 Task.Run 来启动一些后台工作，但这样做没有意义。事实上，这实际上会干扰 ASP.NET 线程池试探法，从而损害您的可伸缩性。如果您在 ASP.NET 上有 CPU 密集型工作要做，最好的办法是直接在请求线程上执行它。作为一般规则，不要将工作排队到 ASP.NET 上的线程池中。

（我推荐阅读his articles，因为它是异步/等待知识的绝佳来源。）

所以你的代码清理了：

ut.ModelJSON = _userTransactionService.ConvertToModelJson(typeof(UserDocument).Name, "", transactionDocs);
_genericUploadService.AddUserDocuments(ut, docs);
List<GenericUploadDocumentViewModel> viewModel = GetDocVM();
return Json(viewModel, JsonRequestBehavior.AllowGet);

但是，我怀疑 _genericUploadService.AddUserDocuments 和 GetDocVM 做了一些与 I/O 相关的工作（如网络或数据库访问）。如果你想提高代码的性能，你应该考虑将它们重写为异步，那么你可以这样：

ut.ModelJSON = _userTransactionService.ConvertToModelJson(typeof(UserDocument).Name, "", transactionDocs);
await _genericUploadService.AddUserDocumentsAsync(ut, docs);
List<GenericUploadDocumentViewModel> viewModel = await GetDocVMAsync();
return Json(viewModel, JsonRequestBehavior.AllowGet);