ListViewの複数項目削除で意図しない挙動

質問

前回質問した内容でほとんど複数削除は解決したように
見えたのですが、どういう条件か分からないのですが
複数選択(1件選択の場合は発生しない)して削除ロジックを
実行すると選択している項目と異なる内容の配列(ArrayList)の
データを削除してしまうという事が発生します。

これは何が原因で発生しているのでしょうか？
またこれの解決法をご教授頂けないでしょうか。

環境はVS2005(C#)、.NET Framework2.0です。

以下に該当部分のコードを載せます。
2000文字を超えてしまったので問題の部分以外は省略しました。

※添付画像が見にくかったので以下のURLに再掲示しましたので
参考にしてもらえるとありがたいです。
http://angelteatime.punyu.net/questionbox_20090312.jpg

画像では350件ある受信メール(ArrayListに1件ずつMailクラスで
格納されている)の先頭から8件ぐらい選択削除をしようとしている
画面です。

private void menuDelete_Click(object sender, EventArgs e)
{
// 選択アイテムが0のときは反応にしない
if(listView1.SelectedItems.Count == 0){
return;
}

if(listView1.Columns[0].Text == "差出人"){
// 受信メールのとき
for (int sel_index = listView1.Items.Count - 1; sel_index > -1; sel_index--) {
if (listView1.Items[sel_index].Selected) {
int nIndex = (int)listView1.Items[sel_index].Tag;
if (listView1.Items[sel_index].SubItems[1].Text == ((Mail)collectionMail[RECEIVE][nIndex]).subject) {
collectionMail[DELETE].Add(collectionMail[RECEIVE][nIndex]);
collectionMail[RECEIVE].RemoveAt(nIndex);
}
else{
MessageBox.Show(string.Format("選択した件名と削除するデータが異なっています。\nリスト位置件名:{0}\n実位置件名:{1}\n(リスト位置:{2}, 実位置:{3}", listView1.Items[sel_index].SubItems[1].Text, ((Mail)collectionMail[RECEIVE][nIndex]).subject, sel_index, nIndex), "エラー", MessageBoxButtons.OK, MessageBoxIcon.Warning);
}
}
}
this.textBody.Text = "";
}
else if(listView1.Columns[0].Text == "宛先"){

(上記のRECEIVEがSENDに変更のみでロジック変更なし)

}
else if(listView1.Columns[0].Text == "差出人または宛先"){

(上記のDELETEに変更されDELETEの配列に格納されたデータを削除)

}
// ツリービューとリストビューの表示を更新する
UpdateTreeView();
UpdateListView();

(ListViewのフォーカスを設定し直す)

}

回答No.2

redfox63 · ベストアンサー

ごめんなさい
IndexOfではなく IndexKeyOfでした

int nIndex = listView1.SelectedItems.IndexKeyOf( nIndices[nLen-1].ToString() );

お礼

どおりでコンパイルエラーなわけでした。
という事で何とかこの問題も無事？解決致しました。
本当にありがとうございます。

とりあえずArrayListの問題は解決しましたが、他の機能で
困ったものがあったので別項目で質問致します。

回答No.1

redfox63

原因はListViewに登録したアイテムが最初からソートされてしまって
Tagに登録したインデックスと食い違っているためです

登録時にListViewItemのNameプロパティを設定します
ListViewItem item = new ListViewItem( mail.Subject );
item.name = n.ToString();
// 他の項目の設定は省略

削除処理の中で
// 選択アイテムの数を取得
int nLen = listView1.SelctedItems.Count;
if ( nLen == 0 )
　　return;
// 選択アイテムのキーを取得
int[] nIndices = new int[nLen];
for( int n = 0; n < nLen; n++ ) {
　　nIndices[n] = int.Parse( listView1.SelectedItems[n].Name );
}
// キーの並べ替え
Array.Sort( nIndices );
ArrayList sList = collectionMail[RECIVE];
ArrayList dList = collectionMail[DELETE];

while( nLen > 0 )
{
　　// 選択アイテムのキーから 選択アイテム群の位置を取得
　　int nIndex = listView1.SelectedItems.IndexOf( nIndices[nLen-1].ToString() );
　　ListViewItem item = listView1.SelectedItems[ nIndex ];
　　// 元リストからメールアイテムを取得
　　mail mail = (mail)sList[nIndices[nLen-1]];
　　if ( item.Text == mail.suject ) {
　　　　dList.add( mail );
　　　　sList.Remove( mail );
　　} else {
　　　　MesageBox.Show( item.Text + "\n" + mail.Subject );
　　}
　　nLen--;
}

といった具合でしょう

補足

IndexOfのメソッドでエラーが出てしまい先に進めません。
IndexOfの引数はListViewItemのようなのですが・・・。

あとcollectionMail[RECEIVE]、collectionMail[DELETE]の
内容って正しく反映されるのでしょうか？

作りかけのコードを載せておきます。
public void UpdateListView()
{
ArrayList list = null;
int i = 0;

listView1.BeginUpdate();

// リストビューの内容をクリアする
listView1.Items.Clear();

(listに受信、送信、削除のArrayListを代入する)

foreach (Mail mail in list){
ListViewItem item = new ListViewItem(mail.address);
if(mail.subject != ""){
item.SubItems.Add(mail.subject);
}
else{
item.SubItems.Add("(no subject)");
}
item.SubItems.Add(mail.date);
item.SubItems.Add(mail.size);

// 各項目のタグに要素の番号を格納する
item.Tag = i; // メールデータ表示用
item.Name = i.ToString();

// 未読(未送信)の場合は、フォントを太字にする
if (mail.notReadYet == true) {
item.Font = new Font(this.Font, FontStyle.Bold);
}

i++;

listView1.Items.Add(item);
}
listView1.EndUpdate();
}

private void menuDelete_Click(object sender, EventArgs e)
{
// 受信メールのとき
if(listView1.Columns[0].Text == "差出人"){
// 選択アイテムの数を取得
int nLen = listView1.SelectedItems.Count;

if(nLen == 0)
return;

// 選択アイテムのキーを取得
int[] nIndices = new int[nLen];

for(int n = 0; n < nLen; n++){
nIndices[n] = int.Parse(listView1.SelectedItems[n].Name);
}

// キーの並べ替え
Array.Sort(nIndices);
ArrayList sList = collectionMail[RECEIVE];
ArrayList dList = collectionMail[DELETE];

while (nLen > 0) {
// 選択アイテムのキーから 選択アイテム群の位置を取得
// ↓ここの行でコンパイルエラー
int nIndex = listView1.SelectedItems.IndexOf(nIndices[nLen - 1].ToString());
ListViewItem item = listView1.SelectedItems[nIndex];

// 元リストからメールアイテムを取得
Mail mail = (Mail)sList[nIndices[nLen - 1]];

if(item.Text == mail.subject){
// これってcollectionMail[RECEIVE]に反映されるの？
dList.Add(mail);
// これってcollectionMail[DELETE]に反映されるの？
sList.Remove(mail);
}
else{
MessageBox.Show(item.Text + "\n" + mail.subject);
}
nLen--;
}
}
else if(listView1.Columns[0].Text == "宛先"){
// 送信メールのとき
}
else if(listView1.Columns[0].Text == "差出人または宛先"){
// 削除メールのとき
}
}

// 表示されている本文を削除
this.textBody.Text = "";

// ツリービューとリストビューの表示を更新する
UpdateTreeView();
UpdateListView();
}