来自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 _execute
和WeakFunc
。 WeakAction
尝试在UI由于某种原因仍然引用该命令时允许GC清理视图模型。
跳过一些细节,底线是:分配一个viewmodel方法,因为处理程序工作得很好,因为只要viewmodel保持活动状态, WeakAction
就会保持活着状态。 对于动态创建的Action
,情况就不同了。 如果对该操作的唯一引用在RelayCommand
,则只存在弱引用,GC可以随时收集操作,将整个RelayCommand
变为死砖。
好的,详情的时间。 WeakAction
的实现不是盲目地存储对动作的弱引用 – 这将导致许多消失的引用。 而是存储弱Delegate.Target
引用和Delegate.MethodInfo
的组合。 对于静态方法,该方法将通过强引用存储。
现在,这导致了三类lambda:
- static method:
() => I_dont_access_anything_nonstatic()
将存储为强引用 - 成员变量的闭包:
() => DoIt(field)
闭包方法将在viewmodel类中创建,操作目标是viewmodel,只要viewmodel保持活动状态,它就会保持活动状态。 - 局部变量的闭包:
() => 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”。