Changeset 17478 for branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface/JsonItemConverter.cs

Timestamp:

03/17/20 14:55:23 (5 years ago)

Author:

dpiringe

Message:

#3026:

• removed type extension for type comparison (was a complicated comparison, which should not be a public extension method for Type)
  • comparison is now located (as private method) in JsonItemConverter
  • also simplified it a lot (did some tests to measure the relevance of specific checks -> last check got completely removed)

File:

• branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface/JsonItemConverter.cs (modified) (2 diffs)

branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface/JsonItemConverter.cs

@@ -40 +40 @@
       
       foreach (var x in Converters)
-        if (type.IsEqualTo(x.Key))
+        if (CompareTypes(type, x.Key))
           possibleConverters.Add(x.Value);
 
@@ -91 +91 @@
       Converters = converters;
     }
+
+    private bool CompareGenericTypes(Type t1, Type t2) =>
+      (t1.IsGenericType && t1.GetGenericTypeDefinition() == t2) ||
+      (t2.IsGenericType && t2.GetGenericTypeDefinition() == t1);
+
+    private bool CompareTypes(Type t1, Type t2) =>
+      t1 == t2 || t1.IsAssignableFrom(t2) ||
+      t1.GetInterfaces().Any(
+        i => i == t2 || CompareGenericTypes(i, t2)
+      ) ||
+      CompareGenericTypes(t1, t2);
   }
 }