问IReadOnlyCollection对ReadOnlyCollection

EN

您肯定应该尝试使您的公共方法返回接口。

如果您担心类的调用方将转换和修改您的内部结构，如本例中所示，那么类的内部队列不应该从外部访问：

public class QueueThing
{
    private List<QueueItem> _cantTouchThis;

    public IReadOnlyCollection<QueueItem> GetQueue()
    {
        return _cantTouchThis;
    }
}

然后，您可以使用AsReadOnly()返回一个来自私有List<T>的新ReadOnlyList<T>。

public class QueueThing
{
    private List<QueueItem> _cantTouchThis;

    public IReadOnlyCollection<QueueItem> GetQueue()
    {
        return _cantTouchThis.AsReadOnly();
    }
}

现在调用方可以随意转换返回的值，他们将无法修改_cantTouchThis成员(当然，在使用反射时除外，但无论如何，所有的赌注都取消了)。

考虑到许多类型可以实现一个接口，这种方法的用户绝对不应该假定将该方法的返回值转换为任何具体类型是安全的。

只有当基础对象是该类型时，才能将IReadOnlyCollection<T>转换为List<T>。例如，ReadOnlyCollection<T>也实现了IReadOnlyCollection<T>。

因此，我的建议是返回IReadOnlyCollection<T>，如果您担心调用方会错误地将其转换为不应该的内容，请确保底层类型为ReadOnlyCollection<T>

public IReadOnlyCollection<User> GetUsers()
{
   return new ReadOnlyCollection<User>();
}

但是返回IReadOnlyCollection<T>应该足够让函数调用者理解它应该是只读的。

请注意，您永远无法使用ReadOnlyCollection<T>完全保护您的代码，调用方仍然可以使用反射访问内部列表并对其进行操作。

在这种情况下，唯一的选择是创建一个副本，如果列表并返回该副本。

微软的指南这里声明：

ReadOnlyCollection是ReadOnlyCollection<T>的一个子类，✓确实使用IEnumerable<T>来表示只读集合的属性或返回值。

因此，基本上，您应该返回ReadOnlyCollection<T>。它在其他情况下指定了接口IEnumerable，所以如果它打算使用接口IReadOnlyCollection<T>，它就会这样说。