FormアプリケーションのListViewのSelectedIndexChangedのハンドラでSelectedIndexを変更したいがうまくいかない。

Question

FormアプリケーションにListView(MultiSelect=false)を追加して下記コードを実行して、SelectedIndexをマウスで変えた時に、クリックしたアイテムが一瞬選択状態になった後に選択が解除されてしまいます。
想定では普通にクリックしたアイテムが選択状態になると思ったのですが、原因は何でしょうか。

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace WindowsFormsApp9
{
    public partial class Form1 : Form
    {
        private int disableEventCounter = 0;

        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            func();
        }

        private void func()
        {
            var selectedIndex = listView1.SelectedItems.Count > 0 ? listView1.SelectedIndices[0] : -1;

            if (selectedIndex != -1)
                disableEventCounter++;
            listView1.Items.Clear();
            
            listView1.Items.Add("aaaaaaaaaaa");
            listView1.Items.Add("bbbbbbbbbbb");
            listView1.Items.Add("ccccccccccc");

            if (selectedIndex != -1)
            {
                disableEventCounter++;
                listView1.SelectedIndices.Add(selectedIndex);
            }
        }

        private void listView1_SelectedIndexChanged(object sender, EventArgs e)
        {
            Console.WriteLine("enter");
            if (disableEventCounter > 0)
            {
                disableEventCounter--;
                return;
            }

            Console.WriteLine("pass");

            func();
        }
    }
}

Answer 1

目的の達成方法を考えて頂き嬉しいのですが、どちらかと言うと質問文のコードがうまくいかない原因を知りたいという勉強的な理由なのでそちらの回答も頂けると嬉しいです。

https://docs.microsoft.com/ja-jp/dotnet/desktop/winforms/controls/how-to-select-an-item-in-the-windows-forms-listview-control?view=netframeworkdesktop-4.8

に書いてあるように選択項目に対してフォーカスを移さないと駄目だと思います。
なので下記部分を

listView1.SelectedIndices.Add(selectedIndex);

下記のようにすると正しく動作します

listView1.Items[selectedIndex].Focused = true;
listView1.Items[selectedIndex].Selected = true;

また、カウンターを使用してイベントの無効化を行うと私的には関数外での変数等を使用するのでバグを生みやすいと感じます。
ここで行いたいのは

　リストの再構築時はSelectedIndexChangedを発生させたくない

ということだと思うので下記のようにイベント自体の 紐づけ/解除 をする方法もありだと思いますがどうでしょうか？
また、SelectedIndicesで選択されたIndexを取得しているのに選択項目があるかどうかはSelectedItemsを使用するというのはよくないと思うのでそちらも統一したほうがいいと思います。

private void Form1_Load(object sender, EventArgs e) {
    func();
}

private void func() {

    var selectedIndex = listView1.SelectedIndices.Count > 0 ? listView1.SelectedIndices[0] : -1;

    this.listView1.SelectedIndexChanged -= listView1_SelectedIndexChanged;

    listView1.Items.Clear();

    listView1.Items.Add("aaaaaaaaaaa");
    listView1.Items.Add("bbbbbbbbbbb");
    listView1.Items.Add("ccccccccccc");

    if (selectedIndex != -1) {                
        listView1.Items[selectedIndex].Focused = true;
        listView1.Items[selectedIndex].Selected = true;
    }

    this.listView1.SelectedIndexChanged += listView1_SelectedIndexChanged;
}

private void listView1_SelectedIndexChanged(object sender, EventArgs e) {
    func();
}

Comments

[dekaaki]

補足ですが今回の方法はSelectedIndexChangedで項目を再構築し、元の選択状態を復元するという目的になります。
何が言いたいかというと復元する選択項目を強制的に別の項目にすると正しく動作しなくなります。
例：2番目の項目をマウスで選択したのに3番目の項目を選択状態にするとか前回選択してたindex位置の項目がなくなったとか

他の方も言われてるようにそもそもユーザーの操作を阻害する可能性があるので、

方法１：
SelectedIndexChangedで項目の作り変えは行わない
この場合、例えばボタンのクリック処理とかで再構築とかにすると特殊なことはしなくてもSelectedIndices.Addで正常に動作します。

方法２：
SelectedIndexChangedで項目を作り変えたいのであれば、毎回作り変えるのではなく、再構築が必要な状態かどうか判断して必要な場合だけ再構築して選択状態は復元しない

といったことを検討したほうがいいかもしれません

Answer 2

マウスでアイテムを選択すると listView1_SelectedIndexChanged() が呼ばれますが、その最終行で func() を呼び出しています。
func() の中でアイテムが全削除されている(listView1.Items.Clear())ので、その時点で選択アイテムがなくなります。func()を呼び出す意図が分かってないですが、そこをコメントアウトしてはいかがでしょうか。

Answer 3

どうも触った感じだと、
・通常はMouseDown直後にSelectedIndexChangedが呼ばれるだけ
・SelectedIndexChanged中にlistView1.Items.Clear();した場合はマウスから指を離した直後にもSelectedIndexChangedが呼ばれ、選択が解除される（その後MouseDownイベント）

なので、提案として、MouseDown、MouseUpイベントを使うのはいかがでしょうか。

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Diagnostics;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace WinFormListViewSelectedIndex {
    public partial class Form1 : Form {
        public Form1() {
            InitializeComponent();

            func();
        }
        ListViewItem selectedItem = null;
        private void func() {
            var selectedIndex = listView1.Items.IndexOf(selectedItem);
            listView1.Items.Clear();

            var r = new Random();
            var cnt = r.Next(10, 50);
            for (int i = 1; i <= cnt; i++) {
                listView1.Items.Add("Item" + string.Format("{0:D2}", i));
            }

            if (selectedIndex != -1 && listView1.Items.Count > selectedIndex) {
                listView1.SelectedIndices.Add(selectedIndex);
                listView1.EnsureVisible(selectedIndex);
            }
        }

        private void listView1_MouseUp(object sender, MouseEventArgs e) {
            selectedItem = listView1.GetItemAt(e.X, e.Y);
            if (selectedItem == null) return;
            func();
        }
    }
}

Comments

[abroad128]

回答ありがとうございます。
目的の達成方法を考えて頂き嬉しいのですが、どちらかと言うと質問文のコードがうまくいかない原因を知りたいという勉強的な理由なのでそちらの回答も頂けると嬉しいです。

[Hey_CH]

中身についてはさっぱりわかりませんねー申し訳ないです。

[Hey_CH]

ソースコード見ればわかるかもしれません。
（僕は面倒くさがりなので理解しようという気にはなりませんがｗ）
https://referencesource.microsoft.com/#system.windows.forms/winforms/Managed/System/WinForms/ListView.cs
https://github.com/dotnet/winforms/blob/main/src/System.Windows.Forms/src/System/Windows/Forms/ListView.cs

Answer 4

listView1.Items.Clear();

で選択状態も全部クリアされます。

これでなにをしたいんでしょう

Comments

[abroad128]

＞これでなにをしたいんでしょう
実際には変更されたリストを再リストアップするので最初に前回のリストアップを消しています。

＞選択状態も全部クリアされます。
listView1.SelectedIndices.Add(selectedIndex);
で選択状態を復元しようとしているのですがうまくいきません。

[y_waiwai]

そのクリアしたときもSelectedIndexChangedでたりしてませんかｗ

[abroad128]

何かが選択されているとSelectedIndexChangedが発火するので
if (selectedIndex != -1)
disableEventCounter++;
や
if (disableEventCounter > 0)
{
disableEventCounter--;
return;
}
などで回避しようとしているのですがうまくいかない原因が分からないです。

[y_waiwai]

ならそれを聞けばどうなんです？
質問に書いてないことを後付でどーこー言われても困りますぜw

[abroad128]

私の質問は元から添付されたコードが期待した動作をしない原因を知るためのもので後付けはありません。

[y_waiwai]

なら質問に書いてない事柄には答えないことにしますねｗ

[abroad128]

書いてありますが、不快なので通報しておきました。