changeset 7357:9d3e44ab25f2

Refactoring: Move functionality of BedHeightAccess into BedHeightFacet for now. Idea is that Artifact and Access are lightweight. Access access the 'data' ('parameterization') attached to artifact, not the data delivered by means of artifact and its parameterization.
author Felix Wolfsteller <felix.wolfsteller@intevation.de>
date Wed, 16 Oct 2013 10:42:45 +0200
parents 534850c9fabc
children bb1f05a235f0
files artifacts/src/main/java/org/dive4elements/river/artifacts/access/BedHeightAccess.java artifacts/src/main/java/org/dive4elements/river/artifacts/model/minfo/BedHeightFacet.java
diffstat 2 files changed, 43 insertions(+), 32 deletions(-) [+]
line wrap: on
line diff
--- a/artifacts/src/main/java/org/dive4elements/river/artifacts/access/BedHeightAccess.java	Wed Oct 16 10:33:09 2013 +0200
+++ b/artifacts/src/main/java/org/dive4elements/river/artifacts/access/BedHeightAccess.java	Wed Oct 16 10:42:45 2013 +0200
@@ -11,11 +11,7 @@
 import java.util.List;
 
 import org.dive4elements.river.artifacts.D4EArtifact;
-import org.dive4elements.river.artifacts.model.minfo.BedHeight;
-import org.dive4elements.river.artifacts.model.minfo.BedHeightFactory;
 import org.dive4elements.river.artifacts.states.SoundingsSelect;
-import org.dive4elements.river.model.BedHeightSingle;
-import org.dive4elements.river.model.BedHeightSingleValue;
 
 import gnu.trove.TIntArrayList;
 
@@ -33,8 +29,12 @@
 
     private String yearEpoch;
 
+    private String type;
+
     private Integer heightId;
 
+    private Integer time;
+
     public BedHeightAccess(D4EArtifact artifact) {
         super(artifact);
     }
@@ -91,11 +91,11 @@
     }
 
 
-    public String getYearEpoch() {
-        if (yearEpoch == null) {
-            yearEpoch =  getString("ye_select");
+    public String getType() {
+        if (type == null) {
+            type = getString("type");
         }
-        return yearEpoch;
+        return type;
     }
 
     public String getYearEpoch() {
@@ -138,30 +138,18 @@
         return epochIDs;
     }
 
-    /**
-     * Return the {@link BedHeight} at the height_id and time of the artifact
-     * @return {@link BedHeight}
-     */
-    public BedHeight getHeight() {
-        logger.debug("getHeight");
-        return BedHeightFactory.getHeight(
-            artifact.getDataAsString("type"),
-            Integer.parseInt(artifact.getDataAsString("height_id")),
-            Integer.parseInt(artifact.getDataAsString("time")));
+    public Integer getHeightId() {
+        if (heightId == null) {
+            heightId = getInteger("height_id");
+        }
+        return heightId;
     }
 
-    /**
-     * Return a {@link List} of {@link BedHeightSingleValue}s
-     * at the range of the artifact
-     * @return List of {@link BedHeightSingleValue}s
-     */
-    public List<BedHeightSingleValue> getSingleValues() {
-        logger.debug("getSingleValues");
-        BedHeightSingle single = BedHeightSingle.getBedHeightSingleById(
-                artifact.getDataAsInteger("height_id"));
-        return BedHeightSingleValue.getBedHeightSingleValues(single,
-                getFrom(),
-                getTo());
+    public Integer getTime() {
+        if (time == null) {
+            time = getInteger("time");
+        }
+        return time;
     }
 }
 // vim:set ts=4 sw=4 si et sta sts=4 fenc=utf8 :
--- a/artifacts/src/main/java/org/dive4elements/river/artifacts/model/minfo/BedHeightFacet.java	Wed Oct 16 10:33:09 2013 +0200
+++ b/artifacts/src/main/java/org/dive4elements/river/artifacts/model/minfo/BedHeightFacet.java	Wed Oct 16 10:42:45 2013 +0200
@@ -13,7 +13,11 @@
 import org.dive4elements.river.artifacts.D4EArtifact;
 import org.dive4elements.river.artifacts.access.BedHeightAccess;
 import org.dive4elements.river.artifacts.model.BlackboardDataFacet;
+import org.dive4elements.river.artifacts.model.minfo.BedHeight;
+import org.dive4elements.river.artifacts.model.minfo.BedHeightFactory;
 import org.dive4elements.river.artifacts.model.FacetTypes;
+import org.dive4elements.river.model.BedHeightSingle;
+import org.dive4elements.river.model.BedHeightSingleValue;
 
 public class BedHeightFacet
 extends      BlackboardDataFacet
@@ -40,10 +44,29 @@
     public Object getData(Artifact artifact, CallContext context) {
         BedHeightAccess access = new BedHeightAccess((D4EArtifact)artifact);
         if (type.equals("singlevalues")) {
-            return access.getSingleValues();
+            /* Former doc (from BedHeightAccess):
+             * Return a {@link List} of {@link BedHeightSingleValue}s
+             * at the range of the artifact
+             * @return List of {@link BedHeightSingleValue}s
+             */
+            BedHeightSingle single = BedHeightSingle.getBedHeightSingleById(
+                    access.getHeightId());
+            return BedHeightSingleValue.getBedHeightSingleValues(single,
+                    access.getFrom(),
+                    access.getTo());
         }
-        return access.getHeight();
+        else {
+            /* Former doc (from BedHeightAccess):
+             * Return the {@link BedHeight} at the height_id and time of the artifact
+             * @return {@link BedHeight}
+             */
+            return BedHeightFactory.getHeight(
+                access.getType(),
+                access.getHeightId(),
+                access.getTime());
+        }
     }
+
     /**
      * Create a deep copy of this Facet.
      * @return a deep copy.

http://dive4elements.wald.intevation.org