チェックリストボックスにチェック後(2回目以降)、フォームが二回表示される

Question

【質問】
・2回目以降のチェックからフォームが二回表示されるようになっているため、一度だけ表示したい。
・チェックリストボックスのチェックが複数チェックできてしまうので、1つだけチェックできるようにしたい。

【実装する目的】
・検索をした際に、検索した値が複数もしくは単一でチェックリストボックスに表示されるため、チェックボックスを一つだけ選択できるようにし(複数チェック不可)、チェックボックスをクリックすると違うフォームを表示するようにしたい。
※チェックしたときにフォームが二回表示されることがあるためチェックボックスの方から検証

【仮説】
・チェックが二回扱いになっている？
・チェックの動作から確認

【流れ】
テキストボックスに入力し、検索ボタンを押すと、チェックリストボックスに表示され、チェックすると別のフォームを表示

【相談前実装】
参考記事を参照し、実装しました。
キータの記事を参考に試しに動かしてみるとクリック回数を重ねるにつれてフリーズする。ため1行コメントアウトして実装
マイクロソフトの記事を参考にしましたが、どこのファイルに何を記述すればいいのかがわからなかったため、実装中止。
チェックは一つになりましたが、二回目以降のチェック後、フォームが出現し、閉じると、また同じフォームが出現

■Form1.cs[デザイン]
checkedListBox1のプロパティ変更
CheckedOnClick: true
SelectionMode: One
Tab index:1
ThreeDCheckBoxes: true

■Form1.cs

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.IO;
using System.Windows.Forms;

namespace csvreadtest
{
    
    public partial class Form1 : Form
    {
         public Form1()
        {
              InitializeComponent();
         }
　　　 private void Form1_Load(object sender, EventArgs e)
        {
            checkedListBox1.Items.Add("りんご");
            checkedListBox1.Items.Add("おれんじ");
            checkedListBox1.Items.Add("みかん");
            checkedListBox1.Items.Add("もも");
            checkedListBox1.Items.Add("すいか");
            checkedListBox1.Items.Add("ぎんなん");
        }

       //クリックが二回判定？
        int lastCheckedIndex = -1;
        private void checkedListBox1_ItemCheck(object sender, ItemCheckEventArgs e)
        {
            if (e.Index != lastCheckedIndex)
            {
                if (lastCheckedIndex != -1)
                    checkedListBox1.SetItemCheckState(lastCheckedIndex, CheckState.Unchecked);
                lastCheckedIndex = e.Index;
            }
            //フォーム2を表示
            Form2 form2 = new Form2();
            form2.ShowDialog();
        }
        //To register event
        //checkedListBox1.ItemCheck += checkedListBox1_ItemCheck; //エラーのためコメントアウト
　　
        
    }
}

■Form2.cs

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 csvreadtest
{
    public partial class Form2 : Form
    {
        public Form2()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            this.Close();
        }

        private void button2_Click(object sender, EventArgs e)
        {
            this.Close();
        }
    }
}

【使用環境】
C# visual studio2022 .net framework windowsForm
【参考記事】
QIITA
https://qiita.com/noixip/items/3d1bd016b4cecfb48d2e
※実装しましたが、チェックをしすぎるとフリーズする。
microsoft
https://docs.microsoft.com/ja-jp/dotnet/api/system.windows.forms.checkedlistbox.checkonclick?view=windowsdesktop-6.0

プログラミング初心者です。
プログラマーの先輩方にご教授いただければ幸いです。よろしくお願いいたします。

Comments

[dodox86]

提示のコードを読んで：
省略し過ぎてForm2がShowDialog()される位置が変なところになっています。この位置ではコンパイルエラーでは？

[退会済みユーザー]

提示されてないコードが影響しているのではないですか？

> チェックリストボックスのチェックが複数チェックできてしまうので、1つだけチェックできるようにしたい。

それは参考にした Stackoverflow の記事の通りにして可能なはずです。

> 2回目以降のチェックからフォームが二回表示されるようになっているため、一度だけ表示したい。

それも提示したコードでそのようになると思いますけど？　試してみますね。

[sueteacher]

最低限の記述に修正しました。
二回目以降は、新しいフォームをボタンで閉じるとまたフォームがでてきます。
よろしくお願いいたします。

[退会済みユーザー]

試してみました。

> チェックリストボックスのチェックが複数チェックできてしまうので、1つだけチェックできるようにしたい。

Stackoverflow の記事の通りにして問題なく可能でした。

> 2回目以降のチェックからフォームが二回表示されるようになっているため、一度だけ表示したい。

2回目以降は先に選択した項目のチェックを SetItemCheckState メソッドで外すようにしていますが、外した際に再度 ItemCheck イベントが発生するからでした。

あとで回答欄にそれを回避するための案を書いておきます。

[sueteacher]

ありがとうございます。無事に動きました。

Answer 1

試してみました。

チェックリストボックスのチェックが複数チェックできてしまうので、1つだけチェックできるようにしたい。

Stackoverflow の記事の通りにして問題なく可能でした。

2回目以降のチェックからフォームが二回表示されるようになっているため、一度だけ表示したい。

2回目以降は先に選択した項目のチェックを SetItemCheckState メソッドで外すようにしていますが、外した際に再度 ItemCheck イベントが発生するからでした。

質問者さんのコードの「//フォーム2を表示」以下の 2 行を if (e.Index != lastCheckedIndex) に移動すればその問題は避けることができるはずです。

試したコードを載せておきます。Form5 を新規に作ってデザイン画面で CheckListBox を Form にドラッグ＆ドロップし、ItemCheck イベントのハンドラを生成しています。

using System.Windows.Forms;

namespace WindowsFormsApp1
{
    public partial class Form5 : Form
    {
        public Form5()
        {
            InitializeComponent();

            string[] myFruit = { "りんご", "おれんじ", "みかん", "もも", "すいか", "ぎんなん" };
            checkedListBox1.Items.AddRange(myFruit);

            checkedListBox1.CheckOnClick = true;

        }

        int lastCheckedIndex = -1;

        private void checkedListBox1_ItemCheck(object sender, ItemCheckEventArgs e)
        {
            if (e.Index != lastCheckedIndex)
            {
                if (lastCheckedIndex != -1)
                {
                    checkedListBox1.SetItemCheckState(lastCheckedIndex, CheckState.Unchecked);
                }
                lastCheckedIndex = e.Index;


                Form1 form1 = new Form1();
                form1.ShowDialog();
            }            
        }
    }
}