Changeset 15034 for branches/PersistenceReintegration/HeuristicLab.Persistence/4.0/Transformers/StorableClassTransformer.cs

Timestamp:

06/16/17 11:32:56 (7 years ago)

Author:

gkronber

Message:

#2520: improved conversions

File:

• branches/PersistenceReintegration/HeuristicLab.Persistence/4.0/Transformers/StorableClassTransformer.cs (modified) (3 diffs)

branches/PersistenceReintegration/HeuristicLab.Persistence/4.0/Transformers/StorableClassTransformer.cs

@@ -72 +72 @@
       var typeBox = mapper.GetBox(box.TypeId).GetExtension(TypeBox.Type);
       var version = typeBox.HasVersion ? typeBox.Version : 1;
+      var typeGuid = typeInfo.StorableTypeAttribute.Guid;
 
       var components = new Dictionary<uint, uint>();
@@ -79 +80 @@
 
       var conversionMethods =
-        type.GetMethods(BindingFlags.NonPublic | BindingFlags.Static)
+        type.Assembly.GetTypes().SelectMany(t =>
+          t.GetMethods(BindingFlags.NonPublic | BindingFlags.Static)
           .Where(StorableConversionAttribute.IsStorableConversionMethod)
-          .Where(mi => StorableConversionAttribute.GetVersion(mi) >= version)
-          .OrderBy(StorableConversionAttribute.GetVersion);
+          .Where(mi => StorableConversionAttribute.GetGuid(mi) == typeGuid &&
+                       StorableConversionAttribute.GetVersion(mi) >= version))
+          .OrderBy(StorableConversionAttribute.GetVersion)
+          .ToArray();
 
-      // put all objects into dictionary for optional conversion
+      // put all objects into dictionary for conversion 
       var dict = new Dictionary<string, object>();
       foreach (var component in components) {
@@ -90 +94 @@
       }
 
-      // TODO: check that all entries in the dictionary can be mapped to a field or property
       foreach (var convMeth in conversionMethods) {
+        if (StorableConversionAttribute.GetVersion(convMeth) != version)
+          throw new PersistenceException(string.Format("No conversion method defined for type {0} version {1}", typeGuid, version));
         dict = (Dictionary<string, object>)convMeth.Invoke(null, new object[] { dict });
+        version++;
+      }
+      if (version != typeInfo.StorableTypeAttribute.Version)
+        throw new PersistenceException(string.Format("Missing one or more conversion methods for type {0} version {1}",
+          typeGuid, typeInfo.StorableTypeAttribute.Version));
+
+      // set default values for all fields and properties
+      foreach (var componentInfo in typeInfo.Fields) {
+        var field = (FieldInfo)componentInfo.MemberInfo;
+        if (componentInfo.StorableAttribute != null && componentInfo.StorableAttribute.DefaultValue != null)
+          field.SetValue(obj, componentInfo.StorableAttribute.DefaultValue);
+      }
+      foreach (var componentInfo in typeInfo.Properties.Where(x => x.Writeable)) {
+        var property = (PropertyInfo)componentInfo.MemberInfo;
+        if (componentInfo.StorableAttribute != null && componentInfo.StorableAttribute.DefaultValue != null)
+          property.SetValue(obj, componentInfo.StorableAttribute.DefaultValue, null);
       }
 
-      foreach (var componentInfo in typeInfo.Fields) {
-        var field = (FieldInfo)componentInfo.MemberInfo;
-        object val = null;
-        bool found = dict.TryGetValue(componentInfo.Name, out val);
-        if (found)
+      // set all members as generated by conversion method chain
+      foreach (var kvp in dict.ToArray()) {
+        var key = kvp.Key;
+        var val = kvp.Value;
+        var fieldInfo = typeInfo.Fields.FirstOrDefault(fi => fi.Name == key);
+        if (fieldInfo != null) {
+          var field = (FieldInfo)fieldInfo.MemberInfo;
           field.SetValue(obj, val);
-        else if (componentInfo.StorableAttribute.DefaultValue != null)
-          field.SetValue(obj, componentInfo.StorableAttribute.DefaultValue);
+          dict.Remove(fieldInfo.Name);
+          continue;
+        }
+        var propInfo = typeInfo.Properties.Where(x => x.Writeable).FirstOrDefault(pi => pi.Name == key);
+        if (propInfo != null) {
+          var prop = (PropertyInfo)propInfo.MemberInfo;
+          prop.SetValue(obj, val, null);
+          dict.Remove(propInfo.Name);
+          continue;
+        }
       }
 
-      foreach (var componentInfo in typeInfo.Properties.Where(x => x.Writeable)) {
-        var property = (PropertyInfo)componentInfo.MemberInfo;
-        object val = null;
-        bool found = dict.TryGetValue(componentInfo.Name, out val);
-        if (found)
-          property.SetValue(obj, val, null);
-        else if (componentInfo.StorableAttribute.DefaultValue != null)
-          property.SetValue(obj, componentInfo.StorableAttribute.DefaultValue, null);
-      }
+      if (dict.Any())
+        throw new PersistenceException(string.Format("Invalid conversion method. The following members are undefined in type {0} version {1}: {2}",
+          typeGuid, typeInfo.StorableTypeAttribute.Version,
+          string.Join(", ", dict.Keys)));
 
       var emptyArgs = new object[0];