从线程获取错误的结果

时间:2013-03-14 13:57:16

标签: c# wpf multithreading file dispatcher

我有一个小WPF个应用程序需要枚举指定目录中的所有文件,并检查其中是否存在某个字符串。这是搜索方法:

private void btnSearch_Click_1(object sender, RoutedEventArgs e)
{
  Thread t = new Thread(()=>search(@"c:\t", "url", true));
  t.Start();
}

private void search(string path, string textToSearch, bool ignoreCase)
{
  foreach (string currentFile in Directory.EnumerateFiles(path, "*.*", SearchOption.AllDirectories))
  {
    int lineNumber = 0;
    foreach (string line in File.ReadLines(currentFile))
    {
      lineNumber++;
      if (line.Contains(textToSearch))
      {
        lbFiles.Dispatcher.BeginInvoke((Action)(() =>
        {
          //add the file name and the line number to a ListBox
          lbFiles.Items.Add(currentFile + "     " + lineNumber);
        }));
      }
    }
  }
}

我的问题是,如果在文件中多次找到指定的字符串,则所有出现的行号都是后者。对于包含以下行的文本文件:

ABCD
EFG
网址
hijk123
网址

listbox将如下所示:

ListBoxResult

当使用断点逐步执行代码时,我可以看到,在逐步退出搜索方法后,它会“跳转”回BeginInvoke声明。
请指教。
感谢

1 个答案:

答案 0 :(得分:1)

问题在于您正在关闭变量lineNumberBeginInvoke是异步的,它不等待在UI线程上调用委托。当它设法被调用时lineNumber已经增加了很多次。

有两种解决方案。创建一个更加本地化的lineNumber副本以便关闭,以便以后看不到更改:

foreach (string line in File.ReadLines(currentFile))
{
  lineNumber++;
  if (line.Contains(textToSearch))
  {
    var lineNumberCopy = lineNumber;
    lbFiles.Dispatcher.BeginInvoke((Action)(() =>
    {
      //add the file name and the line number to a ListBox
      lbFiles.Items.Add(currentFile + "     " + lineNumberCopy );
    }));
  }
}

或使用Invoke代替BeginInvoke,以便在lineNumber有机会增加之前读取{{1}}。