Question

例如，我有一个列表视图，其中包含3个项目。每个项目包含一个文件夹路径列和一个列，用于计算该文件夹中文件数的计数。

如果我开始一个单独的背景工作者来计算每个文件夹中的文件，我会得到一个意想不到的结果。我想我已经跟踪了问题，但我不确定如何解决它。

在下面的示例中，我使用两种不同的方法计算了每个文件夹中的文件;第一种方法为每个文件夹创建一个后台工作程序，每个后台工作程序在计算文件时同时运行。第二种方法创建一个后台工作程序，用于计算每个文件夹中的文件。串联计数确实有效，而同时计数却没有。

问题似乎出现在方法GetPicturesConcurrently（）中，特别是在读取的行上：

fileCounter.DoWork += new DoWorkEventHandler((obj, e) => CountFilesInFolder(item.Text));

似乎正在发生的事实是，实际传递给CountFilesInFolder（string）的每次调用的字符串最终使用来自上一个创建的backgroundworker的字符串到达方法;好像来自item.Text的字符串是通过引用而不是按值传递的。所以我最后一遍又一遍地计算同一文件夹中的文件。

当我在创建backgroundworker时打破，我可以看到每次都传递正确的字符串;当我在CountFilesInFolder上打破时，每次调用都会处理输入的最后一个字符串。

以下是演示此问题的示例：

public partial class Form1 : Form
{
    private ConcurrentDictionary<string, int> MyFiles;
    private List<string> Folders;

    public Form1()
    {
        MyFiles = new ConcurrentDictionary<string,int>();
        Folders = new List<string>();

        InitializeComponent();
        PopulateListview();
    }

    private void PopulateListview()
    {
        ListViewItem item1 = new ListViewItem();
        ListViewItem item2 = new ListViewItem();
        ListViewItem item3 = new ListViewItem();

        item1.Text = @"V:\";
        item2.Text = @"D:\";
        item3.Text = @"C:\";

        item1.SubItems.Add("");
        item2.SubItems.Add("");
        item3.SubItems.Add("");

        listView1.Items.Add(item1);
        listView1.Items.Add(item2);
        listView1.Items.Add(item3);
    }



    private void GetPicturesInSeries()
    {
        Reset();

        foreach (ListViewItem item in listView1.Items)
        {
            Folders.Add(item.Text);
        }

        BackgroundWorker fileCounter = new BackgroundWorker();
        fileCounter.DoWork += new DoWorkEventHandler((obj, e) => GetPictures());
        fileCounter.RunWorkerCompleted += new RunWorkerCompletedEventHandler((obj, e) => UpdateCountListView());
        fileCounter.RunWorkerAsync();            
    }        

    private void GetPicturesConcurrently()
    {
        Reset();


        foreach (ListViewItem item in listView1.Items)
        {
            BackgroundWorker fileCounter = new BackgroundWorker();
            fileCounter.DoWork += new DoWorkEventHandler((obj, e) => CountFilesInFolder(item.Text));
            fileCounter.RunWorkerCompleted += new RunWorkerCompletedEventHandler((obj, e) => UpdateCountListView(item.Index));
            fileCounter.RunWorkerAsync();               
        }

    }

    private void GetPictures()
    {
        foreach (string folder in Folders)
        {
            CountFilesInFolder(folder);
        }
    }

    private void CountFilesInFolder(string folder)
    {
        DirectoryInfo dirInfo = new DirectoryInfo(folder);

        IEnumerable<FileInfo> files = dirInfo.EnumerateFiles();

        int count = files.Count();

        MyFiles.AddOrUpdate(folder, count, (key, oldvalue) => files.Count());

    }

    private void UpdateCountListView(int index)
    {
        string key = listView1.Items[index].Text;

        int count;
        MyFiles.TryGetValue(key,out count);

        listView1.BeginUpdate();
        listView1.Items[index].SubItems[1].Text = count.ToString();
        listView1.EndUpdate();
        listView1.Refresh();
    }

    private void UpdateCountListView()
    {
        listView1.BeginUpdate();

        foreach (ListViewItem item in listView1.Items)
        {
            string key = item.Text;

            int count;
            MyFiles.TryGetValue(key, out count);

            listView1.Items[item.Index].SubItems[1].Text = count.ToString();
        }

        listView1.EndUpdate();
        listView1.Refresh();
    }

    private void Reset()
    {
        listView1.BeginUpdate();
        foreach (ListViewItem item in listView1.Items)
        {
            item.SubItems[1].Text = "";
        }
        listView1.EndUpdate();
        listView1.Refresh();


        Folders.Clear();
        MyFiles.Clear();
    }
}

Answer 1

我认为您可能正在修改GetPicturesConcurrently()中的捕获变量，因此在使用之前将其更改为复制变量，如下所示：

private void GetPicturesConcurrently()
{
    Reset();

    foreach (ListViewItem item in listView1.Items)
    {
        var copy = item;
        BackgroundWorker fileCounter = new BackgroundWorker();
        fileCounter.DoWork += new DoWorkEventHandler((obj, e) => CountFilesInFolder(copy.Text));
        fileCounter.RunWorkerCompleted += new RunWorkerCompletedEventHandler((obj, e) => UpdateCountListView(copy.Index));
        fileCounter.RunWorkerAsync();               
    }
}

其次，您的CountFilesInFolder()可能会枚举所有文件两次：

private void CountFilesInFolder(string folder)
{
    DirectoryInfo dirInfo = new DirectoryInfo(folder);

    IEnumerable<FileInfo> files = dirInfo.EnumerateFiles();

    int count = files.Count();

    MyFiles.AddOrUpdate(folder, count, (key, oldvalue) => files.Count());
}

当您致电folder时，如果MyFiles已在AddOrUpdate，则会再次致电files.Count() - 这将再次枚举所有文件！

如果folder无法进入MyFiles，那么只需致电MyFiles.Add()而不是MyFiles.AddOrUpdate()

如果folder已经<{1}} ，则将其更改为：

MyFiles

了解使用多个背景工作者

1 个答案: