Si on ajoute le test sur les nullables, ta méthode fonctionne et a pour elle d’être concise. C’est probablement celle que j’aurais utilisée.

Une seule (petite) remarque cependant : le mot clé default repose sur l’opération initobj en CIL, ce qui le rend plus performant qu’une instanciation dynamique via la méthode CreateInstance. Lorsqu’on est dans un contexte générique, ou qu’on connait le type de l’objet, autant l’utiliser…

public bool IsDefaultValue(Type type, object value)
{
if (type == null)
throw new ArgumentNullException(“type”);

if (!type.IsValueType)
return (value == null);

if (value == null)
return false;

return ((ValueType)value).Equals(Activator.CreateInstance(type));
}

1. Appel à GetMethod pour récupérer les infos de la méthode générique
2. Appel à MakeGenericMethod pour récupérer les infos de la méthode générique fermée
3. Creation d’un délégué via l’appel à CreateDelegate sur la classe statique Delegate.

Celle qui implique la LCG reprend les 2 premières étapes, puis ajoute :

3. Création d’une DynamicMethod et génération de son corps en faisant référence au MethodInfo récupéré à l’étape 2.
4. Creation d’un délégué via l’appel à CreateDelegate sur la classe statique DynamicMethod.

La génération d’une méthode dynamique est ici inutile, puisqu’il n’y a qu’une invocation de méthode à faire. la LCG devient utile dans des scénarios plus complexes. Je vais essayer de faire un petit post à ce sujet.