Konvertieren von einer IEnumerable <T> in eine ICollection <T> nicht möglich

Ich habe folgendes definiert:

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

Wenn ich diesen Code ausführe:

Items = _item.Get("001");

Ich erhalte die folgende Nachricht:

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

Kann jemand erklären, was ich falsch mache. Ich bin sehr verwirrt über den Unterschied zwischen Enumerable, Collections und der Verwendung der ToList ()

Informationen hinzugefügt

Später in meinem Code habe ich Folgendes:

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

Wäre es in Ordnung, Elemente als IEnumerable zu definieren?

ICollection<T>erbt von IEnumerable<T>so, um das Ergebnis von zuzuweisen

IEnumerable<T> Get(string pk)

Zu einem ICollection<T>gibt es zwei Möglichkeiten.

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

Die zweite Option ist flexibler, hat jedoch einen viel größeren Einfluss auf die Leistung. Eine andere Möglichkeit besteht darin, das Ergebnis als zu speichern, es IEnumerable<T>sei denn, Sie benötigen die zusätzliche Funktionalität, die von der ICollection<T>Schnittstelle hinzugefügt wird .

Zusätzlicher Leistungskommentar

Die Schleife, die Sie haben

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

arbeitet an einem, IEnumerable<T>aber es ist ineffizient; Jeder Aufruf von Count()erfordert eine vollständige Aufzählung aller Elemente. Verwenden Sie entweder eine Sammlung und die CountEigenschaft (ohne die Klammer) oder konvertieren Sie sie in eine foreach-Schleife:

foreach(var item in Items)

Sie können nicht direkt von IEnumerable<T>nach konvertieren ICollection<T>. Sie können die ToListMethode verwenden IEnumerable<T>, um es zu konvertierenICollection<T>

someICollection = SomeIEnumerable.ToList();

Bis weitere Informationen zu der Frage vorliegen:

Bitte geben Sie weitere Informationen zur Art des Artikels und zur Unterschrift von Get an

Zwei Dinge, die Sie ausprobieren können, sind:

• Um den Rückgabewert von _item.Get to (ICollection) umzuwandeln
• zweitens, um _item.Get ("001") zu verwenden. ToArray () oder _item.Get ("001"). ToList ()

Bitte beachten Sie, dass die zweite einen Leistungseinbruch für die Array-Kopie zur Folge hat. Wenn die Signatur (Rückgabetyp) von Get keine ICollection ist, funktioniert die erste nicht. Wenn sie nicht IEnumerable ist, funktioniert die zweite nicht.

Nach Ihrer Klarstellung zur Frage und in Kommentaren würde ich persönlich den Rückgabetyp von _item.Get ("001") an ICollection deklarieren. Dies bedeutet, dass Sie kein Casting oder keine Konvertierung (über ToList / ToArray) durchführen müssen, die einen unnötigen Erstellungs- / Kopiervorgang erfordern würde.

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

Freundliche Grüße,