Question

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

MSDN提供的解决此违规的一个选项是在访问属性时返回一个集合（或由集合实现的接口），但显然它不能解决问题，因为大多数集合不是不可变的也可以改变。

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

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

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

如果我故意想在我的类中使用数组（而不是作为不可变集合），那么当我为ReadOnlyCollection分配新内存并用它封装我的数组而不是返回数组的副本时，性能是否更好？ / p>

请澄清一下......

Answer 1

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

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属性根本不涉及任何分配 - 如果所有来电者都看到相同的包装，这并不重要，因为他们可以＆＃39 ; t改变它。

Answer 2

您无需复制数组，只需将返回作为IReadOnlyCollection<T>：

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

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

属性不应返回数组

2 个答案: