- PR -

ListViewのCheckedItemsが正しく取得できない

1
投稿者投稿内容
べる
ぬし
会議室デビュー日: 2003/09/20
投稿数: 1093
投稿日時: 2003-09-22 17:26
初めて投稿させていただきます。

VisualStudio.netを使ってC#でWindowsアプリケーションを作っています。

ListViewクラスのCheckedItemsプロパティはListViewコントロールでチェックされているすべての項目を含むコレクションを返します。
ListViewはCheckBoxesプロパティをtrueにすると各行の左側にチェックボックスが
配置されます。ここでいう「チェックされている」とはそのことです。
ところが以下の操作を行ってもListViewのCheckedItemsが正しく取得できません。


ListViewコントロール(listView1)とButtonコントロール(button1)を配置します。

listView1のCheckBoxesプロパティをtrueにし、ViewプロパティをListにします。

listView1に以下のように３つのListViewItemを追加します。
listView1.Items.Add("１行目");
listView1.Items.Add("２行目");
listView1.Items.Add("３行目");

//button1が押されたときのイベントハンドラを以下のように記述します。
private void button1_Click(object sender, System.EventArgs e)
{
//listView1でチェックされている最初の行のTextを表示する
MessageBox.Show(listView1.CheckedItems[0].Text);
}

実行時に
１、はじめは３行目だけにチェックしてbutton1を押す
→メッセージボックスで「３行目」と表示される
２、次に１行目だけにチェックしてbutton1を押す
→メッセージボックスで「３行目」と表示される

１行目だけにチェックしているのに「３行目」と出てしまいます。

このことに関して何かおわかりの方、ご教授願います。

環境は WindowsXP pro、Visual Studio.net 2002です。
べる
ぬし
会議室デビュー日: 2003/09/20
投稿数: 1093
投稿日時: 2003-09-22 18:07
自己レスです。

ちなみに、イベントハンドラの部分を以下のようにすると

private void button1_Click(object sender, System.EventArgs e)
{
　　for(int i=0;i<listView1.CheckedItems.Count;i++)
　　{
　　　　MessageBox.Show(listView1.CheckedItems[i].Text);
　　}
}

実行時
１、最初は１行目と２行目にチェックしてbuttun1を押す
→「１行目」「２行目」の順でメッセージボックスが表示される
２、次に２行目と３行目にチェックしてbuttun1を押す
→「２行目」「３行目」の順でメッセージボックスが表示される

のように正常です。

しかし、
１、最初は３行目だけにチェックしてbuttun1を押す
→メッセージボックスで「３行目」と表示される
２、次に１行目と２行目にチェックしてbuttun1を押す
→メッセージボックスで「３行目」と表示されたあと
for文の２週目で「指定された引数は、有効な値の範囲内にありません。」
のエラーになります。

３行目だけにチェックした場合も、１行目と２行目にチェックした場合も
listView1.CheckedItems.Countの値は正しく取得できています。

また、foreach文で代用するとなぜか正常に動作します。

[ メッセージ編集済み 編集者: べる 編集日時 2003-09-22 18:08 ]
べる
ぬし
会議室デビュー日: 2003/09/20
投稿数: 1093
投稿日時: 2003-09-25 18:14
また自己レスです。

以下の冒頭で紹介されていた逆コンパイラを使ってライブラリの中を見てみました。
http://www.atmarkit.co.jp/fdotnet/tools/dotfuscator/dotfuscator_02.html

CheckedItems[n] は内部的に CheckedIndices[n] を参照しています。
ListView.CheckedIndexCollection (CheckedIndicesの戻り値クラス) の
インデクサ ( get_Item(int index) ) を見てみると以下のようになっています。

public int get_Item(int index)
{
　int num1;
　int num2;
　ListViewItem item1;
　if (index < 0)
　{
　　throw new ArgumentOutOfRangeException("index");
　}
　if ((this.lastIndex == -1) || (index < this.lastIndex))
　{
　　this.lastItemIndex = 0;
　　this.lastIndex = 0;
　}
　else
　{
　　if (index == this.lastIndex)
　　{
　　　return this.lastItemIndex;
　　}
　}
　num1 = this.owner.Items.Count;
　for (num2 = this.lastItemIndex; (num2 < num1); num2 = (num2 + 1))
　{
　　item1 = this.owner.Items[num2];
　　if (item1.Checked)
　　{
　　　if (this.lastIndex == index)
　　　{
　　　　this.lastItemIndex = num2;
　　　　return num2;
　　　}
　　　this.lastIndex = (this.lastIndex + 1);
　　}
　}
　throw new ArgumentOutOfRangeException("index");
}


lastIndex、lastItemIndex はCheckedIndexCollection のメンバ変数で共に CheckedIndexCollection のコンストラクタで-1 に設定されているようです。
１回目と２回目のボタン押下(１回目と２回目のCheckedItems呼び出し)で同じ
CheckedIndexCollectionインスタンスを参照しているなら(推測ですが)、
当然、おかしなこと(１回目の投稿に書いたこと)は起こります(上のget_Itemを
少し調べてみるとわかります)。

これはライブラリのバグではないのでしょうか。
1

スキルアップ／キャリアアップ（JOB@IT）