svn commit: r1552933 - /mahout/trunk/core/src/main/java/org/apache/mahout/classifier/df/data/DataConverter.java

[smarthi]

Author: smarthi
Date: Sat Dec 21 18:26:55 2013
New Revision: 1552933

URL: http://svn.apache.org/r1552933
Log:
Code cleanup, removed continue statements in loops

Modified:
    
mahout/trunk/core/src/main/java/org/apache/mahout/classifier/df/data/DataConverter.java

Modified: 
mahout/trunk/core/src/main/java/org/apache/mahout/classifier/df/data/DataConverter.java
URL: 
http://svn.apache.org/viewvc/mahout/trunk/core/src/main/java/org/apache/mahout/classifier/df/data/DataConverter.java?rev=1552933&r1=1552932&r2=1552933&view=diff
==============================================================================
--- 
mahout/trunk/core/src/main/java/org/apache/mahout/classifier/df/data/DataConverter.java
 (original)
+++ 
mahout/trunk/core/src/main/java/org/apache/mahout/classifier/df/data/DataConverter.java
 Sat Dec 21 18:26:55 2013
@@ -49,22 +49,20 @@ public class DataConverter {
 
     int aId = 0;
     for (int attr = 0; attr < nball; attr++) {
-      if (ArrayUtils.contains(dataset.getIgnored(), attr)) {
-        continue; // IGNORED
-      }
-
-      String token = tokens[attr].trim();
+      if (!ArrayUtils.contains(dataset.getIgnored(), attr)) {
+        String token = tokens[attr].trim();
 
-      if ("?".equals(token)) {
-        // missing value
-        return null;
-      }
+        if ("?".equals(token)) {
+          // missing value
+          return null;
+        }
 
-      if (dataset.isNumerical(aId)) {
-        vector.set(aId++, Double.parseDouble(token));
-      } else { // CATEGORICAL
-        vector.set(aId, dataset.valueOf(aId, token));
-        aId++;
+        if (dataset.isNumerical(aId)) {
+          vector.set(aId++, Double.parseDouble(token));
+        } else { // CATEGORICAL
+          vector.set(aId, dataset.valueOf(aId, token));
+          aId++;
+        }
       }
     }