CheckListBox CheckedItemsを削除する

Question

私はいくつかのことを行うタイマーイベントを持っています。私がそれを達成しようとしている1つのアイテムは、タイマーが私が実行している完了したアクションにヒットしたときにチェックされるCheckListBoxアイテムをプログラムで削除することです。

これはタイマーのコードで、私が何をしようとしているのですか？

private void timer1_Tick(object sender, EventArgs e) 
{ 
    string s; 
    if (DbFirmwareUpdateComplete.WaitOne(1)) 
    { 
     DbFirmwareUpdateComplete.Reset(); 
     mnuLoadKeyFile.Enabled = true; 
    } 

    if (DbUpdateComplete.WaitOne(1)) 
    { 
     DbUpdateComplete.Reset(); 
     mnuLoadKeyFile.Enabled = true; 
     btnLoad.Enabled = true; 
    } 
    if (CacheComplete.WaitOne(1)) 
    { 
     CacheComplete.Reset(); 
     btnLoad.Enabled = true; 
    } 
    if (UpdateRunning) 
    { 
     bool UpdateDone = true; 
     int StillActive = 0; 
     // loop through all active jobs to check if all have completed 
     foreach (clsCnaPair cna in ActiveJobs) 
     { 
      if (cna.Job.JobComplete == false) 
      { 
       UpdateDone = false; 
       StillActive++; 
      } 

      else 
      { 
       if (cna.Job.UpdateSuccess) 
       { 
        // Update color of CLB.Items.Selected if success. 
        int count = CLB.Items.Count; 
        for (int index = count; index > 0; index--) 
        { 
         if(CLB.CheckedItems.Contains(CLB.Items[index-1])) 
         { 
          CLB.Items.RemoveAt(index - 1); 
         } 
        } 
       } 
       else 
       { 
        // Update color of CLB.Items.Selected if failed. 
       } 
      } 
     } 
     if (UpdateDone) 
     { 
      UpdateRunning = false; 
      log("All Update jobs have finished."); 
     } 
     if (ckTop.Checked == true) 
     { 
      ckTop.Checked = false; 
     } 
     else 
     { 
      ckTop.Checked = false; 
     } 

私はこのプログラムを実行し、このピースにヒットします。

 if (cna.Job.UpdateSuccess) 
     { 
      // Update color of CLB.Items.Selected if success. 
      int count = CLB.Items.Count; 
      for (int index = count; index > 0; index--) 
      { 
       if(CLB.CheckedItems.Contains(CLB.Items[index-1])) 
       { 
        CLB.Items.RemoveAt(index - 1); 
       } 
      } 
     } 

私はエラーを取得する：

System.ArgumentOutOfRangeException: InvalidArgument=Value of '-1' is not valid for 'index'. Parameter name: index

エラーコードのこの部分の後に発生します。

ラインで

private void CLB_SelectedIndexChanged(object sender, EventArgs e) 
{ 
    // One of the CNA IPs was selected. sender is the CheckedListBox. 
    // Here we want to display its fingerprint in the text box, or if the push is running, the status. 

    // get the CnaPair class represented by this IP: 
    clsCnaPair CnaPair = (clsCnaPair)CLB.Items[CLB.SelectedIndex]; 

    // Display the corresponding fingerprint string in the editBox: 
    if (CnaPair.Job != null) txtStatus.Text = CnaPair.Job.GetStatus(); 
    else txtStatus.Text = CnaPair.GetInfo(); 
} 

以上、特に：

clsCnaPair CnaPar = (clsCnaPair)CLB.Items[CLB.SelectedIndex]; 

私は何をしないのですか？ Googleを検索して、私が削除をやっている方法を示して、そこに見つかった例と一致しています。

おかげで、

Answer 1

ループの条件がChecklistBoxの内容に依存する場合、ChecklistBoxの内容をループ内で変更することは危険です。 RemoveAt（）を呼び出すと、CheckedItemsリストとCLB.Items.Countが変更され、問題が発生します。この場合、ループは無効なインデックス（-1）でSelectedIndexChanged（）イベントを発生させました。

良いがdo-whileループでこれを行うには：

bool done; 
do 
{ 
    done = true; 
    for (int index = CLB.Items.Count; index > 0; index--) 
    { 
     if(CLB.CheckedItems.Contains(CLB.Items[index-1])) 
     { 
      CLB.Items.RemoveAt(index - 1); 
      done = false; 
      break; 
     } 
    } 

}while(!done); 

この方法では、アイテムが削除されるたびに、あなたが抜け出すと、すべての繰り返しループを開始します。

Answer 2

いくつかの実験の後、私はCLB_SelecteIndexChangedコードをコメントアウトし、それは今、元のコードで完了します。

これは1つの問題を残します。残っているCLB_SelectedIndexChangedコードの回避策は何ですか？私はもう1つのことを行い、あなたが提供してくれたものを理解することができます。

m.rogalskiとmcNetsの両方に感謝します。