C# construction de la méthode générique

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

Pourquoi ce code lance-t-il InvalidOperationException (Les opérations liées tardivement ne peuvent pas être effectuées sur les types ou les méthodes pour lesquels ContainsGenericParameters est vrai.)?

NotEqualProperties est méthode générique statique ..

UPD: Je l'ai déjà trouvé la solution. Juste oublié d'assigner de nouvelles MethodInfo ... (Epic Fail ..)

Mais que diriez-vous de la performance?

Answer 1

MakeGenericMethod renvoie une nouvelle instance MethodInfo. (MethodInfo est immuable)

Votre code crée cette nouvelle instance, la rejette, puis continue à utiliser le code ouvert (non paramétré) MethodInfo.

Vous devez utiliser la nouvelle instance, comme ceci:

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

---

Oui; la réflexion est beaucoup plus lente que les appels de méthode normaux.
Cependant, à moins que vous ne l'appeliez dans une boucle serrée, ce n'est pas nécessairement un problème.

Answer 2

Vous n'avez pas affecté le résultat de

mi.MakeGenericMethod(leftCurrent.GetType()); 

à quoi que ce soit. Notez que MakeGenericMethod ne mute pas l'instance invoquante.

P.S Ce code est-il beaucoup plus lent que la méthode d'appel directement (sans mi.Invoke)?

Beaucoup? Je ne sais pas. La seule façon de savoir est de définir des benchmarks de performance et de profiler.

Answer 3

Oh, je suis stupide ... Il devrait être:

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

(Facepalm ...). Mais qu'en est-il de la performance?