Warum funktioniert die Überladungsauflösung von C# nicht zwischen Func <T,T> und Action <T>?

Question

also eine recht häufige Erweiterungsmethode für IEnumerable, Run:

public static IEnumerable<T> Run<T>(this IEnumerable<T> source, Action<T> action) 
{ 
    foreach (var item in source) 
    { 
     action(item); 
     yield return item; 
    } 
} 

Wenn ich versuche, das zu verwenden, mit zum Beispiel DbSet.Add:

invoice.Items.Run(db.InvoiceItems.Add); 
// NB: Add method signature is 
// public T Add(T item) { ... } 

... der Compiler beschwert mich, dass Es hat den falschen Rückgabetyp, weil es eine ungültige Methode erwartet. So fügen Sie eine Überlastung für Run, die eine Func statt Aktion nimmt:

public static IEnumerable<T> Run<T>(this IEnumerable<T> source, Func<T, T> action) 
{ 
    return source.Select(action).ToList().AsEnumerable(); 
} 

Und nun der Compiler beschwert sich, dass „Der Anruf zwischen den folgenden Methoden nicht eindeutig ist ...“

Also meine Frage ist, Wie kann die Action-Überladung der Run-Methode zu Mehrdeutigkeiten führen, wenn sie für die Methodengruppe nicht gültig ist?

Answer 1

Dies wurde bereits von Eric und Jon in Antworten auf this question erklärt. Lange Rede, kurzer Sinn - so funktioniert der C# -Compiler; Gerade, wenn es mit Methode Gruppenumwandlung zu tun zu entscheiden, welche Delegierten wird es Überladungsauflösung verwendet werden umgewandelt, die nicht zurück nehmen Arten in Rechnung:

Das Prinzip dabei ist, dass die Konvertierbarkeit Bestimmungsverfahren Gruppe erfordert ein Verfahren der Auswahl aus einer Methodengruppe, die die Überladungsauflösung verwendet, und die Überladungsauflösung berücksichtigt keine Rückgabetypen.

In Ihrem Beispiel Compiler sieht sowohl Action<T> und Func<T, T> als beste Übereinstimmung für Add. Dies führt zu zwei möglichen Auswahlmöglichkeiten, und da es einen erfordert, wird ein entsprechender Fehler ausgegeben.

Answer 2

try Überlastung in der rechten Weise:

public static IEnumerable<TDest> Run<TSource, TDest>(this IEnumerable<TSource> source, 
    Func<TSource, TDest> action) 
{ 
return source.Select(action).ToList(); 
} 

Answer 3

kann ich nicht beantworten, warum aber die Zweideutigkeit lösen können Sie ausdrücklich Ihre Funktion Stimmen:

invoice.Items.Run((Func<T,T>)db.InvoiceItems.Add); 

oder eine Lambda verwenden

invoice.Items.Run(x => db.InvoiceItems.Add(x)); 

Answer 4

Ich weiß nicht, warum es nicht automatisch lösen kann, aber hier sind zwei Problemumgehungen:

Warum brauchen Sie diese Methoden überhaupt? Was ist los mit:

foreach (var item in invoice.Items) 
    db.InvoiceItems.Add(item); 

Die Lesbarkeit dieser ist viel besser. Wenn Sie keinen guten Grund haben, die Run Methode zu verwenden, würde ich empfehlen, sie nicht zu verwenden. Von dem, was ich gesehen habe, gibt es keinen solchen Grund, zumindest für die Action<T> Überlastung.