Nie można przekonwertować z IEnumerable <T> na ICollection <T>

Zdefiniowałem co następuje:

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

Kiedy uruchamiam ten kod:

Items = _item.Get("001");

Otrzymuję następujący komunikat:

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?)

Czy ktoś może wyjaśnić, co robię źle. Jestem bardzo zdezorientowany różnicą między Enumerable, Collections i użyciem ToList ()

Dodatkowe informacje

Później w moim kodzie mam:

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

Czy byłbym w stanie zdefiniować elementy jako IEnumerable?

ICollection<T>dziedziczy z IEnumerable<T>tak, aby przypisać wynik

IEnumerable<T> Get(string pk)

do ICollection<T>są dwa sposoby.

// 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();

Druga opcja jest bardziej elastyczna, ale ma znacznie większy wpływ na wydajność. Inną opcją jest zapisanie wyniku jako, IEnumerable<T>chyba że potrzebujesz dodatkowej funkcjonalności dodanej przez ICollection<T>interfejs.

Dodatkowy komentarz dotyczący wydajności

Pętla, którą masz

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

działa, IEnumerable<T>ale jest nieefektywny; każde wywołanie Count()wymaga pełnego wyliczenia wszystkich elementów. Użyj kolekcji i Countwłaściwości (bez nawiasów) lub przekonwertuj je na pętlę foreach:

foreach(var item in Items)

Nie możesz dokonać konwersji bezpośrednio z IEnumerable<T>na ICollection<T>. Możesz użyć ToListmetody, IEnumerable<T>aby przekonwertować go naICollection<T>

someICollection = SomeIEnumerable.ToList();

W oczekiwaniu na więcej informacji na pytanie:

proszę podać więcej informacji na temat rodzaju przedmiotu i podpisu Get

Możesz spróbować dwóch rzeczy:

• Aby rzutować wartość zwracaną przez _item. Get to (ICollection)
• po drugie, aby użyć _item.Get („001”). ToArray () lub _item.Get („001”). ToList ()

Należy pamiętać, że drugi spowoduje spadek wydajności kopii tablicy. Jeśli podpis (typ zwracany) Get nie jest ICollection, to pierwsza nie będzie działać, jeśli nie jest IEnumerable, to druga nie będzie działać.

Po wyjaśnieniu pytania i komentarzach osobiście zadeklarowałbym zwracany typ _item.Get („001”) do ICollection. Oznacza to, że nie będziesz musiał wykonywać żadnego rzutowania ani konwersji (przez ToList / ToArray), co wymagałoby niepotrzebnej operacji tworzenia / kopiowania.

// 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++) 

Z poważaniem,