Resharper说我不应该使用List <T>

问题描述

|| 我有一个方法：

static void FileChangesDetected(List<ChangedFiles> files)

我使用了Visual Studio 2010和Resharper。 Resharper总是建议将ѭ1更改为ѭ2，我想知道为什么会这样。 在方法中，我只是这样做：

 foreach (var file in files)
 { ... }

使用

IEnumerable<T>

而不是

List<T>

有好处吗？     

解决方法

        这一切都与LSP（Liskov替代原理）有关。 基本上，与其使用实现，不如将其编码为抽象。 在这种特定情况下，如果您要做的只是循环遍历列表，则可以将ѭ2用作最简单的抽象-这样，您就不必使用ѭ1，而是函数中的任何集合类型。 这使您的功能更加可重用，并减少了耦合。     ,        Resharper建议您的方法实际上不需要

List<T>

作为参数，并且可以轻松地使用

IEnumerable<T>

。这意味着您可以使方法更通用。     ,        如果仅迭代

files

，则不必是List <>。您的代码也可以使用数组。或更笼统：它将与您可以迭代的所有内容一起使用。用IEnumerable <>表示。因此，使用List <>无需任何限制即可使用您的消息。 ReSharper方法只是一个提示。     ,        因为在代码中，您仅使用

files

是

IEnumerable<ChangedFiles>

的事实，所以不使用Count或Add。 即使以后要使用List特定方法（与Add或Count方法一起使用），也总是最好使用接口：:13ѭ而不是具体的实现。     ,        即使将其更改为，您仍然可以进行foreach

IEnumerable<ChangedFiles>

要么

ICollection<ChangedFiles>