c# - 集合属性应该是只读的——漏洞？

Question

在坚持代码分析错误的过程中，我正在将我的属性更改为具有私有(private) setter 。然后我开始尝试更多地了解为什么。根据一些研究，MS 说 this :

A writable collection property allows a user to replace the collection with a completely different collection.

答案 here 指出:

Adding a public setter on a List<T> object is dangerous.

但是没有列出它危险的原因。这就是我很好奇的部分。

如果我们有这个集合:

public List<Foo> Foos { get; set; }

为什么要将 setter 设为私有(private)？显然我们不希望客户端代码替换集合，但是如果客户端可以删除每个元素，然后添加他们想要的任何元素，那有什么意义？这与完全替换集合不一样吗？遵循此代码分析规则如何提供值(value)？

Answer 1

不公开 setter 可以防止集合被赋值为 null 的情况。 null 和没有任何值的集合是有区别的。考虑:

for (var value in this.myCollection){//做某事

当没有值时(即有人对每个值都调用了 Remove)，没有什么不好的事情发生。但是，当 this.myCollection 为 null 时，将抛出 NullReferenceException。

代码分析假设您的代码在操作之前不检查 myCollection 是否为 null。

它可能也是对 System.Collections.Concurrent 中定义的线程安全集合类型的额外保护。想象一些线程试图通过覆盖它来替换整个集合。通过摆脱公共(public) setter，线程唯一的选择就是调用线程安全的 Add 和 Remove 方法。