属性不应返回数组

时间:2015-12-03 12:04:41

标签: c# .net

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

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

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

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

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

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

请澄清一下......

2 个答案:

答案 0 :(得分:13)

  

如果在任何时候引用该属性,它需要为封装数组的新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改变它。

答案 1 :(得分:1)

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

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

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