Changeset 1891 for trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/TaskInfo.java

Timestamp:

03/05/15 11:43:42 (9 years ago)

Author:

pharms

Message:

• remove support for tasks contexts in value measurements

File:

• trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/TaskInfo.java (modified) (8 diffs)

trunk/autoquest-core-tasktrees/src/main/java/de/ugoe/cs/autoquest/tasktrees/treeimpl/TaskInfo.java

@@ -16 +16 @@
 
 import java.util.ArrayList;
-import java.util.HashMap;
 
 import de.ugoe.cs.autoquest.tasktrees.treeifc.ITask;
@@ -89 +88 @@
 
     /* (non-Javadoc)
-     * @see de.ugoe.cs.autoquest.tasktrees.treeifc.ITaskInfo#getMeasureValue(java.lang.String, de.ugoe.cs.autoquest.tasktrees.treeifc.ITask)
-     */
-    @Override
-    public int getMeasureValue(TaskMetric metric, ITask context) {
-        Measure measure = getMeasure(metric);
-        
-        if (measure == null) {
-            throw new IllegalArgumentException("unknown metric " + metric);
-        }
-        
-        return measure.getValue(context);
-    }
-
-    /* (non-Javadoc)
      * @see java.lang.Object#toString()
      */
@@ -112 +97 @@
     /**
      * <p>
-     * must be called to indicate that a new new measures for the provided metric are about to
+     * must be called to indicate that new measures for the provided metric are about to
      * be calculated and added.
      * </p>
@@ -131 +116 @@
     /**
      * <p>
-     * sets a specific value for a measure of a specific metric in the provided context of the task
+     * sets a specific value for a measure of a specific metric
      * </p>
      * 
      * @param metric  the metric to which the value belongs
-     * @param context the context of the task in which the measure was recorded
      * @param value   the value of the measure
      */
-    void setCount(TaskMetric metric, ITask context, int value) {
+    void setCount(TaskMetric metric, int value) {
         Measure measure = getMeasure(metric);
         
@@ -146 +130 @@
         }
         
-        measure.set(context, value);
-    }
-
-    /**
-     * <p>
-     * increases a specific value for a measure of a specific metric in the provided context of the
-     * task
+        measure.set(value);
+    }
+
+    /**
+     * <p>
+     * increases a specific value for a measure of a specific metric
      * </p>
      * 
      * @param metric    the metric to which the value belongs
-     * @param context   the context of the task in which the measure was recorded
      * @param increment the increment to be added to the value of the measure
      */
-    void increaseCount(TaskMetric metric, ITask context, int increment) {
+    void increaseCount(TaskMetric metric, int increment) {
         Measure measure = getMeasure(metric);
         
@@ -167 +149 @@
         }
         
-        measure.increase(context, increment);
+        measure.increase(increment);
     }
 
@@ -204 +186 @@
         /**
          * <p>
-         * the observed values for the difference contexts of the task
-         * </p>
-         */
-        private HashMap<ITask, Integer> values;
-        
-        /**
-         * <p>
-         * the context free value of the measure independent of the task context
-         * </p>
-         */
-        private int contextFreeValue = 0;
+         * the value of the measure independent
+         * </p>
+         */
+        private int value = 0;
         
         /**
@@ -239 +214 @@
         @Override
         public int getValue() {
-            return contextFreeValue;
-        }
-
-        /* (non-Javadoc)
-         * @see de.ugoe.cs.autoquest.tasktrees.treeifc.ITaskInfo.IMeasure#getValue(de.ugoe.cs.autoquest.tasktrees.treeifc.ITask)
-         */
-        @Override
-        public int getValue(ITask context) {
-            if ((context != null) && (values != null)) {
-                Integer currentValue = values.get(context);
-                
-                if (currentValue != null) {
-                    return currentValue;
-                }
-            }
-            
-            return Integer.MIN_VALUE;
-        }
-
-        /**
-         * <p>
-         * sets the value of the measure context free as well as specific to the provided context
-         * </p>
-         */
-        private void set(ITask context, int value) {
-            contextFreeValue = value;
-            
-            if (context != null) {
-                if (values == null) {
-                    values = new HashMap<ITask, Integer>();
-                }
-                
-                values.put(context, value);
-            }
-        }
-
-        /**
-         * <p>
-         * increases the value of the measure context free as well as specific to the provided
-         * context according to the provided increment
-         * </p>
-         */
-        private void increase(ITask context, int increment) {
-            contextFreeValue += increment;
-            
-            if (context != null) {
-                if (values == null) {
-                    values = new HashMap<ITask, Integer>();
-                }
-                
-                Integer currentValue = values.get(context);
-                
-                if (currentValue == null) {
-                    currentValue = 0;
-                }
-                
-                values.put(context, currentValue + increment);
-            }
+            return value;
+        }
+
+        /**
+         * <p>
+         * sets the value of the measure
+         * </p>
+         */
+        private void set(int newValue) {
+            value = newValue;
+        }
+
+        /**
+         * <p>
+         * increases the value of the measure
+         * </p>
+         */
+        private void increase(int increment) {
+            value += increment;
         }