ICollection 属性和封装

Question

考虑到我想要封装它们，为我的属性使用接口是不好的做法吗？

public class Order
{
    private readonly ICollection<OrderItem> _orderItems;
    public IReadOnlyCollection OrderItems => _orderItems; // not possible without ToList
}

没有简单的方法来公开它，因为如果不首先使用ToList，就无法从ICollection 转换为IReadOnlyCollection 或IEnumerable，这涉及复制整个集合。

我应该将其定义为：

private readonly Collection<OrderItem> _orderItems;

或

private readonly HashSet<OrderItem> _orderItems;

或

private readonly List<OrderItem> _orderItems;

改为？

Answer 1

您应该遵循的规则是私有成员仅对定义它们的类可见，因此您可以将类型定义为List<T> 或Hashset<T> 或Collection<T>，其他使用您的类程序集的开发人员不会如果您稍后修改类型，则会受到重大更改的影响。

只要您知道Hasset<T> vs List<T> 和List<T> vs Collection<T> 之间的区别，您就可以选择您在问题中提出的三个备选方案之一。

因此，如果您不想在OrderItems 实现中调用ToList()，请删除接口ICollection<OrderItem> 作为您的private 字段类型并使用具体类。

Answer 2

您可以编写自己的扩展来将 ICollection 转换为 IReadOnlyCollection

public static class CollectionExtensions
{
    public static IReadOnlyCollection<T> AsReadOnly<T>(this ICollection<T> source)
    {
        if (source == null)
        {
            throw new ArgumentNullException("source");
        }
        return source as IReadOnlyCollection<T> ?? new ReadOnlyCollectionAdapter<T>(source);
    }
    sealed class ReadOnlyCollectionAdapter<T> : IReadOnlyCollection<T>
    {
        ICollection<T> source;
        public ReadOnlyCollectionAdapter(ICollection<T> source) { this.source = source; }
        public int Count { get { return source.Count; } }
        public IEnumerator<T> GetEnumerator() { return source.GetEnumerator(); }
        IEnumerator IEnumerable.GetEnumerator() { return GetEnumerator(); }
    }
}