C# построение общего метода

Question

var leftCurrent = leftArray.GetValue(i); 
var rightCurrent = rightArray.GetValue(i); 

var mi = typeof (PropertyCompare).GetMethod("NotEqualProperties"); 
mi.MakeGenericMethod(leftCurrent.GetType()); 

var notEqualProps = mi.Invoke(null,new []{leftCurrent, rightCurrent}); 

if(notEqualProps != null) 
    result.Add(new ArraysDiffResult(i, notEqualProps as List<string>)); 

Почему этот код вызывает InvalidOperationException (операции с поздней строкой не могут выполняться для типов или методов, для которых ContainsGenericParameters является истинным.)?

NotEqualProperties статический универсальный метод ..

UPD: Я уже нашел решение. Просто забыл назначить новый MethodInfo ... (Epic Fail ..)

Но как насчет производительности?

Answer 1

MakeGenericMethod возвращает новый экземпляр MethodInfo. (MethodInfo является неизменным)

Ваш код создает этот новый экземпляр, отбрасывает его, а затем продолжает использовать открытый (непараметрированный) MethodInfo.

Вы должны использовать новый экземпляр, как это:

mi = mi.MakeGenericMethod(leftCurrent.GetType()); 

---

Да; отражение намного медленнее, чем обычные вызовы методов.
Однако, если вы не вызываете его в узкой петле, это не обязательно проблема.

Answer 2

Вы не присвоить результат

mi.MakeGenericMethod(leftCurrent.GetType()); 

ни к чему. Обратите внимание, что MakeGenericMethod не мутирует вызывающий экземпляр.

P.S Этот код намного медленнее, чем метод вызова напрямую (без mi.Invoke)?

Много? Я не знаю. Единственный способ узнать - установить тесты производительности и профиль.

Answer 3

О, я глуп ... Он должен быть:

mi = mi.MakeGenericMethod(leftCurrent.GetType()); 

(Facepalm ...). Но как насчет производительности?