FormアプリケーションのINotifyPropertyChangedの実装について

Question

前提・実現したいこと

INotifyPropertyChangedを実装しバインドされたプロパティの変更通知を送りたい

発生している問題・エラーメッセージ

https://docs.microsoft.com/ja-jp/dotnet/framework/winforms/change-notification-in-windows-forms-data-binding
https://garafu.blogspot.jp/2014/09/wpf-inotifypropertychanged.html
上記を参考にOnPropertyChanged()を実装したがPropertyChangedが常にnullなので何も怒らない。
どこの実装例を見てもPropertyChangedにイベントハンドラは登録されておらずなぜこれで変更の通知ができるのかわからない。

該当のソースコード

参考ページのソースになります。
実際にはFormアプリケーションでINotifyPropertyChangedの実装は全く同じ。

namespace WpfApplication.ViewModels
   using System.ComponentModel;
   using System.Runtime.CompilerServices;
   /// <summary>
   /// ViewModel 基底クラス を表現します。
   /// </summary>
   public class ViewModelBase : INotifyPropertyChanged
   {
       /// <summary>
       /// プロパティ値が変更されたことをクライアントに通知します。
       /// </summary>
       public event PropertyChangedEventHandler PropertyChanged;
       /// <summary>
       /// PropertyChanged イベント を発生させます。
       /// </summary>
       /// <param name="propertyName">変更されたプロパティの名前</param>
       protected void OnPropertyChanged([CallerMemberName] string propertyName = "")
       {
           if (this.PropertyChanged != null)
           {
               this.PropertyChanged(this, new PropertyChangedEventArgs(propertyName));
           }
       }
   }

以下のように使用しています。

public class Form1 : Form, INotifyPropertyChanged
{
  public event PropertyChangedEventHandler PropertyChanged;
  protected void OnPropertyChanged(string propertyName)
  {
    if (PropertyChanged != null)
      PropertyChanged(this, new PropertyChangedEventArgs(propertyName));
  }

  public Form1()
  {
    InitializeComponent();

    // コンボボックスにリストを設定
    this.ComboBox.DataSource = list;
    // リスト以外から選択不可
    this.ComboBox.DropDownStyle = ComboBoxStyle.DropDownList;
    // バインディングソースのついか
    this.bindingSource.DataSource = currentValue;
    // テキストボックスにバインドを設定
    this.TextBox.DataBinding.Add("Text", bindingSource, null);
    // コンボボックス選択値変更イベントハンドラ登録
    this.ComboBox.SelectedValueChanged += new EventHandler(SelectedValueChanged);
  }

  List<string> list = new List<string()
  {
    "X",
    "Y"
  };

  int currentValue
  {
    // コンボボックスの値がXなら0、そうでなければ1をテキストボックスに表示
    get { this.ComboBox.SelectedValue.ToString() == "X" ? 0 : 1; }
  }

  void SelectedValueChanged(object sender, EventArgs e)
  {
    // 変更を通知
    OnPropertyChanged("currentValue");
  }
}

Answer 1

Bindingの仕組みを根本的に理解されていないように思います。
余計なお世話かもしれませんが闇雲に尋ねたりせず、サンプル等でしっかり理解されたほうが良いのでは？

■bindingSourceの発火元になるのはFormなのですから

this.bindingSource.DataSource = this;

では？

■発火元となるFormのプロパティ　currentValue　をBindしたいのなら

   this.TextBox.DataBinding.Add("Text", bindingSource, "currentValue");

です。

あとBindするプロパティはPublicじゃなきゃダメなはず

  public int currentValue
  {
    // コンボボックスの値がXなら0、そうでなければ1をテキストボックスに表示
    get { this.ComboBox.SelectedValue.ToString() == "X" ? 0 : 1; }
  }

Answer 2

前の質問 が解決したならベストアンサーを決めてください。
そこで少し触れているようにコントロールとバインドしなければなりません。

バインドはデザイナからできますしその方が簡単です。
コードでやりたいなら DataBindings プロパティと Bindings クラスについて調べてください。

Comments

[anonymousss]

ベストアンサー忘れていました。
前回の回答頂いた実装で自分なりに改変してみたのですがうまくいきません。
バインドはできているとおもうのですが...

[Zuishin]

Form は INotifyPropertyChanged をすでに実装していますので追加することはできません。
このコードはコンパイルできないはずです。

あと前回の私の回答の次の部分

args.Value += "が選択中";

これを args.Value = args.Value == "X" ? "0" : "1"; に変更するだけでここまでしなくとも望みの動作になるのではないかと思います。

[Zuishin]

あと、編集前の質問にあった INotifyPropertyChanged の実装についての記述を元に戻し、新しい情報は追記とわかるように追記してください。

でないと今の質問にある問題だらけのソースを見て Tak1wa さんが問題ないとおっしゃっているように見えます。

[anonymousss]

消した部分は戻しておきました。
コンパイルができることは確認しています。
FormはINotyfiPropertyChangedを実装しているとはどういうことでしょうか？
WPFの知識が少しだけあったものでそれに寄せるためプロパティを分け変更通知を実施するという実装を行ってみたかったのですが。。。

[Zuishin]

すみません。記憶違いでした。
Form は INotifyPropertyChanged を実装していません。

とりあえず今回はどこにもバインドされていない Form が INotifyPropertyChanged を実装する必要はありません。
バインドされているのは BindingSource です。

BindingSource は TextBox と ComboBox の両方にバインドされているので、コンボボックスの SelectedValueChanged も不要です。SelectedValue の変化は BindingSource が TextBox に通知してくれます。

ComboBox の変更によって TextBox の値が変わることは前回示した通りです。
今回もやっていることは同じで「が選択されています」が「0 か 1」に変わっただけだと思います。
であるなら、その部分の変更だけで対応できるのではないですか？

変更方法については二つ前のコメントで示しました。

[anonymousss]

INotifyPropertyChangedを実装するのはバインドされているクラスなんですね。
勘違いをしていました。

[Zuishin]

バインドされているのは ComboBox と BindingSource の対、そして TextBox と BindingSource の対なので、Form は関係ありません。
それらは Form のプロパティなので公開してバインドするという手もあります(ebiryo さんの方法)が、 Form のプロパティ同士をバインドするよりコントロールとデータをバインドする方が素直かと思います。
このためだけに公開・非公開を決めるのもどうかと思いますし。
もっともシナリオによってはプロパティ同士をバインドした方がいい時もありますが、今回はリストデータが用意されているので。

Answer 3

こんにちは。

INotifyPropertyChangedの実装は問題ないように見えますが、
PropertyChangedの購読部分のコードも掲載できますか？

WPFだとXamlからバインドしてたりしますが、
WinFormsだとどのようにバインドしているのだろう。

Comments

[anonymousss]

追記いたしました