0

我有一个修改我的列表视图“inputList”的函数，列表视图有 5 列。该功能应该检查列表视图中的单元格，如果没有错误，最后一列中的单元格将具有绿色背景的“NORMAL”，如果有问题，它将在单元格中显示带有红色背景的“ERROR”在最后一列中，并更改其他列中同一行中错误单元格的颜色。

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

我正在使用带有 backgroundworker 的函数，但它不够快，并且认为 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); }
            }));
        });
    }

如何在没有参考的情况下将列表视图的所有项目复制到列表中？我认为直接在列表视图中添加/修改，并且调用会减慢函数的速度，因此创建：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 都是引用...如何通过创建引用进行复制？

4

1 回答 1

0

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

无论如何，您可能想尝试一些事情：

  • 用 inputList.BeginUpdate 和 inputList.EndUpdate 封闭整个循环。这可以防止列表在您每次添加项目时刷新。

  • 尝试SynchronizationContext和 Post 方法，而不是老式的 Invoke。这可能会使 UI 调用更流畅。请参阅此答案以检查其工作原理。

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

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

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

如果这不令人满意，您可能不得不在虚拟模式下使用 ListView 找到解决方法。这个答案给出了一些提示，以异步方式填充虚拟 ListView。

于 2015-09-23T19:43:39.867 回答