Question

我有一个修改我的listview＆＃34; inputList＆＃34;的功能，listview有5列。该函数应该检查列表视图中的单元格，最后一列中的单元格将具有＆＃34; NORMAL＆＃34;有绿色背景，如果没有错，如果有什么不对，它将有＆＃34; ERROR＆＃34;在最后一列的单元格中使用红色背景，并且还更改其他列中同一行中错误的单元格的颜色。

有4k项目，我知道拥有这么多项目是没有意义的，因为用户不会阅读它们，但我被问到这样。

我在背景工作中使用了一个功能但速度不够快，并认为parallelFor会更快。但是当我尝试使用parallelFor时，它会冻结程序。

这是backgroundworker的功能，但它太慢了：

private void bw3_DoWork(object sender, DoWorkEventArgs e)
    {
        String tempTrain = "";
        String tempPredic = "";
        String tempNet = "";
        Boolean bTrain;
        Boolean bPredic;
        Boolean bNetErro;
        int n;
        int nNet = 0;
        String tTrain = "";
        String tPredic = "";
        int nList = 0;
        this.Invoke(new MethodInvoker(() => { 
            nList = inputList.Items.Count; 
            nNet = menuNetwork.Items.Count;
            tTrain = dtrainTextBox.Text;
            tPredic = dpredicTextBox.Text;
        }));
        for (int i = 0; i < nList; i++)
        {
            ListViewItem.ListViewSubItem temp1 = new ListViewItem.ListViewSubItem();
            temp1.BackColor = Color.LightGreen;
            temp1.Text = "NORMAL";
            this.Invoke(new MethodInvoker(() =>
            {
                inputList.Items[i].SubItems[0].BackColor = Color.White;
                inputList.Items[i].SubItems[1].BackColor = Color.White;
                inputList.Items[i].SubItems[2].BackColor = Color.White;
                tempTrain = String.Format("{0}\\{1}", tTrain, inputList.Items[i].SubItems[1].Text);
                tempPredic = String.Format("{0}\\{1}", tPredic, inputList.Items[i].SubItems[2].Text);
                tempNet = (String)inputList.Items[i].SubItems[0].Tag;
            }));
            bTrain = (!File.Exists(tempTrain));
            bPredic = (!File.Exists(tempPredic));
            bNetErro = !(int.TryParse(tempNet, out n));       
            if (!bNetErro)
            {
                if (!(n < nNet))
                {
                    bNetErro = true;
                }
            }
            this.Invoke(new MethodInvoker(delegate
            {
                if (bTrain) inputList.Items[i].SubItems[1].BackColor = Color.Red;
                if (bPredic) inputList.Items[i].SubItems[2].BackColor = Color.Red;
                if (bNetErro) inputList.Items[i].SubItems[0].BackColor = Color.Red;
                if (bTrain | bPredic | bNetErro) { temp1.Text = "Erro"; temp1.BackColor = Color.Red; }
                try { inputList.Items[i].SubItems[4] = temp1; }
                catch (ArgumentOutOfRangeException) { inputList.Items[i].SubItems.Add(temp1); }
            }));
        }
    }

这是ParallelFor的功能，即使我只尝试一个非常小的例子，程序也会停止工作。

 private void tent1()
    {
        ParallelOptions options = new ParallelOptions();
        options.MaxDegreeOfParallelism = 4;

        String tempTrain = "";
        String tempPredic = "";
        String tempNet = "";
        String tTrain = dtrainTextBox.Text;
        String tPredic = dpredicTextBox.Text;
        Boolean bTrain;
        Boolean bPredic;
        Boolean bNetErro;
        int n;

        int nNet = networkList.Items.Count;
        Parallel.For(0, inputList.Items.Count,
               i =>
        {
            ListViewItem.ListViewSubItem temp1 = new ListViewItem.ListViewSubItem();
            temp1.BackColor = Color.LightGreen;
            temp1.Text = "NORMAL";
            Console.WriteLine(i);
            this.Invoke(new MethodInvoker(() =>
            {
                inputList.Items[i].SubItems[0].BackColor = Color.White;
                inputList.Items[i].SubItems[1].BackColor = Color.White;
                inputList.Items[i].SubItems[2].BackColor = Color.White;
                tempTrain = String.Format("{0}\\{1}", tTrain, inputList.Items[i].SubItems[1].Text);
                tempPredic = String.Format("{0}\\{1}", tPredic , inputList.Items[i].SubItems[2].Text);
                tempNet = (String)inputList.Items[i].SubItems[0].Tag;
            }));
            bTrain = (!File.Exists(tempTrain));
            bPredic = (!File.Exists(tempPredic));
            bNetErro = !(int.TryParse(tempNet, out n));
            if (!bNetErro)
            {
                if (!(n < nNet))
                {
                    bNetErro = true;
                }
            }
            this.Invoke(new MethodInvoker(delegate
            {
                if (bTrain) inputList.Items[i].SubItems[1].BackColor = Color.Red;
                if (bPredic) inputList.Items[i].SubItems[2].BackColor = Color.Red;
                if (bNetErro) inputList.Items[i].SubItems[0].BackColor = Color.Red;
                if (bTrain | bPredic | bNetErro) { temp1.Text = "Erro"; temp1.BackColor = Color.Red; }
                try { inputList.Items[i].SubItems[4] = temp1; }
                catch (ArgumentOutOfRangeException) { inputList.Items[i].SubItems.Add(temp1); }
            }));
        });
    }

如何在不参考的情况下将列表视图的所有项目复制到列表中？我认为直接在listview中添加/修改调用会降低函数的速度，因此创建：List<ListViewItem> tList

我会修改tList中的所有内容然后我会使用inputList.Items.AddRange(tList.ToArray()); 这将删除循环内的所有调用。

    ListViewItem[] tItemsTemp = null;

this.Invoke(new MethodInvoker(() =>
            {
                tItemsTemp = new ListViewItem[inputList.Items.Count];
                inputList.Items.CopyTo(tItemsTemp, 0);
                nList = inputList.Items.Count;
                nNet = menuNetwork.Items.Count;
                tTrain = dtrainTextBox.Text;
                tPredic = dpredicTextBox.Text;
            }));
            List<ListViewItem> tList = new List<ListViewItem>(tItemsTemp);
            ListViewItem[] tItems = (ListViewItem[]) tItemsTemp.Clone();

    //Modifies the list or array of listviewitems

this.Invoke(new MethodInvoker(delegate 
    { 
        inputList.Items.Clear(); 
        // Just one of them,not the 3,just showing how i would call them.
        inputList.Items.AddRange(tItems);
        inputList.Items.AddRange(tItemsTemp);
        inputList.Items.AddRange(tList.ToArray());
    }));

但是tItemsTemp，tItems，tList都是引用... 如何通过创建参考进行复制？

Answer 1

不保证混合UI和并行化可以提供良好的结果。 UI本身就是一个瓶颈，因为它运行在一个独特的线程上。更糟糕的是，你的包括一些磁盘IO操作（File.Exists）。

您可能还想尝试一些事情：

用inputList.BeginUpdate和inputList.EndUpdate封装整个循环。这可以防止每次添加项目时刷新列表。
尝试SynchronizationContext和Post方法，而不是旧式的Invoke。这可能会使UI调用更加顺畅。请参阅this answer以查看其工作原理。

顺便说一句，你可以删除第一个Invoke，如果你可以在第二个Invoke中移动以。backcolor = Color.White结尾的三行。

如果不是更好，那么尝试分离构建项目并显示它们的过程：

使用并行化填充要更新的项目列表（不是使用ForEach，而是针对LINQ查询使用.AsParallel（））
使用循环更新ListView。

如果这不满意，您可能必须在虚拟模式下使用ListView找到解决方法。 This answer给出了一些提示，以异步方式填充虚拟ListView。

使用ParallelFor更改listview

1 个答案: