You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mahout.apache.org by gs...@apache.org on 2010/01/03 18:13:06 UTC

svn commit: r895448 - /lucene/mahout/trunk/core/src/main/java/org/apache/mahout/common/distance/WeightedEuclideanDistanceMeasure.java

Author: gsingers
Date: Sun Jan  3 17:13:04 2010
New Revision: 895448

URL: http://svn.apache.org/viewvc?rev=895448&view=rev
Log:
slight refactoring

Modified:
    lucene/mahout/trunk/core/src/main/java/org/apache/mahout/common/distance/WeightedEuclideanDistanceMeasure.java

Modified: lucene/mahout/trunk/core/src/main/java/org/apache/mahout/common/distance/WeightedEuclideanDistanceMeasure.java
URL: http://svn.apache.org/viewvc/lucene/mahout/trunk/core/src/main/java/org/apache/mahout/common/distance/WeightedEuclideanDistanceMeasure.java?rev=895448&r1=895447&r2=895448&view=diff
==============================================================================
--- lucene/mahout/trunk/core/src/main/java/org/apache/mahout/common/distance/WeightedEuclideanDistanceMeasure.java (original)
+++ lucene/mahout/trunk/core/src/main/java/org/apache/mahout/common/distance/WeightedEuclideanDistanceMeasure.java Sun Jan  3 17:13:04 2010
@@ -31,7 +31,8 @@
   public double distance(Vector p1, Vector p2) {
     double result = 0;
     Vector res = p2.minus(p1);
-    if (getWeights() == null) {
+    Vector theWeights = getWeights();
+    if (theWeights == null) {
       Iterator<Vector.Element> iter = res.iterateNonZero();
       while (iter.hasNext()) {
         Vector.Element elt = iter.next();
@@ -41,7 +42,7 @@
       Iterator<Vector.Element> iter = res.iterateNonZero();
       while (iter.hasNext()) {
         Vector.Element elt = iter.next();
-        result += elt.get() * elt.get() * getWeights().get(elt.index());
+        result += elt.get() * elt.get() * theWeights.get(elt.index());
       }
     }
     return Math.sqrt(result);