Changeset 17407 for branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface/Converters/ValueTypeArrayConverter.cs

Timestamp:

01/27/20 12:18:06 (4 years ago)

Author:

dpiringe

Message:

#3026:

• simplified converter inheritance:
  • BaseConverter now only has Inject and Extract from IJsonItemConverter as abstract methods
  • removed ParameterBaseConverter
  • concrete converters have to initialize their JsonItem now -> enables better handling with different types of JsonItem

File:

• branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface/Converters/ValueTypeArrayConverter.cs (modified) (1 diff)

branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface/Converters/ValueTypeArrayConverter.cs

@@ -34 +34 @@
     where T : struct 
   {
-    public override void InjectData(IItem item, IJsonItem data, IJsonItemConverter root) => 
+    public override void Inject(IItem item, IJsonItem data, IJsonItemConverter root) => 
       CopyArrayData(((ArrayType)item), CastValue<T[]>(data.Value));
 
-    public override void Populate(IItem value, IJsonItem item, IJsonItemConverter root) {
-      item.Name = "[OverridableParamName]";
-      item.Value = ((ArrayType)value).CloneAsArray();
-    }
+    public override IJsonItem Extract(IItem value, IJsonItemConverter root) =>
+      new JsonItem() {
+        Name = "[OverridableParamName]",
+        Value = ((ArrayType)value).CloneAsArray()
+      };
 
     #region Helper