在Task.Run闭包内外分配结果变量有何差异？哪种写法更规范？

先看第一种在Task内部给外部变量赋值的写法：

var cat;
var house;
var car;

var catTask   = Task.Run(() => { cat   = FeedCat();   } );
var houseTask = Task.Run(() => { house = SellHouse(); } );
var carTask   = Task.Run(() => { car   = BuyCar();    } );

await Task.WhenAll(catTask, houseTask, carTask);

这种写法确实不算明智，也不符合常规编程惯例，问题主要有几点：

• 线程安全隐患：虽然当前三个变量独立不会出现竞争，但这种依赖闭包修改外部变量的写法，后续代码迭代时很容易引入线程安全问题——比如不小心复用变量、新增共享状态时，极易踩坑。
• 可读性差：变量声明和赋值拆成两部分，阅读代码时得来回跳转找赋值逻辑，没法一眼理清每个Task对应什么操作的结果。
• 调试麻烦：如果某个Task执行出问题（比如FeedCat抛出异常），得先定位到对应Task，再去lambda里查代码，远不如直接返回Task<T>的方式直观。
• 违背异步编程设计原则：.NET异步API的设计逻辑，就是让Task<T>承载异步操作的结果，这种通过“副作用”修改外部变量的写法，不符合异步编程的干净风格，可维护性大打折扣。

再看第二种通过Task<T>返回结果、外部await获取的写法：

var catTask   = FeedCat();
var houseTask = SellHouse();
var carTask   = BuyCar();

await Task.WhenAll(catTask, houseTask, carTask);

var cat   = await catTask;
var house = await houseTask;
var car   = await carTask;

这种写法是异步编程里的常规操作，优势很明显：

• 线程安全：每个Task<T>独立存储结果，await时只是从已完成的Task中取出结果，完全没有多线程共享变量的问题。
• 可读性拉满：Task创建和结果获取逻辑一一对应，一眼就能看明白每个Task对应什么操作，后续取结果的代码清晰直白。
• 调试方便：每个Task的状态、结果、异常都能直接通过Task对象查看，出问题时定位效率很高。
• 符合最佳实践：充分利用Task<T>的设计初衷，通过返回值传递结果，没有副作用，代码干净易维护，也是.NET社区推荐的异步编程写法。

额外补充：在Task.WhenAll之后await已完成的Task，不会造成额外等待——因为Task已经处于完成状态，await只是直接取出结果，性能上完全没有问题。

内容的提问来源于stack exchange，提问作者amonroejj