Changeset 16210 for branches/PersistenceReintegration/HeuristicLab.Persistence/4.0/Core/StorableAttribute.cs

Timestamp:

10/03/18 17:48:20 (6 years ago)

Author:

jkarder

Message:

#2520: worked on new persistence

• added cache for different attributes
• refactored StorableTypeBoxTransformer
• implemented paths for Storable members
• implemented more unit tests

File:

• branches/PersistenceReintegration/HeuristicLab.Persistence/4.0/Core/StorableAttribute.cs (modified) (3 diffs)

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

@@ -21 +21 @@
 
 using System;
+using System.Collections.Generic;
 using System.Reflection;
-using System.Text;
 
 namespace HeuristicLab.Persistence {
-
-
   /// <summary>
   /// Mark the member of a class to be considered by the <c>StorableSerializer</c>.
@@ -36 +34 @@
     AttributeTargets.Field | AttributeTargets.Property,
     AllowMultiple = false,
-    Inherited = false)]
-  public class StorableAttribute : Attribute {
-    public static bool IsStorable(MemberInfo memberInfo) {
-      return Attribute.IsDefined(memberInfo, typeof(StorableAttribute), false);
-    }
-    public static StorableAttribute GetStorableAttribute(MemberInfo memberInfo) {
-      return (StorableAttribute)Attribute.GetCustomAttribute(memberInfo, typeof(StorableAttribute), false);
-    }
+    Inherited = false
+  )]
+  public sealed class StorableAttribute : Attribute {
+    private static IDictionary<MemberInfo, StorableAttribute> attributeCache = new Dictionary<MemberInfo, StorableAttribute>();
 
+    #region Properties
     /// <summary>
     /// An optional name for this member that will be used during serialization.
@@ -67 +62 @@
     /// deserialized (setter only) but not serialized again.
     /// </summary>
+    [Obsolete("Use OldName instead.")]
     public bool AllowOneWay { get; set; }
 
-    /// <summary>
-    /// Returns a <see cref="System.String"/> that represents this instance.
-    /// </summary>
-    /// <returns>
-    /// A <see cref="System.String"/> that represents this instance.
-    /// </returns>
-    public override string ToString() {
-      StringBuilder sb = new StringBuilder();
-      sb.Append("[Storable");
-      if (Name != null || DefaultValue != null)
-        sb.Append('(');
-      if (Name != null) {
-        sb.Append("Name = \"").Append(Name).Append("\"");
-        if (DefaultValue != null)
-          sb.Append(", ");
+    public string OldName { get; set; }
+    #endregion
+
+    public static bool IsStorable(MemberInfo memberInfo) {
+      return GetStorableAttribute(memberInfo) != null;
+    }
+    public static StorableAttribute GetStorableAttribute(MemberInfo memberInfo) {
+      StorableAttribute attrib;
+
+      if (!attributeCache.TryGetValue(memberInfo, out attrib)) {
+        attrib = (StorableAttribute)GetCustomAttribute(memberInfo, typeof(StorableAttribute), false);
+        if (attrib != null) attributeCache[memberInfo] = attrib;
       }
-      if (DefaultValue != null)
-        sb.Append("DefaultValue = \"").Append(DefaultValue).Append("\"");
-      if (Name != null || DefaultValue != null)
-        sb.Append(')');
-      sb.Append(']');
-      return sb.ToString();
+
+      return attrib;
     }
   }