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

Timestamp:

01/24/14 13:50:29 (10 years ago)

Author:

pharms

Message:

• corrected and extended usability evaluations
• checking now more text field specific for required input formats and word repetitions
• checking also for required scrolls

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

@@ -46 +46 @@
     private void init() {
         rules.add(new TextInputStatisticsRule());
+        rules.add(new RequiredScrollRule());
     }
 
@@ -54 +55 @@
         Console.traceln(Level.INFO, "evaluating usability of task model " + taskModel);
 
-        List<UsabilityEvaluationResult> results = new ArrayList<UsabilityEvaluationResult>();
+        List<UsabilityEvaluationResult> interimResults = new ArrayList<UsabilityEvaluationResult>();
 
         for (UsabilityEvaluationRule rule : rules) {
             Console.traceln(Level.INFO, "applying rule " + rule.getClass().getSimpleName());
             UsabilityEvaluationResult result = rule.evaluate(taskModel);
-            results.add(result);
+            interimResults.add(result);
             Console.traceln(Level.INFO, "the rule found " + result.getAllDefects().size() +
                             " usability defects, of which " + result.getSevereDefects().size() +
@@ -65 +66 @@
         }
 
-        UsabilityEvaluationResult result = mergeResults(results);
+        UsabilityEvaluationResult result = new UsabilityEvaluationResult(interimResults);
         Console.println("the evaluation result contains " + result.getAllDefects().size() +
                         " defects, of which " + result.getSevereDefects().size() + " are severe.");
-        return result;
-    }
 
-    /**
-     *
-     */
-    private UsabilityEvaluationResult mergeResults(List<UsabilityEvaluationResult> results) {
-        UsabilityEvaluationResult result = new UsabilityEvaluationResult();
-
-        for (UsabilityEvaluationResult ruleResult : results) {
-            for (UsabilityDefect defect : ruleResult.getAllDefects()) {
-                result.addDefect(defect);
-            }
+        List<UsabilityDefect> defects = result.getAllDefects();
+        for (int i = 0; i < defects.size(); i++) {
+            Console.println((i + 1) + ": " + defects.get(i).getParameterizedDescription());
         }