複数のコントロールをまとめて値が変更されたかどうか知りたい

Question

お世話になります。

VSでC＃を使ってWindowsコンソールアプリケーションを作っています。

そのアプリの設定画面なのですが、複数のテキストボックスやラジオボタン等のコントロールがあります。

その画面には「設定を適用する」ボタンがあり、これを押さないと設定が保存されないようになっています。なので、設定を変更したにもかかわらず、このボタンが押されなかった場合、アプリを終了するときなどに「設定が保存されていませんが本当に終了してよろしいですか？」といったダイアログボックスを出したいと思っています。

各々が変更されたかどうかを確認するためにはたとえばTextBoxならTextChangedイベントがあり、それを見れば変更されたかどうかがわかると思います。

ですが、すべてのコントロールで

    private bool bChanged = false;    //変更された場合はフラグがTrue
    private void textBox_TextChanged(object sender, EventArgs e)
    {
        bChanged = true;
    }

    private void Form_closing(略){
        if(bChanged){
            DialogResult re =  MessageBox.Show("本当に終了してもよいですか?",(略));
        }
    }

などとするのは少々面倒ですし、bChangedフラグを書き忘れればチェック漏れにもなります(もちろんそんなことがあってはいけないのですが･･･)

一応すべてのコントロールはGroupBoxの上に配置されているので、GroupBoxに対してまとめて監視をできたりしないかな、と思ったのですが･･･

よろしくお願いいたします。

Answer 1

ものぐさな方法ですが、

public Form1()
{
	InitializeComponent();
	textBox1.TextChanged += anythingChanged;
	checkBox1.CheckedChanged += anythingChanged;
}

private void anythingChanged<T>(object sender,T e) where T : EventArgs
{
	bChanged = true;
}

とするとanythingChangedは.NETの各種イベントに対応できるので
コントロールが増えてもコンストラクタに1行追加するだけで追加のイベント処理として対応できます。
(なんならプロパティエディタに直接入力して設定もできます。)

ちゃんとそれぞれの値を確認するほうがいいかとは思いますが、、、

Comments

[catsforepaw]

それいいですね。むかーし、別のことでそのようなコードを書いたような記憶がありますが、コントロールの変更イベントと結びつきませんでした。
思い出させてくれてありがとうございます。

[ElecDove]

回答ありがとうございます

なるほど!
textBox○.TextChanged += anythingChanged;
さえ書き忘れなければいいですね笑

しかもこれなら、最終的に値が変更されていなくても、何らかの変更があったことをキャッチできますし、簡単ですし。

今回の件以外にもいろいろと使い道も多そうですし覚えておきたいと思います

Answer 2

保存する設定項目をメンバーに持つクラスを定義する
保存されたデータを読み込んでメンバーに保存し、その複製を作る
画面のデータは複製と連動させる（バインドなど）
保存するかどうかのチェック用にそのクラスにメソッドを用意して比較させる
副作用としてリセットボタンで保存データのインスタンスから画面データのインスタンスにディープコピーで実現可能
デメリットはメモリを大目に食うこと
それが嫌な場合はチェックメソッドで保存データを読み込んで比較するように変更し、リセット時も保存データからメンバーのデータを書き換えるようにする
メリットは消費メモリが減るはずだけど、デメリットでメモリ内の比較から保存データ（多分HDDに保存されてる？）の読み込みというコストのかかる処理が追加されること

Comments

[ElecDove]

回答ありがとうございます

>>デメリットはメモリを大目に食うこと

なるほど･･･メモリを意識したプログラミングをまだしたことがないので着目点としてそういうのもあるのだなぁ、と勉強になります

そこまでメモリを食うわけではなさそうですし、まあそれは妥協してもいいかな、なんて思ってます
それよりむしろ、アクセスの遅いHDDから読み込むほうが大変そうです(汗

Answer 3

2パターン思いつきました。

1.MVVMのように論理的なクラスを用意してクラスの変更側で検出する。
コントロール(イベント)→クラス(プロパティ)→クラスで変更を検出
2.各コントロールに変更検出の機能を拡張する

1は省略します(MVVM辺りのサンプルモデルはいっぱいあるでしょうし)。
2の場合のイメージは以下の通りです。

    //変更を検出するためのインターフェイス
    public interface IChanged
    {
        //変更があったかどうかを取得するプロパティ
        bool IsChanged { get; }
        //変更状態をクリアする
        void Clear();
    }

    //テキストボックスに実装
    public class TextBoxEx : TextBox, IChanged
    {
        private bool isChanged = false;
        //実使用上で煩雑にならないように明示的に実装(隠す)
        bool IChanged.IsChanged
        {
            get
            {
                return isChanged;
            }
        }
        void IChanged.Clear()
        {
            isChanged = false;
        }

        //変更を検出してフラグをセット
        protected override void OnTextChanged(EventArgs e)
        {
            base.OnTextChanged(e);
            isChanged = true;
        }
    }

このようにTextBoxを拡張しておいて変更を取得するインターフェイスを実装しておきます。
あとはこういう形で調べればいいかと

    //初期化時
    foreach(var control in this.Controls.OfType<IChanged>())
    {
        control.Clear();  //コントロールのフラグをリセット
    }

    //変更を検出したい時
    var changes = from control in this.Controls.OfType<IChanged>()
                  where control.IsChanged
                  select control;
    if (changes.Any())
    {
        //どこか変更があった
    }

ポイントは以下の点です。

・インターフェイスで実現しているので、他のコントロールも同様に実装すればフォーム側は一切手を加える必要がなく拡張に対応できる。
・インターフェイスを明示的に実装させてるので普段TextBoxを使用しているときは拡張分は見えない。

Answer 4

そういう気の利いた機能は聞いたことがありません。
……。

やはり一つ一つイベントを処理するのが王道だと思います。というか、私は今までそうやってきました。
プログラミングというのは面倒な作業なのです。

面倒ですが、その代わり、例えば設定フォームを開いたときに設定内容を覚えておいて、テキストを入力したり、チェックを付けたり外したりするたびに元の情報と照合して、結果的に変更がなければ適用ボタンを無効にする、などということもよくやります。

Comments

[ElecDove]

回答ありがとうございます

>>その代わり、例えば設定フォームを開いたときに設定内容を覚えておいて、テキストを入力したり、チェックを付けたり外したりするたびに元の情報と照合して、結果的に変更がなければ適用ボタンを無効にする、などということもよくやります。

やはり、これが一番簡単なのでしょうか･･･

>>やはり一つ一つイベントを処理するのが王道だと思います。というか、私は今までそうやってきました。

本当、地味な作業ですね･･･。あるところまで行くと単純作業というかなんというか･･･