Resharper说我不应该使用List

|| 我有一个方法：

static void FileChangesDetected(List<ChangedFiles> files)

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

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

使用IEnumerable<T>而不是List<T>有好处吗？

已邀请:

5 个回复

这一切都与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>

要回复问题请先登录或注册