C# と VB.NET の質問掲示板

イベントは、ソース上は

public event PropertyChangedEventHandler PropertyChanged;

という風に記述できますが、コンパイル時には大体次のようなコードに展開されます。

private PropertyChangedEventHandler _PropertyChanged;

[MethodImpl(MethodImplOptions.Synchronized)]
public event PropertyChangedEventHandler PropertyChanged
{
    add { this._PropertyChanged += value; }
    remove { this._PropertyChanged -= value; }
}

ここで注意してほしいのは、_PropertyChangedは普通のインスタンスフィールドであることです。
BinaryFormatterはフィールド全てをシリアライズします。
イベント構文で暗黙に生成されるデリゲートインスタンスも当然その対象です。

さて、オブジェクトをBindingSource.DataSourceに関連づけたとき、
変更通知のためにPropertyChangedイベントにBindingSourceのメソッドが
イベントハンドラとして追加されます。

ここまでを整理しましょう。
初めにサブ画面が開くとき、ファイルが存在しないので新たにpersonインスタンスを生成します。
DataSourceに設定して、PropertyChangedイベントにイベントハンドラが追加されます。
サブ画面終了時、シリアライズします。
最初に述べたように、PropertyChangedイベントに追加されたイベントハンドラもシリアライズされます。

次にサブ画面を開くとき、ファイルが存在するので、デシリアライズします。
もちろん最初に追加されたイベントハンドラごとです。
DataSourceに設定して、PropertyChangedイベントにイベントハンドラが追加されます。
「追加」なので、当然ながらPropertyChangedイベントに登録されたイベントハンドラは
ここで2つになります。
サブ画面終了時のシリアライズでは、この2つのイベントハンドラを両方ともシリアライズします。

以下、やればやるほどシリアライズするイベントハンドラは増えていくことになります。


こういう用途ではイベントハンドラをシリアライズする意味は無いと思われます。
ですので、[NonSerialized]属性を付けてシリアライズの対象外とするのがいいでしょう。
ただしNonSerialized属性はフィールドにしか付けられないので、最初に書いたevent構文には付けられません。
二番目に書いた、add/removeを明示的に記述する構文を使用し、
そのバッキングフィールドとなる_PropertyChangedの方に付けるようにします。

Hongliang さん
ご回答有難うございます。

> public event PropertyChangedEventHandler PropertyChanged;
を
> private PropertyChangedEventHandler _PropertyChanged;
> 
> public event PropertyChangedEventHandler PropertyChanged
> {
>     add { this._PropertyChanged += value; }
>     remove { this._PropertyChanged -= value; }
> }
に変更し、RaisePropertyChanged内のthis.Propertychangedをthis._PropertyChangedに変更したところ
望みどおりに機能するようになりました。

ただ、MethodImpl属性をつけると以下のエラーが出ます。
「属性 'MethodImpl' は、この宣言型では無効です。'constructor, method' の宣言でのみ有効です」
エラー内容から考えるとコンストラクタ、メソッドでしか使えないようなのですがなぜでしょうか？


解説も書いて頂きありがとうございます。
教えて頂くと、確かにその通りだ！と思えましたが、自分自身では導き出す事ができませんでした。
私の閻魔帳に大事に保存させて頂きたいと思います。