Compare Tuples based on the element, not the element & score
Set uses equals to compare if the set is already in the set. Redis Sorted Sets are unique on the value, not the value with the score. The Tuple.equals should return true when the elements are equal regardless of what the score is. The score should be used when comparing to another element in the set to see if it is < or >.
This commit is contained in:
@@ -4,7 +4,7 @@ import java.util.Arrays;
|
||||
|
||||
import redis.clients.util.SafeEncoder;
|
||||
|
||||
public class Tuple {
|
||||
public class Tuple implements Comparable {
|
||||
private byte[] element;
|
||||
private Double score;
|
||||
|
||||
@@ -38,12 +38,22 @@ public class Tuple {
|
||||
return false;
|
||||
} else if (!Arrays.equals(element, other.element))
|
||||
return false;
|
||||
if (Double.doubleToLongBits(score) != Double
|
||||
.doubleToLongBits(other.score))
|
||||
return false;
|
||||
return true;
|
||||
}
|
||||
|
||||
public int compareTo(Tuple other) {
|
||||
if (Arrays.equals(this.element, other.element))
|
||||
return 0;
|
||||
else
|
||||
return this.score < other.getScore() ? -1 : 1;
|
||||
}
|
||||
public int compareTo(Object obj) {
|
||||
if (getClass() != obj.getClass())
|
||||
throw new ClassCastException();
|
||||
return compareTo((Tuple) obj);
|
||||
}
|
||||
|
||||
|
||||
public Tuple(String element, Double score) {
|
||||
super();
|
||||
this.element = SafeEncoder.encode(element);
|
||||
|
||||
Reference in New Issue
Block a user