属性不应返回数组

Question

是的，我知道这个问题之前已经讨论过很多次了，我阅读了所有关于这个问题的帖子和 cmets，但似乎仍然无法理解。

MSDN 提供的解决此违规问题的选项之一是返回一个 collection（或由 collection实现的 interface >) 访问属性时，但很明显它并不能解决问题，因为大多数集合都不是不可变的，也可以更改。

我在这个问题的答案和 cmets 中看到的另一种可能性是用 ReadOnlyCollection 封装数组并返回它或它的基本接口（如 IReadOnlyCollection），但我不明白如何这解决了性能问题。

如果在任何时候引用该属性，它需要为封装数组的新ReadOnlyCollection 分配内存，那么有什么区别（在性能问题上，不是编辑数组/集合）而不是简单地返回原始数组的副本？

此外，ReadOnlyCollection 只有一个带有IList 参数的构造函数，因此需要在创建数组之前用列表包装数组。

如果我有意在我的类中使用一个数组（而不是作为不可变集合），当我为 ReadOnlyCollection 分配新内存并用它封装我的数组而不是返回数组的副本时，性能会更好吗?

请澄清这一点。

Answer 1

如果在任何时候引用该属性，它都需要为封装数组的新 ReadOnlyCollection 分配内存，那么与简单地返回一个副本有什么区别（在性能问题上，不编辑数组/集合）原始数组？

A ReadOnlyCollection<T> 包装集合 - 它不会复制集合。

考虑：

public class Foo
{
    private readonly int[] array; // Initialized in constructor

    public IReadOnlyList<int> Array => array.ToArray(); // Copy
    public IReadOnlyList<int> Wrapper => new ReadOnlyCollection<int>(array); // Wrap
}

假设您的数组包含一百万个条目。考虑Array 属性必须完成的工作量——它必须获取所有百万条目的副本。考虑Wrapper 属性必须完成的工作量 - 它必须创建一个仅包含引用的对象。

此外，如果您不介意额外的内存占用，您可以改为执行一次：

public class Foo
{
    private readonly int[] array; // Initialized in constructor
    private readonly IReadOnlyList<int> Wrapper { get; }

    public Foo(...)
    {
        array = ...;
        Wrapper = new ReadOnlyCollection<int>(array);
    }
}

现在访问 Wrapper 属性根本不涉及任何分配 - 如果 所有 调用者看到相同的包装器也没关系，因为他们无法改变它。

Answer 2

你不需要复制一个数组，只需返回它为IReadOnlyCollection<T>:

  public class MyClass {
    private int[] myArray = ...

    public IReadOnlyCollection<int> MyArray {
      get {
        return myArray;
      }
    }
  }