Changeset 1493 for trunk/autoquest-core-usability/src/main/java/de/ugoe/cs/autoquest/usability/UsabilityEvaluationManager.java

Timestamp:

04/11/14 11:21:45 (10 years ago)

Author:

pharms

Message:

• state of the HCSE 2014 Paper. An appropriate tag will follow.

File:

• trunk/autoquest-core-usability/src/main/java/de/ugoe/cs/autoquest/usability/UsabilityEvaluationManager.java (modified) (3 diffs)

trunk/autoquest-core-usability/src/main/java/de/ugoe/cs/autoquest/usability/UsabilityEvaluationManager.java

@@ -19 +19 @@
 import java.util.logging.Level;
 
+import de.ugoe.cs.autoquest.tasktrees.treeifc.IMarkingTemporalRelationship;
+import de.ugoe.cs.autoquest.tasktrees.treeifc.IStructuringTemporalRelationship;
+import de.ugoe.cs.autoquest.tasktrees.treeifc.ITask;
 import de.ugoe.cs.autoquest.tasktrees.treeifc.ITaskModel;
 import de.ugoe.cs.util.console.Console;
@@ -45 +48 @@
      */
     private void init() {
-        rules.add(new TextInputStatisticsRule());
-        rules.add(new RequiredScrollRule());
+        //rules.add(new TextInputStatisticsRule());
+        rules.add(new MissingFeedbackRule());
+        rules.add(new EventCoverageRatioRule());
+        rules.add(new RequiredInefficientActionsRule());
+        //rules.add(new TaskCooccurrenceRule());
+        rules.add(new TargetDistanceRule());
+        //rules.add(new UnusedGUIElementsRule());
+        //rules.add(new TaskTreeTestRule());
     }
 
@@ -64 +73 @@
                             " usability defects, of which " + result.getSevereDefects().size() +
                             " are severe.");
+            
+            if ((rule instanceof EventCoverageRatioRule) ||
+                (rule instanceof RequiredInefficientActionsRule) ||
+                (rule instanceof TargetDistanceRule))
+            {
+                ITask[] referredTasks = new ITask[result.getAllDefects().size()];
+
+                for (int i = 0; i < result.getAllDefects().size(); i++) {
+                    referredTasks[i] =
+                        (ITask) result.getAllDefects().get(i).getDescriptionFragments().get(1);
+                }
+                
+                int counter = 0;
+                for (int i = 0; i < referredTasks.length; i++) {
+                    for (int j = 0; j < referredTasks.length; j++) {
+                        if (isChildOf(referredTasks[i], referredTasks[j])) {
+                            counter++;
+                            break;
+                        }
+                    }
+                }
+                
+                if (counter > 0) {
+                    Console.traceln(Level.INFO, counter + " of the findings are duplicates in " +
+                                    "that they refer to tasks whose parent tasks are also " +
+                                    "referred by the findings");
+                }
+            }
         }
 
-        UsabilityEvaluationResult result = new UsabilityEvaluationResult(interimResults);
+        UsabilityEvaluationResult result = new UsabilityEvaluationResult(taskModel, interimResults);
         Console.println("the evaluation result contains " + result.getAllDefects().size() +
                         " defects, of which " + result.getSevereDefects().size() + " are severe.");
-
-        List<UsabilityDefect> defects = result.getAllDefects();
-        for (int i = 0; i < defects.size(); i++) {
-            Console.println((i + 1) + ": " + defects.get(i).getParameterizedDescription());
-        }
 
         return result;
     }
 
+    /**
+     * <p>
+     * TODO: comment
+     * </p>
+     *
+     * @param iTask
+     * @param iTask2
+     * @return
+     */
+    private boolean isChildOf(final ITask potChild, ITask potParent) {
+        
+        if (potParent instanceof IStructuringTemporalRelationship) {
+            for (ITask child : ((IStructuringTemporalRelationship) potParent).getChildren()) {
+                if ((child == potChild) || isChildOf(potChild, child)) {
+                    return true;
+                }
+            }
+        }
+        else if (potParent instanceof IMarkingTemporalRelationship) {
+            ITask child = ((IMarkingTemporalRelationship) potParent).getMarkedTask();
+            if ((child == potChild) || isChildOf(potChild, child)) {
+                return true;
+            }
+        }
+        
+        return false;
+    }
+
 }