svn commit: r1617859 - /incubator/devicemap/trunk/devicemap/java/classifier/src/main/java/org/apache/devicemap/data/Pattern.java

[re...@apache.org]

Author: rezan
Date: Thu Aug 14 01:04:41 2014
New Revision: 1617859

URL: http://svn.apache.org/r1617859
Log:
rank is final

Modified:
    incubator/devicemap/trunk/devicemap/java/classifier/src/main/java/org/apache/devicemap/data/Pattern.java

Modified: incubator/devicemap/trunk/devicemap/java/classifier/src/main/java/org/apache/devicemap/data/Pattern.java
URL: http://svn.apache.org/viewvc/incubator/devicemap/trunk/devicemap/java/classifier/src/main/java/org/apache/devicemap/data/Pattern.java?rev=1617859&r1=1617858&r2=1617859&view=diff
==============================================================================
--- incubator/devicemap/trunk/devicemap/java/classifier/src/main/java/org/apache/devicemap/data/Pattern.java (original)
+++ incubator/devicemap/trunk/devicemap/java/classifier/src/main/java/org/apache/devicemap/data/Pattern.java Thu Aug 14 01:04:41 2014
@@ -29,7 +29,7 @@ public class Pattern {
 
     private final String type;
     
-    private int rank;
+    private final int rank;
     
     private final int boost;
 
@@ -41,7 +41,7 @@ public class Pattern {
         this.pattern = pattern;
         this.type = type;
         this.boost = boost;
-        setRank();
+        rank = genRank();
     }
 
     @Override
@@ -58,18 +58,20 @@ public class Pattern {
         return sb.toString();
     }
 
-    private void setRank() {
-        rank = 0;
+    private int genRank() {
+        int r = 0;
         
         if ("weak".equals(type)) {
-            rank += 1000;
+            r += 1000;
         }
 
-        rank += pattern.size() * 100;
+        r += pattern.size() * 100;
 
         for (String part : pattern) {
-            rank += part.length();
+            r += part.length();
         }
+        
+        return r;
     }
     
     public int getRank() {