当前位置:  开发笔记 > 编程语言 > 正文

属性不应返回数组

如何解决《属性不应返回数组》经验,为你挑选了1个好方法。

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

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

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

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

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

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

请澄清一下......



1> Jon Skeet..:

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

A ReadOnlyCollection 包装集合 - 它不会复制集合.

考虑:

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

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

想象一下,你的数组包含一百万个条目.考虑一下该Array物业必须做的工作量- 它需要获取所有百万条目的副本.考虑Wrapper属性必须要做的工作量- 它必须创建一个只包含引用的对象.

此外,如果你不介意额外的内存命中,你可以做一次:

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

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

现在访问该Wrapper属性根本不涉及任何分配 - 如果所有调用者都看到相同的包装器并不重要,因为它们不能改变它.

推荐阅读
乐韵答题
这个屌丝很懒,什么也没留下!
DevBox开发工具箱 | 专业的在线开发工具网站    京公网安备 11010802040832号  |  京ICP备19059560号-6
Copyright © 1998 - 2020 DevBox.CN. All Rights Reserved devBox.cn 开发工具箱 版权所有