我刚刚开始使用FxCop来查看我的代码对其完整规则的影响有多么糟糕.我开始使用“Breaking”规则,我遇到的第一个是CA2227,它基本上说你应该只读取一个collection属性的setter,这样你就不会意
由于我使用MVVM,我发现使用带有get / set属性的ObservableCollection非常方便,因为它使我的GUI更新在代码隐藏中变得简单和简洁.但是,我也可以看到FxCop在抱怨什么.
我遇到的另一种情况是使用WF,我需要在创建工作流时设置参数,我不得不围绕我正在使用的集合编写一个包装类,以避免出现这种特殊的错误消息.
例如,这是我在readonly创建属性时获得的示例运行时错误消息:
The activity 'MyWorkflow' has no public writable property named 'MyCollectionOfStuff'
你对此有何看法?我可以忽略这个特定的错误,但这可能不好,因为我可以想象在MVVM不适用的代码中的其他地方违反了这个规则(例如,仅建模代码).我想我也可以通过操作底层集合的方法将它从属性更改为类,然后从setter方法引发必要的通知.我有点困惑……任何人都可以对此有所了解吗?
此特定规则告诉我们,应该将集合属性设置为只读,因为您不需要将整个集合分配给属性.例如,想象一个这样的类:
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属性具有完全不同的对象.
另一方面,如果属性是只读的,则会触发事件,一切都会按预期运行.