Changeset 15034 for branches/PersistenceReintegration/HeuristicLab.Persistence

Timestamp:

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

Author:

gkronber

Message:

#2520: improved conversions

Location:

branches/PersistenceReintegration/HeuristicLab.Persistence/4.0

Files:

• Core/StorableConversionAttribute.cs (modified) (2 diffs)
• Core/StorableTypeAttribute.cs (modified) (1 diff)
• Core/TypeInfo.cs (modified) (2 diffs)
• Transformers/StorableClassTransformer.cs (modified) (3 diffs)
• Transformers/Transformers.cs (modified) (1 diff)

branches/PersistenceReintegration/HeuristicLab.Persistence/4.0/Core/StorableConversionAttribute.cs

@@ -26 +26 @@
   [AttributeUsage(AttributeTargets.Method, AllowMultiple = false, Inherited = false)]
   public sealed class StorableConversionAttribute : Attribute {
-    public uint SrcVersion { get; set; }
+    public Guid Guid { get; private set; }
+    public uint SrcVersion { get; private set; }
 
-    public StorableConversionAttribute(uint srcVersion) {
+    public StorableConversionAttribute(string guid, uint srcVersion) {
+      this.Guid = new Guid(guid);
       this.SrcVersion = srcVersion;
     }
@@ -35 +37 @@
       return Attribute.IsDefined(mi, typeof(StorableConversionAttribute), false);
     }
-    public static StorableConversionAttribute GetTransformerAttribute(MethodInfo mi) {
+
+    public static StorableConversionAttribute GetStorableConversionAttribute(MethodInfo mi) {
       return (StorableConversionAttribute)Attribute.GetCustomAttribute(mi, typeof(StorableConversionAttribute), false);
     }
+    public static Guid GetGuid(MethodInfo mi) {
+      return GetStorableConversionAttribute(mi).Guid;
+    }
     public static uint GetVersion(MethodInfo mi) {
-      return GetTransformerAttribute(mi).SrcVersion;
+      return GetStorableConversionAttribute(mi).SrcVersion;
     }
   }

branches/PersistenceReintegration/HeuristicLab.Persistence/4.0/Core/StorableTypeAttribute.cs

@@ -37 +37 @@
 
     public Guid Guid { get; private set; }
-    public uint Version { get; set; }
+    public uint Version { get; private set; }
 
     /// <summary>

branches/PersistenceReintegration/HeuristicLab.Persistence/4.0/Core/TypeInfo.cs

@@ -59 +59 @@
       if (StorableTypeAttribute != null) {
         // check constructors (
-        if (!type.IsValueType && !type.IsEnum && !type.IsInterface && 
+        if (!type.IsValueType && !type.IsEnum && !type.IsInterface &&
           GetStorableConstructor() == null && GetDefaultConstructor() == null)
           throw new PersistenceException("No storable constructor or parameterless constructor found.");
 
-          // traverse type hierarchy from base type to sub types
-          Stack<Type> types = new Stack<Type>();
+        // traverse type hierarchy from base type to sub types
+        Stack<Type> types = new Stack<Type>();
         while (type != null) {
           types.Push(type);
@@ -95 +95 @@
             foreach (var property in propertyInfos) {
               var attrib = StorableAttribute.GetStorableAttribute(property);
-              if (!attrib.AllowOneWay && (!property.CanRead || !property.CanWrite))
+              if ((!property.CanRead || !property.CanWrite) && (attrib == null || !attrib.AllowOneWay))
                 throw new PersistenceException("Properties must be readable and writable or explicity enable one way serialization.");
 

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

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

@@ -332 +332 @@
         return ((Type)mapper.GetObject(b.GetGenericTypeIds(0))).MakeArrayType();
       } else {
+        
         return type;
       }