使用后台工作程序动态更新DataGrid

时间:2012-11-21 13:21:18

标签: c# multithreading backgroundworker

我有一个ObservableCollection自定义类,它包含一个字符串和一个int:

public class SearchFile
{
    public string path { set; get; }
    public int occurrences { set; get; }
}

我想在dataGrid中显示该集合。该集合具有在更新时通知的方法,因此到目前为止,只需将其链接到DataGrid.ItemsSource(正确吗?)。这是网格XAML(在C#代码隐藏中使用dataGrid1.ItemsSource = files;):

        <DataGrid AutoGenerateColumns="False" Height="260" Name="dataGrid1" VerticalAlignment="Stretch" IsReadOnly="True" ItemsSource="{Binding}" >
            <DataGrid.Columns>
                <DataGridTextColumn Header="path" Binding="{Binding path}" />
                <DataGridTextColumn Header="#" Binding="{Binding occurrences}" />
            </DataGrid.Columns>
        </DataGrid>

现在事情变得更复杂了。我首先要显示path s,默认值为occurrence为零。然后,我想查看每个SearchFile并使用计算值occurrence更新它。这是辅助函数:

    public static void AddOccurrences(this ObservableCollection<SearchFile> collection, string path, int occurrences)
    {
        for(int i = 0; i < collection.Count; i++)
        {
            if(collection[i].path == path)
            {
                collection[i].occurrences = occurrences;
                break;
            }
        }
    }

这是占位符工作者功能:

    public static bool searchFile(string path, out int occurences)
    {
        Thread.Sleep(1000);
        occurences = 1;
        return true; //for other things; ignore here     
    }

我使用BackgroundWorker作为后台主题。方法如下:

    private void worker_DoWork(object sender, DoWorkEventArgs e)
    {           
        List<string> allFiles = new List<string>();
        //allFiles = some basic directory searching

        this.Dispatcher.Invoke(new Action(delegate
        {
            searchProgressBar.Maximum = allFiles.Count;
            files.Clear(); // remove the previous file list to build new one from scratch
        }));

        /* Create a new list of files with the default occurrences value. */
        foreach(var file in allFiles)
        {
            SearchFile sf = new SearchFile() { path=file, occurrences=0 };
            this.Dispatcher.Invoke(new Action(delegate
            {
                files.Add(sf);
            }));
        }

        /* Add the occurrences. */
        foreach(var file in allFiles)
        {
            ++progress; // advance the progress bar
            this.Dispatcher.Invoke(new Action(delegate
            {
                searchProgressBar.Value = progress;
            }));

            int occurences;
            bool result = FileSearcher.searchFile(file, out occurences);

            files.AddOccurrences(file, occurences);
        }
    }

现在,当我运行它时,有两个问题。首先,更新进度条的值会引发The calling thread cannot access this object because a different thread owns it.异常。为什么?这是一个调度员,所以它应该工作得很好。第二,foreach循环中bool result =...循环中断。我评论它并尝试设置int occurences = 1,然后循环转过来,但是有一些奇怪的事情发生:每当我调用方法时,它都是全零,全部或状态之间,之后开始一个一个看似随机数的零。)

为什么?

1 个答案:

答案 0 :(得分:0)

Dispatcher并不像听起来那么简单。使用调度程序意味着代码将在创建包含对象的同一线程上执行,但这不一定是UI线程。确保在屏幕上显示的实际UI元素中定义了worker_DoWork方法(这只是消除后台线程创建的元素的简单方法)。

真的,看看你的代码,我不确定你为什么要使用后台工作者。这看起来实际上会更慢,因为不断调度到UI线程。相反,我认为更好的选择是将长时间运行的部分放在任务中,并在单个回调中更新UI。例如,在你的代码中,对于UI线程而言,实际上只有一个实际上太慢的东西可能是FileSearcher调用。这很容易放入后台任务,返回找到的结果数。

至于FileSearcher上的问题,您的方法定义不匹配。您发布的方法只接受一个路径和out int,但是当您调用它时,您传递4个参数。没有看到你实际上称之为过载的过载,很难猜出出了什么问题。

编辑:让我再补充一点。您的根本问题只是WPF不支持绑定到从后台线程修改的集合。这就是所有调度和其他复杂代码的原因。我最初的建议(长期工作的单一任务)是解决问题的一种方法。但你可以使用ObservableList class做得更好。这使您可以从后台线程更新集合,并自动通知UI而无需使用调度程序。这可能会使大多数复杂的线程问题消失。我建议阅读前三篇文章以供参考,那里有很多很好的信息。