是否需在MethodTwo中使用并发类型？anotherList能否完整保存？

1. 是否需要在MethodTwo中使用并发类型？ #

不需要。anotherList是MethodTwo的局部变量，每次调用MethodTwo（包括Task触发的异步调用）都会生成一个全新的独立实例，各个线程的anotherList完全隔离，不存在多线程共享修改的场景，因此根本没必要用ConcurrentBag这类并发集合。

2. 不使用并发类型，anotherList能否完整保存正确值？ #

可以。每个anotherList实例只在当前MethodTwo的执行线程中被操作，添加元素的逻辑完全基于传入的prop参数，只要prop的传入值正确，anotherList的内容就一定符合预期：要么是["AAA", "BBB"]（当prop>20），要么是["AAA", "CCC"]（当prop≤20），不会出现数据丢失或错乱。

3. 代码潜在bug排查 #

你遇到的问题大概率出在MethodOne的闭包捕获或变量名错误上：

• 变量名大小写错误：MethodOne中声明的是var Item = GetById(id);（大写I），但Task.Run的lambda里写的是MethodTwo(item.PropOne)（小写i）。如果这是笔误，要么代码无法编译通过；如果存在外部的item变量，所有Task都会复用这个外部变量的值，导致所有MethodTwo调用拿到的都是同一个prop值，直接让SaveList的结果完全不符合预期。
• 闭包捕获的经典问题：如果修正变量名后，若你的项目使用C#5之前的编译器，foreach循环的变量是在循环外声明的，所有lambda会捕获同一个id或Item变量，导致多个Task执行时拿到的是循环最后一次的id/Item值。C#5及之后版本已修复该问题，循环变量会在每次迭代中重新声明，但旧版编译器依然可能触发这个问题。

另外，MethodOne中批量创建Task但未等待所有任务完成，可能导致程序提前退出时部分任务还没执行完SaveList，这属于任务生命周期问题，和anotherList本身的线程安全无关。

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