c# - 在 Expression-Bodied 属性中调用新的 RelayCommand (ICommand) 是否安全

Question

使用表达式主体属性，我们可以创建一个 RelayCommand，如下所示

public RelayCommand Command => _command ?? (_command = new RelayCommand(CommandExecute));

不过这也是可能的

public RelayCommand Command => new RelayCommand(CommandExecute);

显然，每次调用属性 getter 时都会创建一个新的 RelayCommand。虽然我看到周围有评论说底层管道只创建一个命令......

有人对此有明确的答案吗？

Answer 1

Does anyone have a definitive answer on this?

文档不 promise 只检索属性值一次。因此，您必须假设它可以多次检索它。

当然，在实践中，这种假定的行为可能永远不会发生。如果某个属性从未发生过属性更改通知，那么一旦检索到该属性，就永远不会再次检索该属性，这是完全合理的，当然，只读属性永远不会有属性更改通知。

所以你可能可以逃脱它。但就个人而言，我不会冒险。如果底层实现发生变化，或者您的假设出于任何原因是错误的，那么拥有同一命令的两个或更多实例将是一个问题，至少如果该命令曾引发 CanExecuteChanged 事件。我的意思是，我想如果 CanExecute() 状态永远不会改变，您可以拥有任意数量的对象副本，而且它们的工作方式完全相同。但如果它可以改变，那么你可能会在错误的命令对象上引发事件，一个没有人在听的对象。

这不仅仅是学术上的。 Microsoft 或基于 XAML/MVVM 的 API 的某些其他实现者可能有一天会使用您的代码，选择放弃存储命令对象引用，而是指望能够始终从模型对象中检索它，它模型对象本身的常见做法是从属性中检索命令对象。多次读取命令属性的场景是完全合理的，值得担心。

更重要的是，我看不到任一选项背后的动机。 “每次都创建一个新的”对我来说显然是错误的，即使你可以摆脱它。惰性初始化看起来像是过于复杂的代码，没有任何好处。毕竟，在创建模型对象之后，接下来几乎总是会发生的事情是将属性绑定(bind)到 UI，因此此时将检索命令属性。惰性初始化最多会将底层字段的初始化延迟几毫秒(通常比这少得多)。

如果你放弃惰性初始化，你就可以使用自动属性:

public RelayCommand Command { get; } = new RelayCommand(CommandExecute);

没有显式字段!好多了，恕我直言。

当然要注意，要使用该语法 CommandExecute() 必须是 static 成员。大多数命令确实需要访问模型实例，因此以上内容不适用于这些命令。

lazy-init 模式流行的一个原因可能是它允许使用字段初始化器语法，作为通常的“不允许在字段初始化器中使用实例成员”规则的漏洞。

就我个人而言，对于使用当前实例的命令，我仍然会选择构造函数内初始化(这对于只读自动属性很有效……您仍然不需要显式支持字段)。在这种情况下，惰性初始化似乎是过早的和错误优化。