C# チェックボックスの全選択 全解除

Question

.NET Framework4.7.2
WPF

お世話になります。
チェックボックスの全選択・全解除をしたくいろいろと調べてここまでコードを書いたのですが
どうしても解らないので質問させていただきます。

試したことは、
checkBox1_CheckStateChanged
checkBox1_CheckedChanged
各イベントで下記のコード（マウスclickイベント以外）を入れてみたのですがチェックがついたり消えたりして
暴走してしまい希望の結果にはならなかったです。

private void checkBox1_MouseClick(object sender, MouseEventArgs e)
        {
            if (checkBox1.Checked == true)
            {

                foreach (Control item in groupBox2.Controls)
                {
                    if (item.GetType().Equals(typeof(CheckBox)))
                    {
                        ((CheckBox)item).Checked = false;
                    }
                }


            }
            else
            {
                foreach (Control item in groupBox2.Controls)
                {
                    if (item.GetType().Equals(typeof(CheckBox)))
                    {
                        ((CheckBox)item).Checked = true;
                    }
                }
            }
        }

チェックボックスは下記のような仕様になっています

Comments

[退会済みユーザー]

・プロジェクトの種類（Windowsフォームアプリ、WPFアプリ、ASP.NET Webアプリ等）
・フレームワークのバージョン(.NET Framework X.X、.NET Core X.X等）
を質問を編集して追記してください。

[YT0014]

checkBox1とgroupBox2の関係をご提示ください。
checkBox1は、groupBox2に含まれているチェックボックスですか？

[退会済みユーザー]

質問者さん、無言ですが、いくつか回答がでているのでそれらに対するフィードバックを書きましょう。役に立った/立たなかったぐらいはすぐ返せるはず。役に立たなかったなら、どこがダメだったかを書くと、より質問者さんが期待する答えに近いものが出てくるかも。とにかく無言で放置は NG です。

[AMK]

スミマセン、今帰宅しました

checkbox１は、グループ2内に存在します。

Answer 1

SurferOnWwwさんと同じような前提条件の元、CheckedChangedを使用してます。
必要に応じてCheckedChangedイベントの削除を行って連動がスムーズになるようにしてます

親チェックボックス
CheckedChangedイベントをcheckBoxParent_CheckedChangedに紐づけ

子チェックボックス
各CheckedChangedイベントを全てcheckBoxChild_CheckedChangedに紐づけ

        private void checkBoxParent_CheckedChanged(object sender, EventArgs e) {

            foreach (var obj in panel1.Controls.OfType<CheckBox>()) {
                // 子のイベントを削除した状態で親の状態を反映
                obj.CheckedChanged -= checkBoxChild_CheckedChanged;
                obj.Checked = checkBoxParent.Checked;
                obj.CheckedChanged += checkBoxChild_CheckedChanged;
            }

        }

        private void checkBoxChild_CheckedChanged(object sender, EventArgs e) {

            // 対象の子チェックボックスのリスト
            var items = panel1.Controls.OfType<CheckBox>().ToList();
            // 子チェックボックスでチェックのついてる数
            var checkedCount = items.Count(x => x.Checked);

            // 親のイベントを削除した状態で子の状態を反映
            checkBoxParent.CheckedChanged -= checkBoxParent_CheckedChanged;

            if (checkedCount == 0) {
                checkBoxParent.CheckState = CheckState.Unchecked;
            } else if (checkedCount == items.Count) {
                checkBoxParent.CheckState = CheckState.Checked;
            } else {
                checkBoxParent.CheckState = CheckState.Indeterminate;
            }

            checkBoxParent.CheckedChanged += checkBoxParent_CheckedChanged;
        }

Answer 2

多分、Windows Forms アプリで、groupBox2 の中に ChechBox が複数配置されていて、それら全部の Checked プロパティを checkBox1 をマウスでクリックするごとに true/false を一度に切り替えたいのであろうと想像してレスします。

もし違ったらどう違うのか書いてください。あと、回答者・閲覧者が余計な想像をしないで済むよう、そういうことは最初から質問に書くようにしてください。

checkBox1 はどこにあるのですか？

groupBox2 の中にあるとすると、質問に書いてあったコードでは checkBox1 をクリックするたびに、checkBox1 を含めて groupBox2 の中のすべての CheckBox の Checked が書き換えられてしまうということは分かりますか？　それは期待する動きではないですよね。

試しに以下のようにしてみてください。

さらに、質問のコードを以下のように書き換えてください（コメントの部分）。それで checkBox1 にチェックが入ると groupBox2 内の ChechBox は全選択に、checkBox1 のチェックが外れると全解除になるはず。

private void checkBox1_MouseClick(object sender, MouseEventArgs e)
{
    if (checkBox1.Checked == true)
    {
        foreach (Control item in groupBox2.Controls)
        {
            if (item.GetType().Equals(typeof(CheckBox)))
            {
                //((CheckBox)item).Checked = false;
                ((CheckBox)item).Checked = true;
            }
        }
    }
    else
    {
        foreach (Control item in groupBox2.Controls)
        {
            if (item.GetType().Equals(typeof(CheckBox)))
            {
                //((CheckBox)item).Checked = true;
                ((CheckBox)item).Checked = false;
            }
        }
    }
}

これが質問者さんのやりたいことそのものズバリではなくても、応用すればできるのでは？

Comments

[dekaaki]

その方法ではスペースキーでチェック状態を変更したときに正しく動作しないのでは？
一応、SurferOnWwwさんと同じような前提条件でCheckedChangedを使用する回答を投稿しました

[退会済みユーザー]

MouseClick イベントは質問者さんのコードそのままです。回答に「・・・checkBox1 をマウスでクリックするごとに true/false を一度に切り替えたい・・・」「もし違ったらどう違うのか書いてください」と書いた通り、違ったら質問者さんがその旨言ってくることに期待。

[fana]

質問の文章が難読ですが，私は
「checkBox1_CheckedChangedとかで試したら暴走した　→（だから今は暴走をさけるために）クリックのイベントに書いている」
ということを言っているのかな，と読みました．

[dekaaki]

SurferOnWwwさん。確かに質問者のソースを見るとMouseClickになってますね。
私の方こそ質問者のソースをよく見てなかったようですね。失礼しました。

[Zuishin]

MouseClick なら質問のコードでも特に問題にならないんじゃないかと。

[退会済みユーザー]

> 「checkBox1_CheckedChangedとかで試したら暴走した　→（だから今は暴走をさけるために）クリックのイベントに書いている」ということを言っているのかな，と読みました．

そうかもしれませんね。でも、ひどく情報不足の質問ですのでそのあたりは自分は深く考えず、質問者さんのコードをほとんど変えずに絶対「暴走」しないやり方を回答した次第です。

[退会済みユーザー]

> MouseClick なら質問のコードでも特に問題にならないんじゃないかと。

checkBox1 も gruopBox2 の中に入れて、質問者さんのコードをそのままコピペしてやってみたんですが「暴走」はしないまでも動きが期待したものにならないかと思います。

[AMK]

期待通りの動作が得られました！！

ありがとうございます！！

Answer 3

            foreach (Control item in groupBox2.Controls)

で、checkbox1が操作されてしまい、そのイベントによりまた操作されて、、というループに陥ってませんか

Answer 4

多分ですけど、無限ループ状態になっている気がします。

たとえばcheckbox1 ~ 3 があるとして、

checkbox1 がチェックされたら checkbox2,3 はそれぞれfalseになる。
でも checkbox2 も 「チェックされたら自分以外はfalseになる」ので、
本来ならtrueになっているはずの checkbox1 も falseになる。
でも checkbox1 が falseならそれ以外は trueになるはずだから……

のようになっているのではないかと思います。

もし、「性別」とか「年号」、「学年」...とかみたいに排他的(一つがtrueならそれ以外はfalseとか)になる関係であれば、CheckBoxではなく、「RadioButton」を使うべきです。
そっちの方が安全ですし。

どうしてもCheckBoxで表現したいなら、「フィールドとして何らかのデータを持つ」かなぁと。

イベントやその設定方法(Form1_Loadイベントに書くのかForm1のコンストラクタ内で書くのかとか) でイベントが想定しないような状態で発火することがあります。

C#だとイベント設定はForm1_Loadイベントとかでも設定できるので問題なさそうですが、言語やライブラリによっては初期設定としての設定をすると「CheckBoxの値が変わった」とかのようなイベントが発火する事があります。
(しかも本来は発火させたくない)

その場合は、

// Form1_Loadイベント内とかで

CheckBox checkBox1 = new CheckBox();
checkbox1.Location = ...
...

// private bool isAllowedToRunCheckedChanged がフィールドとしてあるとする

isAllowedToRunCheckedChanged = true;

とかのようにして、

// CheckedChangedイベント内で

if( isAllowedToRunCheckedChanged && checkbox1.Checked ){
    // ここでそれ以外のCheckBoxをfalseにする
    isAllowedToRunCheckedChanged = false;
}

のようにするとか。

若しくは、CheckBox1 だけCheckedChanged イベントを受け付けて、それ以外はスルーするとか。

とにかく、そういう風に無限ループを避けるようなロジックにすると思います。

Answer 5

groupBox2 内に配されているcheckbox群のうちの１つ以上のcheckboxに
チェック状態の変化時に走るイベントハンドラが設定されており，
且つ，そのイベントハンドラ内で checkBox1 のチェック状態を操作している

……といったことになっていませんか？

そんなことになっているとしたら，

checkbox1の操作がトリガとなって groupBox2 内のcheckbox群のチェック状態を変えるけども，
その「チェック状態を変える」がトリガとなって，前記イベントハンドラが実行されて checkBox1 のチェック状態を変化させる

という話になります．
で，このとき checkbox1 に

checkBox1_CheckStateChanged
checkBox1_CheckedChanged

みたいなのがあったりすると，「checkBox1 のチェック状態を変化させる」がトリガとなってこれらが実行されて…

という嫌な連鎖になるでしょう．

Comments

[fana]

で，連鎖を断ち切るために，
checkBox1_CheckedChanged
とかを使うのをやめて，代替として
checkBox1_MouseClick
を用いてみている，という状況なのかな？　と想像．

[AMK]

その通りです＾＾

Answer 6

試したことは、

checkBox1_CheckStateChanged
checkBox1_CheckedChanged
各イベントで下記のコード（マウスclickイベント以外）を入れてみたのですがチェックがついたり消えたりして
暴走してしまい希望の結果にはならなかったです。

何となく複数個所にイベントを設定することで、必要以上にイベントが発生しているような気がします。CheckedChangedイベントだけに設定してみては？
あと、CheckedChangedイベントでDebug.WriteLineでログを取ったりしてイベントの発生する回数を確認したり、イベントにブレークポイントを設定してスタックトレースを確認したりしてみてください。

あと、毎回Controlsを検索してTypeの比較するの非効率なので、素直にList<CheckBox>のようなメンバ変数作成して、Loadイベント辺りで操作するCheckBoxを格納すればよいのではないかと。