Changeset 496 for trunk/quest-core-usability/src/main/java/de/ugoe/cs/quest/usability/TextInputStatisticsRule.java

Timestamp:

07/27/12 14:36:59 (12 years ago)

Author:

pharms

Message:

• added further text entry based evaluation rule

File:

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

trunk/quest-core-usability/src/main/java/de/ugoe/cs/quest/usability/TextInputStatisticsRule.java

@@ -60 +60 @@
     checkTextInputRatio(statistics, results);
     checkTextFieldEntryRepetitions(statistics, results);
+    checkTextFieldNoLetterOrDigitInputs(statistics, results);
   }
 
@@ -88 +89 @@
     }
     else if (ratio > 0.5)
+    {
+      severity = UsabilityDefectSeverity.LOW;
+    }
+    else if (ratio > 0.3)
     {
       severity = UsabilityDefectSeverity.INFO;
@@ -180 +185 @@
    * TODO: comment
    *
+   * @param statistics
+   * @param results
+   */
+  //-----------------------------------------------------------------------------------------------
+  private void checkTextFieldNoLetterOrDigitInputs(TextInputStatistics       statistics,
+                                                   UsabilityEvaluationResult results)
+  {
+    int allCharactersCount = 0;
+    int noLetterOrDigitCount = 0;
+
+    for (int i = 0; i < statistics.getNoOfTextFieldInputs(); i++)
+    {
+      String[] fragments = statistics.getTextFieldInputFragments(i);
+      for (String fragment : fragments)
+      {
+        String effectiveFragment = fragment.trim();
+        for (int j = 0; j < effectiveFragment.length(); j++)
+        {
+          if (!Character.isWhitespace(effectiveFragment.charAt(j)))
+          {
+            if (!Character.isLetterOrDigit(effectiveFragment.charAt(j)))
+            {
+              noLetterOrDigitCount++;
+            }
+            allCharactersCount++;
+          }
+        }
+      }
+    }
+    
+    float ratio = (float) noLetterOrDigitCount / (float) allCharactersCount;
+    
+    UsabilityDefectSeverity severity = null;
+    if (ratio > 0.1) // every 10th sign
+    {
+      severity = UsabilityDefectSeverity.HIGH;
+    }
+    else if (ratio > 0.05) // every 20th sign
+    {
+      severity = UsabilityDefectSeverity.MEDIUM;
+    }
+    else if (ratio > 0.02) // every 50th sign
+    {
+      severity = UsabilityDefectSeverity.LOW;
+    }
+    else if (ratio > 0.01) // every 100th sign
+    {
+      severity = UsabilityDefectSeverity.INFO;
+    }
+    
+    if (severity != null)
+    {
+      Map<String, String> parameters = new HashMap<String, String>();
+      parameters.put("noLetterOrDigitRatio", allCharactersCount + " entered characters of " +
+                     "which " + noLetterOrDigitCount + " were no letter or digit");
+
+      results.addDefect
+        (new UsabilityDefect
+           (severity, UsabilityDefectDescription.TEXT_FIELD_NO_LETTER_OR_DIGIT_RATIO, parameters));
+    }
+  }
+
+  //-----------------------------------------------------------------------------------------------
+  /**
+   * TODO: comment
+   *
    * @param taskTree
    * @param statistics