2

所以我有一个方法可以获取 List<myObj> 的字典，然后循环遍历字典的键并将每个 List<myObj> 传递给单独的线程。

这是一些代码/伪代码：

public static void ProcessEntries() {

    Dictionary<string, List<myObj>> myDictionary = GetDictionary();

    foreach(string key in myDictionary.keys)
    {

        List<myObj> myList = myDictionary[key];

        Thread myThread = new System.Threading.Thread(new System.Threading.ThreadStart(delegate() {

            ProcessList(myList);

        }    
    }
}

public static void ProcessList(List<myObj> myList) {

    // Process entries
    // read-only operations on myList

}

问题是在 ProcessList 的执行过程中 myList 参数只是改变了。

在启动线程之前，我已经遍历了列表，然后立即进入了线程，我发现结果有所不同。

我已经通过将 Dictionary 变量设为全局来解决了这个问题（我认为！）。使用 [ThreadStatic] 属性是可能的修复列表中的下一个。

我真正想知道的是，当 myList 对象在 ProcessEntries() 中重新分配时，为什么 myList 对象会在 ProcessList() 中发生变化？这不是两个不同的列表吗？如果默认情况下所有参数传递都是按值传递的，为什么 ProcessList() 函数没有 myList 的本地副本？（可以？）

有没有办法指定您想要将参数传递给线程并且在执行期间不让父线程或其他线程更改它？（这类似于全局变量的 [ThreadSafe] 属性）

4

3 回答 3

2

我怀疑您的伪代码实际上并不能准确反映您的真实代码。我怀疑您的真实代码如下所示：

foreach(var pair in myDictionary)
{
    Thread myThread = new Thread(delegate() {
        ProcessList(pair.Value);
    });
    myThread.Start();
}

如果是这种情况，问题在于pair变量正在被捕获 - 所以当你的线程启动时，它可能指的是不同的键/值对。

修复它的方法是使代码更像您的伪代码：

foreach(var pair in myDictionary)
{
    // You'll get a new list variable on each iteration
    var list = pair.Value;
    Thread myThread = new Thread(delegate() {
        ProcessList(list);
    });
    myThread.Start();
}

有关详细信息，请参阅Eric Lippert 的博客文章。

如果这不是问题所在，请给出一个真实的例子而不是伪代码。一个简短但完整的例子来证明这个问题将是理想的。

于 2010-08-05T17:02:25.587 回答
1

还要确保其他线程不会影响您尝试使用的线程。一定要使用锁和监视器......几周前有一些问题......

于 2010-08-06T19:27:12.303 回答
0

在这种情况下，您是按值传递引用，因此如果您在某处修改它，它会因时而异。

于 2010-08-05T16:53:05.260 回答