c＃ListView.Items [i] .removeが非常に遅い

Question

ここは初めてのことであり、この問題を解決するのに苦労しています。非常に遅いですlista.items[i].remove

try 
{ 
    progressBar1.Maximum = lista.Items.Count; 
    lista.BeginUpdate(); 

    for (int i = 0; lista.Items.Count > i; i++) 

    //for (int i = lista.Items.Count - 1; -1 < i; i--) 
    { 
     if (lista.Items[i].SubItems[1].Text.ToLower().Contains(Text) == false) 
     {       
      lista.Items[i].Remove();       
     } 

     progressBar1.Value = progressBar1.Value + 1; 
    } 

    lista.EndUpdate(); 

    progressBar1.Value = 0; 
} 
catch (Exception errore) 
{ 
    txt_info.Text = "" + errore.Message; 
    progressBar1.Value = 0; 
} 

方法： は、私はこのコードの部分を持っています。 listaはListViewで、50,000行以上のログファイルを作成しています。 処理を高速化する方法はありますか？

Answer 1

ListViewItem[] allElements = new ListViewItem[listView1.Items.Count]; 
listView1.Items.CopyTo(allElements, 0); 
List <ListViewItem> list = allElements.ToList(); 
list.RemoveAll(item => item.SubItems[1].Text.ToLower().Contains(TextToFind) == false); 
listView1.BeginUpdate(); 
listView1.Clear(); 
listView1.Items.AddRange(list.ToArray()); 
listView1.EndUpdate(); 

最初のルールは、forループの更新リストにはなりません。あなたのロジックはリストの半分までしか動かないでしょう。私はあなたが望むものではないと思う。
私は、BeginUpdateとEndUpdateを使ってもlistview.itemsを操作するのが非常に遅いことが分かりました。キーは、リスト内で（リスト内で）操作を行い、AddRange（Addよりはるかに速い）でリストを再作成することです。

Answer 2

は、私は別のアプローチを取るとLINQ、このようなものを使用します。

lista.Items = lista.Items.Where(x=>x.SubItems[1].Text.ToLower.Contains(Text)).AsParallel().ToList(); 

基本的には、リストを再構築し、一度ではなく、何度も何度も、個々のアイテムを削除しようとしています。

Answer 3

最も簡単なオプションは、リスト自体のRemoveAll methodを使用することです。

list.RemoveAll(x => !x.SubItems[1].Text.ToLower().Contains(Text))

P.S.

実際の比較で速度の向上を見たい場合があります。 String.Compareを使用すると、要件が適合した場合にはるかに高速です。サブストリングをチェックしたい場合は、不変関連の問題の場合はToUpperInvariantを使用することをお勧めします。it's designed to be faster

Answer 4

あなたはバックグラウンドワーカーにそれを貼り付けることができ、それを自分自身で行うことができます。したがって、このプロセスが発生している間もユーザーはプログラムを使用できます。