I've got a collection of Duck objects and I'd like to sort them using multiple keys.
class Duck {
DuckAge age; //implements Comparable
DuckWeight weight; //implements Comparable
String name;
}
List<Duck> ducks = Pond.getDucks();
eg. I want to sort them primarily by their weights, and secondarily by their age. If two ducks have the exact same weight and the exact same age, then let's differentiate them using their names as a tertiary key. I might do something like this:
Collections.sort(ducks, new Comparator<Duck>(){
@Override
public int compare(Duck d1, Duck d2){
int weightCmp = d1.weight.compareTo(d2.weight);
if (weightCmp != 0) {
return weightCmp;
}
int ageCmp = d1.age.compareTo(d2.age);
if (ageCmp != 0) {
return ageCmp;
}
return d1.name.compareTo(d2.name);
}
});
Well I do this quite frequently, but this solution doesn't smell right. It doesn't scale well, and it's easy to mess up. Surely there must be a better way of sorting Ducks using multiple keys! Does anybody know of a better solution?
EDIT removed unnecessary else
branches
else
since in theif
you return, so it's not needed.Collections.sort()
modifies the given collection itself (instead of returning a sorted copy of the collection). So theducks
object "gets overwritten" in the sense that it is sorted in-place. (Well not actually in-place: the contents ofducks
is dumped into an array, where it gets sorted, and the result is copied back into the original collection.)