ListViewの名前が正しく変更されない

Question

list viewには、プレフィックス用と接続文字列用の2つの列があります。

例：

Prefix| Connection string 
-------------------------- 
OR1_ | blablabla 
-------------------------- 
OR2_ | blebleble 
-------------------------- 
OR3_ | blublublu 
-------------------------- 

（その場しのぎのテーブルのため申し訳ありませんが、私は理解しやすくなると思う）

私はListViewコントロールから項目を削除するたびだから今、私はすべてのプレフィックスの名前を変更する必要があります私は項目番号2を削除した場合、私はで終わるだろうと想像：

Prefix| Connection string 
-------------------------- 
OR1_ | blablabla 
-------------------------- 
OR3_ | blublublu 
-------------------------- 

しかし、私は実際にそれがOR1_とOR2_

する必要があります

ので、私は、私はアイテムを削除するときことない、このコード作られた：私は、リスト上の最初アイテムを削除する場合

private void buttonDel_Click(object sender, EventArgs e) 
     { 
      if (listViewConnectionStrings.Items.Count == 0) return; 
      else 
      { 
       //Delete the items 
       foreach (ListViewItem eachItem in listViewConnectionStrings.SelectedItems) 
       { 
        Connections.Remove(eachItem.Text); 
        listViewConnectionStrings.Items.Remove(eachItem); 
       } 
       if(listViewConnectionStrings.Items.Count == 0) return; 

       //Rename the prefixes 
       int cntItems = listViewConnectionStrings.Items.Count; 
       for (int i = 0; i <= cntItems; i++) 
       { 
        ListViewItem newItem = new ListViewItem(); 
        newItem = listViewConnectionStrings.Items[0]; 
        int newPrefix = i++; 
        newItem.Text = @"OR" + newPrefix.ToString() + @"_"; 
       } 

      } 
     } 

を最後forが正常に動作しているが、それは、適切に他のすべての名前を変更しますが、途中か下にあるものを削除すると、最初のものではないものもあります。他のものはすべて同じ名前になります。

何が間違っていますか、何が間違っていますか？

Answer 1

あなたはnewPrefixにそれを割り当て、このとき私はあなたのコードで見る最初の事は、forループでは、2番目のブロックでは、最初のループと第二のbegginingで、i 2回増加しているということです問題がある可能性が：あなたは私がnewPrefixに割り当てる際に望ましくない動作を挿入するために、ではない使用すべきか

  int cntItems = listViewConnectionStrings.Items.Count; 
      for (int i = 0; i <= cntItems; i++)//here you do i++ 
      { 
       ListViewItem newItem = new ListViewItem(); 
       newItem = listViewConnectionStrings.Items[0]; 
       int newPrefix = i++;//here you increment i again 
       newItem.Text = @"OR" + newPrefix.ToString() + @"_"; 
      } 

があることi++の代わりにi+1を使用しています。

また、私はあなたのlistViewConnectionStrings.Itemsをあなたのアイテムのインデックスを決して変更しないのであなたがどのように使用するのかを考えています。これはItem [0]のみを使用しています。両方の変更を次のようにしてもう一度試すことができます：

  int cntItems = listViewConnectionStrings.Items.Count; 
      for (int i = 0; i <= cntItems; i++)//here you do i++ 
      { 
       ListViewItem newItem = new ListViewItem(); 
       newItem = listViewConnectionStrings.Items[i]; 
       int newPrefix = i+1;//here you increment i again 
       newItem.Text = @"OR" + newPrefix.ToString() + @"_"; 
      }