C#线程同步问题

Question

所以我看到了使用线程进行有趣的事情。

我有一个启动 2 个线程并继续其生命的方法，我尝试执行以下操作以确保安全。

    bool CopyOk1 = false;
    bool CopyOk2 = false;
    new Thread(() => FirstMethod(tempList1, ref CopyOk1)).Start();
    new Thread(() => SecondMethod(tempList2, ref CopyOk2)).Start();
    var spinner = new SpinWait();
    while (!CopyOk1 || !CopyOk2)
        spinner.SpinOnce();

然后在这两种方法中我都从以下内容开始

    private static void FirstMethod(List<T> templist, ref bool CopyOk)
    {
        var temp = new T[templist.Count];
        templist.CopyTo(temp);
        CopyOk = true;
        //moves on
    }

我的目的是复制传递的列表，以便我可以尽快在线程内安全地更改和使用它以解锁调用者线程。

我的问题是，在第二种方法中，在数组初始化和列表 CopyTo 之间，基本列表会以某种方式发生变化。

这些列表是在调用者线程内创建的，并且在线程启动后并未实际使用，因此我完全不知道这是如何/为什么发生的。

毫无疑问，我在这里做错了什么，但我的菜鸟技能不会让我看到，任何帮助都是值得赞赏的。

Answer 1

使用全局锁而不是锁定“templist”。然后使用同一锁保护所有修改源列表的代码。

另外，也许旁白，但等待线程部分完成可以更优雅地完成，即不涉及循环。使用 ManualResetEvent 将允许您的原始线程等待其他线程中发生“某些事情”。使用 WaitOne 将阻塞线程，直到在另一个线程中调用 Set()。

private object m_Lock = new object();
private ManualResetEvent m_ResetOne = new ManualResetEvent(false);
private ManualResetEvent m_ResetTwo = new ManualResetEvent(false);

(...)

new Thread(() => FirstMethod(tempList1)).Start();
new Thread(() => SecondMethod(tempList2)).Start();
m_ResetOne.WaitOne();
m_ResetTwo.WaitOne();

(...)

private void FirstMethod(List<T> templist)
{
    lock (m_Lock)
    {
        var temp = new T[templist.Count];
        templist.CopyTo(temp);
    }
    m_ResetOne .Set();
    //moves on
}

Answer 2

您发布的代码中没有任何内容表明您做错了什么。问题必须出在其他地方——正如评论者所建议的，最有可能的是在填充 List1/List2 的代码中。如果您也在线程化它，也许您不会等待该线程完成填充列表然后再继续？

Answer 3

尝试这个并检查行为

private static void FirstMethod(List<T> templist, ref bool CopyOk)
{
    T[] temp;

    lock (templist)
    {
        temp = new T[templist.Count];
        templist.CopyTo(temp);
    }
    CopyOk = true;
    //moves on
}

Answer 4

您需要某种类型的同步机制来防止 List 被更改。我们在当前示例中看不到足够的代码来保证它没有被更改。