BeginInvoke 由于跳过了空检查而导致错误

Question

>我遇到了一个无法理解的错误，该错误是由于在多线程应用程序中使用Dispatcher.BeginInvoke而导致的。

我的程序包含List对象，我使用多个线程循环通过这些对象执行一些计算。我已经将我的代码结构简化(并略微修改(为最基本的内容，因此希望它更容易理解：

public class Foo
{
    public void DoCalc()
    {
        //do calculations
    }
}
public class Foo2
{
    public Foo foo = new Foo();
    public Ellipse ellipse;
}
public partial class MainWindow : Window
{
    List<Foo2> myList = new List<Foo2>();
    List<Tuple<int, int>> indicesList = new List<Tuple<int, int>>();
    public MainWindow()
    {
        InitializeComponent();
        for (int i = 0; i < 10; ++i)
            myList.Add(new Foo2 { ellipse = new Ellipse() });
        for (int i = 10; i < 20; ++i)
            myList.Add(new Foo2());
        indicesList.Add(new Tuple<int, int>(0, 9));
        indicesList.Add(new Tuple<int, int>(10, 19));
    }
    private void OnStart(object sender, RoutedEventArgs e)
    {
        foreach (var t in indicesList)
            ThreadPool.QueueUserWorkItem(new WaitCallback(Loop), t);
    }
    private void Loop(object o)
    {
        Tuple<int, int> indices = o as Tuple<int, int>;
        for(int i = indices.Item1; i <= indices.Item2; ++i)
        {
            myList[i].foo.DoCalc();
            if (myList[i].ellipse == null)
                continue;
            Application.Current.Dispatcher.BeginInvoke(DispatcherPriority.Normal, new Action(() => myList[i].ellipse.Fill = Brushes.Black));
        }
    }
}

myList中项目的前半部分ellipse指向实际对象，而后半部分指向null。在Loop内，我会在每次迭代中检查ellipse是否null，并在必要时跳过该迭代。奇怪的是，在所有省略号都指向 null 的第二个线程中，程序最终仍然对第一项(索引 10(调用 BeginInvoke 操作，导致程序崩溃，原因是空引用异常。

如果我在行myList[i].foo.DoCalc();上放置一个断点并逐步缓慢地浏览程序，则不会发生错误。此外，当我将BeginInvoke更改为Invoke时，不会发生任何错误。

据我了解，BeginInvoke 通过向调度程序发送请求以在某个时候执行来异步工作，而 Invoke 会阻止调用线程，直到调度程序执行请求。但是，由于我既不访问两个循环中的相同元素，也不更改列表本身的任何内容，因此我不明白多线程或 BeginInvoke 的异步性质如何以任何方式干扰我在这里所做的事情。

不幸的是，我只在 SO 上添加或删除项目时发现与 List s 相关的 BeginInvoke 错误，但是，当所有线程似乎始终访问不同的项目时，不会发生任何错误。如果我的代码有一些我根本不理解的基本问题(这可能会导致我的程序实际上无法运行(，那么请为我清除这个问题或向我发送一个我找不到的答案的链接。我已经在这个问题上呆了一整天了！

Answer 1

您正在遇到变量捕获。调用调用中使用的i可能indices.Item2 + 1而不是执行 BeginDescribe 时i值。必须将i复制到每个循环迭代新创建的局部变量中。

private void Loop(object o)
{
    Tuple<int, int> indices = o as Tuple<int, int>;
    for(int i = indices.Item1; i <= indices.Item2; ++i)
    {
        myList[i].foo.DoCalc();
        if (myList[i].ellipse == null)
            continue;
        int iLocal = i;
        Application.Current.Dispatcher.BeginInvoke(DispatcherPriority.Normal, new Action(() => myList[iLocal].ellipse.Fill = Brushes.Black));
    }
}

foreach之前C# 5遇到同样的问题，请参阅此处了解更多信息。

Answer 2

这可能与闭包有关...

试试这个：

  var current = myList[i].foo.DoCalc();
            if (current.ellipse == null)
                continue;    
            Application.Current.Dispatcher.BeginInvoke(DispatcherPriority.Normal, 
             new Action(() => current.ellipse.Fill = Brushes.Black));