MVVMでは、ViewModelでモデル項目のイベントをフックするとメモリリークが発生しますか？

Question

私たちのWPF/MVVMアプリケーションで何らかのメモリリークを追跡しようとしていて、何かが発生しました...

関連するViewModelでModelItemイベントをインスタンス固有のハンドラでリッスンすると、 ModelItemがまだ存在する限り、ViewModelItemはハングアップしますか？

が、これは漏れを起こした場合、あなたはそれをどのように/どこ外すん

public class ItemViewModel 
{ 
    public ItemViewModel(ModelItem item) 
    { 
     this.Item = item; 
     item.SomeEvent += ItemSomeEventHandler 
    } 

    Public ModelItem Item{ get; private set; } 

    // Note: This is a handler on this instance, not a shared handler 
    Private void ItemSomeEventHandler(object s, EventArgs e) 
    { 
     Console.WriteLine(“The model changed!”); 
    } 
} 

...このケースを考えてみましょうか？ 'Dispose'ではそれを行うことはできません。これは、モデルアイテムのように見えるものがまだある場合には呼び出されないためです。

このViewModelが使用されているコントロールにDisposeを実装するのが適切なのでしょうか？そこにイベントアンフックルーチンが流れ落ちますか？あなたの内部をきれいにするために何か外部のものに頼るのはちょっと面倒ですが、EventManagerアーキテクチャには足りないので、ここで何をすべきか分かりません。

Answer 1

はい、それはリークが発生します。..またはより具体的にビューがもはやModelItemがItemViewModel

への参照を保持しているので、あなたがDisposeを実装しているのEventHandlerを外すことができ、表示されたときItemViewModelが収集されませんされません。 ViewModelのライフサイクルを制御できる場合は完全に有効です。問題は、どのコンポーネントがItemViewModelを作成しているかです。それが別のビューモデルであれば、それにdisposeを委譲できます。ビューがビューモデルを作成する場合は、Disposeメソッドを呼び出すことができます。

ただし、MVVMLightツールキットのように使用する場合は、その代わりにICleanupインターフェイスを使用できます。詳細はanswerをご覧ください。

Answer 2

なぜ使用しないのですかWeakEvent/DelegateCommands？

決定的なデタッチを必要としないデリゲートへの弱い参照を実装します。