无法从 IEnumerable<T> 转换为 ICollection<T>

Question

我已经定义了以下内容：

public ICollection<Item> Items { get; set; }

当我运行这段代码时：

Items = _item.Get("001");

我收到以下消息：

Error   3   
Cannot implicitly convert type 
'System.Collections.Generic.IEnumerable<Storage.Models.Item>' to 
'System.Collections.Generic.ICollection<Storage.Models.Item>'. 
An explicit conversion exists (are you missing a cast?)

谁能解释我做错了什么。我很困惑 Enumerable、Collections 和使用 ToList() 的区别

添加信息

稍后在我的代码中，我有以下内容：

for (var index = 0; index < Items.Count(); index++) 

我可以将 Items 定义为 IEnumerable 吗？

Answer 1

ICollection<T> 继承自 IEnumerable<T> 所以分配结果

IEnumerable<T> Get(string pk)

到ICollection<T> 有两种方法。

// 1. You know that the referenced object implements `ICollection<T>`,
//    so you can use a cast
ICollection<T> c = (ICollection<T>)Get("pk");

// 2. The returned object can be any `IEnumerable<T>`, so you need to 
//    enumerate it and put it into something implementing `ICollection<T>`. 
//    The easiest is to use `ToList()`:
ICollection<T> c = Get("pk").ToList();

第二个选项更灵活，但对性能的影响更大。另一种选择是将结果存储为IEnumerable<T>，除非您需要ICollection<T> 接口添加的额外功能。

附加性能评论

你的循环

for (var index = 0; index < Items.Count(); index++)

在IEnumerable<T> 上工作，但效率低下；每次调用Count() 都需要完整 枚举所有元素。使用集合和 Count 属性（不带括号）或将其转换为 foreach 循环：

foreach(var item in Items)

Answer 2

您不能直接从IEnumerable<T> 转换为ICollection<T>。您可以使用IEnumerable<T> 的ToList 方法将其转换为ICollection<T>

someICollection = SomeIEnumerable.ToList();

Answer 3

关于该问题的更多信息：

请提供更多关于物品类型和Get签名的信息

您可以尝试以下两件事：

• 将_item.Get的返回值转换为（ICollection）
• 其次使用 _item.Get("001").ToArray() 或 _item.Get("001").ToList()

请注意，第二个会导致阵列副本的性能下降。如果 Get 的签名（返回类型）不是 ICollection，则第一个将不起作用，如果不是 IEnumerable，则第二个将不起作用。

---

根据您对问题和 cmets 的澄清，我会亲自向 ICollection 声明 _item.Get("001") 的返回类型。这意味着您不必进行任何涉及不必要的创建/复制操作的转换或转换（通过 ToList / ToArray）。

// Leave this the same
public ICollection<Item> Items { get; set; }

// Change function signature here:
// As you mention Item uses the same underlying type, just return an ICollection<T>
public ICollection<Item> Get(string value); 

// Ideally here you want to call .Count on the collectoin, not .Count() on 
// IEnumerable, as this will result in a new Enumerator being created 
// per loop iteration
for (var index = 0; index < Items.Count(); index++) 

最好的问候，