来自lambda的RelayCommand和构造函数参数

如果在XAML文件中,我将Button从以下类绑定到“Command”,则单击Button不会导致执行DoIt:

class Thing() { public Thing(Foo p1) { Command = new RelayCommand(() => DoIt(p1)); } private DoIt(Foo p) { p.DoSomething(); } public ICommand Command { get; private set; } } 

但是,如果我从p1初始化一个字段并将该字段作为参数传递给lambda中的方法调用,它确实有效:

 class Thing() { private Foo field; public Thing(Foo p1) { field = p1; Command = new RelayCommand(() => DoIt(field)); } private DoIt(Foo p) { p.DoSomething(); } public ICommand Command { get; private set; } } 

为什么前者会失败,但后者会按预期工作?

可能相关: 闭包如何在幕后工作? (C#)

编辑:澄清一下,以下内容对我也有用。 但是,我仍然想知道为什么第二个例子做了我的预期,但第一个例子没有。

 class Thing() { private Foo field; public Thing(Foo p1) { field = p1; Command = new RelayCommand(DoIt); //Command = new RelayCommand(() => DoIt()); Equivalent? } private DoIt() { field.DoSomething(); } public ICommand Command { get; private set; } } 

这是一个老问题,但我最近偶然发现了这个话题,值得回答。

这种奇怪行为的原因来自RelayCommand的MVVM Light实现。 execute和canexecute处理程序在relay命令中存储为WeakAction _executeWeakFunc _canExecuteWeakAction尝试在UI由于某种原因仍然引用该命令时允许GC清理视图模型。

跳过一些细节,底线是:分配一个viewmodel方法,因为处理程序工作得很好,因为只要viewmodel保持活动状态, WeakAction就会保持活着状态。 对于动态创建的Action ,情况就不同了。 如果对该操作的唯一引用在RelayCommand ,则只存在弱引用,GC可以随时收集操作,将整个RelayCommand变为死砖。

好的,详情的时间。 WeakAction的实现不是盲目地存储对动作的弱引用 – 这将导致许多消失的引用。 而是存储弱Delegate.Target引用和Delegate.MethodInfo的组合。 对于静态方法,该方法将通过强引用存储。

现在,这导致了三类lambda:

  1. static method: () => I_dont_access_anything_nonstatic()将存储为强引用
  2. 成员变量的闭包: () => DoIt(field)闭包方法将在viewmodel类中创建,操作目标是viewmodel,只要viewmodel保持活动状态,它就会保持活动状态。
  3. 局部变量的闭包: () => DoIt(p1)闭包将创建一个单独的类实例来存储捕获的变量。 这个单独的实例将成为操作目标,并且不会有任何强烈的引用 – GC会在某些时候进行清理

重要提示:据我所知,这种行为可能会随着Roslyn而改变: 代表缓存行为在Roslyn中发生了变化,因此今天使用case(2)的代码可能会变成使用Roslyn的非工作代码。 但是,我没有测试这个假设,它可能会完全不同。

你的问题是调用Method DoIt是在lamda表达式创建的另一个匿名方法中。 你的表达

 () => DoIt(p1); 

创建一个没有参数的匿名方法(看作第一个大括号中没有提供变量)。

我建议你使用mvvm-light中的generics构造函数来创建Command:

 class Thing { public Thing() { Command = new GalaSoft.MvvmLight.Command.RelayCommand(DoIt); } private void DoIt(bool p) { p.DoSomething(p); } public System.Windows.Input.ICommand Command { get; private set; } } 

然后将Button绑定到“Command”。