Dlaczego funkcja przeciążania C# nie działa między funkcjami <T,T> i Action <T>?

Question

Tak, to dość powszechna metoda rozszerzenie dla 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; 
    } 
} 

Kiedy próbuję użyć tego z, na przykład, DbSet.Add:

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

... kompilator narzeka, że ma zły typ zwrotu, ponieważ oczekuje metody void. Więc dodać przeciążenie dla Run, które ma Func zamiast Działanie:

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

a teraz kompilator narzeka, że ​​„Wywołanie jest niejednoznaczna między następujących metod ...”

więc moje pytanie brzmi, w jaki sposób przeciążenie akcji metodą Run może powodować niejednoznaczność, gdy nie jest poprawne dla grupy metod?

Answer 1

Zostało to już wyjaśnione przez Eric i Jon w odpowiedzi na this question. Krótko mówiąc - tak działa kompilator C#; precyzyjnie, gdy ma do czynienia z konwersją grupy metoda decydowania co delegat zostanie skonwertowana do wykorzystuje rozdzielczość przeciążeniem, który nie bierze powrócić rodzajów konta:

Zasada tu jest, że określenie grupy metoda wymienialność wymaga wybierając metodę z grupy metod wykorzystującej rozdzielczość przeciążania, a rozdzielczość przeciążenia nie uwzględnia typów zwracanych.

W przykładzie kompilator widzi zarówno Action<T> i Func<T, T> jak Najlepszy mecz dla Add. Daje to dwa możliwe wybory, a ponieważ wymaga jednego - wydaje odpowiedni błąd.

Answer 2

try przeciążenie w odpowiedni sposób:

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

Answer 3

nie mogę odpowiedzieć dlaczego ale rozwiązać niejednoznaczności można jawnie rzucić swoją funkcję:

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

lub użyć lambda

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

Answer 4

Nie wiem, dlaczego nie może go automatycznie rozwiązać, ale tutaj są dwa obejścia:

// with T replaced with the actual type: 
invoice.Items.Run((Func<T, T>)db.InvoiceItems.Add); 
invoice.Items.Run(new Func<T, T>(db.InvoiceItems.Add)); 

Dlaczego i tak potrzebujesz tych metod? Co jest nie tak z:

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

Czytelność tego jest znacznie lepsza. Jeśli nie masz dobrego powodu, aby potrzebować metody Run, polecam jej używanie. Z tego co widziałem, nie ma takiego powodu, przynajmniej dla przeciążenia Action<T>.