Changeset 17407 for branches/3026_IntegrationIntoSymSpace/HeuristicLab.JsonInterface/Converters/BaseConverter.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/BaseConverter.cs (modified) (1 diff)

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

@@ -14 +14 @@
     public abstract Type ConvertableType { get; }
 
-    public void Inject(IItem item, IJsonItem data, IJsonItemConverter root) {
-
-      InjectData(item, data, root);
-    }
-
-    public IJsonItem Extract(IItem value, IJsonItemConverter root) {
-      IJsonItem data = new JsonItem() { Name = value.ItemName };
-      Populate(value, data, root);
-      return data;
-    }
-    
-    public abstract void InjectData(IItem item, IJsonItem data, IJsonItemConverter root);
-    public abstract void Populate(IItem value, IJsonItem item, IJsonItemConverter root); // TODO: populate?
+    public abstract void Inject(IItem item, IJsonItem data, IJsonItemConverter root);
+    public abstract IJsonItem Extract(IItem value, IJsonItemConverter root);
 
     #region Helper