Changeset 17354 for branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface

Timestamp:

11/19/19 16:59:18 (5 years ago)

Author:

dpiringe

Message:

#3026:

• relocated BuildJsonItem from JCInstantiator into JsonItem
• in JsonItem: removed JContainer usage in setter for Value (now in BuildJsonItem)

Location:

branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface

Files:

• JCGenerator.cs (modified) (3 diffs)
• JCInstantiator.cs (modified) (3 diffs)
• JsonItem.cs (modified) (7 diffs)

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

@@ -75 +75 @@
 
       obj.Property(nameof(JsonItem.Parameters))?.Remove();
-      RefactorFreeParameters(obj);
-      RefactorStaticParameters(obj);
+      RefactorFreeParameters(obj, genData);
+      RefactorStaticParameters(obj, genData);
 
       obj.Property(nameof(JsonItem.Value))?.Remove();
@@ -87 +87 @@
 
     // deletes unnecessary properties for free parameters.
-    private static void RefactorFreeParameters(JToken token) {
+    private static void RefactorFreeParameters(JToken token, GenData genData) {
       IList<JObject> objToRemove = new List<JObject>();
       TransformNodes(x => {
-        var p = x.ToObject<JsonItem>();
+        var p = JsonItem.BuildJsonItem(x, genData.TypeList);
         x.Property(nameof(JsonItem.Type))?.Remove();
         x.Property(nameof(JsonItem.Parameters))?.Remove();
@@ -105 +105 @@
 
     // deletes unnecessary properties for static parameters.
-    private static void RefactorStaticParameters(JToken token) {
+    private static void RefactorStaticParameters(JToken token, GenData genData) {
       IList<JObject> objToRemove = new List<JObject>();
       TransformNodes(x => {
-        var p = x.ToObject<JsonItem>();
+        var p = JsonItem.BuildJsonItem(x, genData.TypeList);
         x.Property(nameof(JsonItem.Range))?.Remove();
         x.Property(nameof(JsonItem.Operators))?.Remove();

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

@@ -83 +83 @@
     private static void CollectParameterizedItems(InstData instData) {
       foreach (JObject item in instData.Template[Constants.Objects]) {
-        JsonItem data = BuildJsonItem(item, instData);
+        JsonItem data = JsonItem.BuildJsonItem(item, instData.TypeList);
         instData.ParameterizedItems.Add(data.Path, data);
       }
@@ -120 +120 @@
       foreach (JObject obj in instData.Config) {
         // build item from config object
-        JsonItem item = BuildJsonItem(obj, instData);
+        JsonItem item = JsonItem.BuildJsonItem(obj, instData.TypeList);
         // override default value
         if (instData.ConfigurableItems.TryGetValue(item.Path, out JsonItem param)) {
@@ -145 +145 @@
       } else throw new TypeLoadException($"Cannot find AssemblyQualifiedName for {data.Name}.");
     }
-
-    #region BuildJsonItemMethods
-    private static JsonItem BuildJsonItem(JObject obj, InstData instData) =>
-      new JsonItem() {
-        Name = obj[nameof(JsonItem.Name)]?.ToString(),
-        Path = obj[nameof(JsonItem.Path)]?.ToString(),
-        Value = obj[nameof(JsonItem.Value)]?.ToObject<object>(),
-        Range = obj[nameof(JsonItem.Range)]?.ToObject<object[]>(),
-        Type = GetType(obj[nameof(JsonItem.Path)]?.ToObject<string>(), instData),
-        ActualName = obj[nameof(JsonItem.ActualName)]?.ToString(),
-        Parameters = PopulateParameters(obj, instData),
-        Operators = PopulateOperators(obj, instData)
-      };
-
-    private static string GetType(string path, InstData instData) {
-      if(!string.IsNullOrEmpty(path))
-        if (instData.TypeList.TryGetValue(path, out string value))
-          return value;
-      return null;
-    }
-
-    private static IList<JsonItem> PopulateParameters(JObject obj, InstData instData) {
-      IList<JsonItem> list = new List<JsonItem>();
-
-      // add staticParameters
-      if (obj[Constants.StaticParameters] != null)
-        foreach (JObject param in obj[Constants.StaticParameters])
-          list.Add(BuildJsonItem(param, instData));
-
-      // merge staticParameter with freeParameter
-      if (obj[Constants.FreeParameters] != null) {
-        foreach (JObject param in obj[Constants.FreeParameters]) {
-          JsonItem tmp = BuildJsonItem(param, instData);
-          
-          // search staticParameter from list
-          JsonItem comp = null;
-          foreach (var p in list)
-            if (p.Name == tmp.Name) comp = p;
-          if (comp == null)
-            throw new InvalidDataException($"Invalid {Constants.FreeParameters.Trim('s')}: '{tmp.Name}'!");
-
-          JsonItem.Merge(comp, tmp);
-        }
-      }
-      return list;
-    }
-
-    private static IList<JsonItem> PopulateOperators(JObject obj, InstData instData) {
-      IList<JsonItem> list = new List<JsonItem>();
-      JToken operators = obj[nameof(JsonItem.Operators)];
-      if (operators != null)
-        foreach (JObject sp in operators)
-          list.Add(BuildJsonItem(sp, instData));
-      return list;
-    }
-    #endregion
     #endregion
   }

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

@@ -1 +1 @@
 using System;
 using System.Collections.Generic;
+using System.IO;
 using System.Linq;
 using System.Text;
@@ -19 +20 @@
     private IEnumerable<object> range;
     #endregion
-    
-    public string Name { 
-      get => name; 
+
+    public string Name {
+      get => name;
       set {
         name = value;
         Path = Name;
         UpdatePath();
-      } 
+      }
     }
     public string Type { get; set; }
@@ -33 +34 @@
     public IList<JsonItem> Operators { get; set; }
     public object Value {
-      get => value; 
+      get => value;
       set {
-        if (value is JContainer)
-          this.value = ((JContainer)value).ToObject<object[]>();
-        else
-          this.value = value;
+        this.value = value;
         CheckConstraints();
-      } 
+      }
     }
-    public IEnumerable<object> Range { 
-      get => range; 
+    public IEnumerable<object> Range {
+      get => range;
       set {
         range = value;
         CheckConstraints();
-      } 
+      }
     }
     public string ActualName { get; set; }
@@ -84 +82 @@
         UpdatePathHelper(Operators);
 
-      if(Reference != null)
+      if (Reference != null)
         UpdatePathHelper(Reference);
     }
@@ -90 +88 @@
 
     #region Helper
-    private void UpdatePathHelper(params JsonItem[] items) => 
+    private void UpdatePathHelper(params JsonItem[] items) =>
       UpdatePathHelper((IEnumerable<JsonItem>)items);
 
@@ -105 +103 @@
     }
 
-    private bool IsInRange() => IsInRangeList() || 
+    private bool IsInRange() => IsInRangeList() ||
       (Value.GetType().IsArray && ((object[])Value).All(x => IsInNumericRange(x))) ||
       (!Value.GetType().IsArray && IsInNumericRange(Value));
@@ -129 +127 @@
             (((T)max).CompareTo(value) == 1 || ((T)max).CompareTo(value) == 0);
     }
-      
+
+    #endregion
+
+    #region BuildJsonItemMethods
+    public static JsonItem BuildJsonItem(JObject obj, IDictionary<string, string> typeList) {
+      object val = obj[nameof(Value)]?.ToObject<object>();
+      if (val is JContainer)
+        val = ((JContainer)val).ToObject<object[]>();
+
+      return new JsonItem() {
+        Name = obj[nameof(Name)]?.ToString(),
+        Path = obj[nameof(Path)]?.ToString(),
+        Value = val,
+        Range = obj[nameof(Range)]?.ToObject<object[]>(),
+        Type = GetType(obj[nameof(Path)]?.ToObject<string>(), typeList),
+        ActualName = obj[nameof(ActualName)]?.ToString(),
+        Parameters = PopulateParameters(obj, typeList),
+        Operators = PopulateOperators(obj, typeList)
+      };
+  }
+
+    private static string GetType(string path, IDictionary<string, string> typeList) {
+      if (!string.IsNullOrEmpty(path))
+        if (typeList.TryGetValue(path, out string value))
+          return value;
+      return null;
+    }
+
+    private static IList<JsonItem> PopulateParameters(JObject obj, IDictionary<string, string> typeList) {
+      IList<JsonItem> list = new List<JsonItem>();
+
+      // add staticParameters
+      if (obj[Constants.StaticParameters] != null)
+        foreach (JObject param in obj[Constants.StaticParameters])
+          list.Add(BuildJsonItem(param, typeList));
+
+      // merge staticParameter with freeParameter
+      if (obj[Constants.FreeParameters] != null) {
+        foreach (JObject param in obj[Constants.FreeParameters]) {
+          JsonItem tmp = BuildJsonItem(param, typeList);
+
+          // search staticParameter from list
+          JsonItem comp = null;
+          foreach (var p in list)
+            if (p.Name == tmp.Name) comp = p;
+          if (comp == null)
+            throw new InvalidDataException($"Invalid {Constants.FreeParameters.Trim('s')}: '{tmp.Name}'!");
+
+          JsonItem.Merge(comp, tmp);
+        }
+      }
+      return list;
+    }
+
+    private static IList<JsonItem> PopulateOperators(JObject obj, IDictionary<string, string> typeList) {
+      IList<JsonItem> list = new List<JsonItem>();
+      JToken operators = obj[nameof(JsonItem.Operators)];
+      if (operators != null)
+        foreach (JObject sp in operators)
+          list.Add(BuildJsonItem(sp, typeList));
+      return list;
+    }
     #endregion
   }