FxCop 讨厌我使用 MVVM

Question

我刚刚开始与 FxCop 合作，看看我的代码在其完整规则集上的表现有多差。我从“破坏”规则开始，我遇到的第一个规则是 CA2227，它基本上表示您应该将集合属性的 setter 设置为只读，这样您就不会意外更改集合数据。

由于我使用 MVVM，我发现使用具有 get/set 属性的 ObservableCollection 非常方便，因为它使我的 GUI 更新在代码隐藏中变得简单而简洁。不过，我也能看出FxCop在抱怨什么。

我刚刚遇到的另一种情况是使用 WF，我需要在创建工作流时设置参数，并且我不想为我正在使用的集合编写一个包装器类，以避免出现此特定的错误消息。

例如，下面是我将属性设置为只读时收到的示例运行时错误消息：

The activity 'MyWorkflow' has no public writable property named 'MyCollectionOfStuff'

您对此有何看法？我可以忽略这个特定的错误，但这可能不好，因为我可能会在不适用 MVVM 的代码中的其他地方违反此规则（例如，仅模型代码）。我认为我还可以将其从属性更改为具有操作底层集合的方法的类，然后从 setter 方法发出必要的通知。我有点困惑......有人能解释一下吗？

Answer 1

这个特定规则告诉我们，集合属性应该设置为只读，因为您不需要将整个集合分配给属性。

例如，想象一个这样的类：

public class Foo
{
   public ObservableCollection<int> Bar { get; set; }
}

如果在代码中的某个位置有以下行，会发生什么：

var f = new Foo();
f.Bar = new ObservableCollection<int>();
f.Bar.AddRange(new int[] { 1, 2, 3, 4 });
// ...
// Attaches and handlers to the collection events
// ...
f.Bar = new ObservableCollection<int>();
f.Bar.AddRange(new int[] { 5, 6, 7, 8 });

当执行最后两行代码时，附加的事件处理程序不会被触发，因为 Bar 属性有一个完全不同的对象。

另一方面，如果该属性是只读的，则事件将被触发，并且一切都会按预期运行。