diff options
| author | Samuel Tardieu <sam@rfc1149.net> | 2012-04-28 12:50:14 +0200 |
|---|---|---|
| committer | Samuel Tardieu <sam@rfc1149.net> | 2012-04-28 13:30:09 +0200 |
| commit | 9586b506fd44b035dd057e9fd4ff65f49ee3c7b7 (patch) | |
| tree | adaee8f47f1c4bb8d010c5d4b322fa11628c3ad4 /main/src/cgeo/geocaching/sorting/GeocodeComparator.java | |
| parent | c8b7b76cc248ceb35cf51b7d14cfedc836215a06 (diff) | |
| download | cgeo-9586b506fd44b035dd057e9fd4ff65f49ee3c7b7.zip cgeo-9586b506fd44b035dd057e9fd4ff65f49ee3c7b7.tar.gz cgeo-9586b506fd44b035dd057e9fd4ff65f49ee3c7b7.tar.bz2 | |
Refactoring: use simpler comparaisons
Diffstat (limited to 'main/src/cgeo/geocaching/sorting/GeocodeComparator.java')
| -rw-r--r-- | main/src/cgeo/geocaching/sorting/GeocodeComparator.java | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/main/src/cgeo/geocaching/sorting/GeocodeComparator.java b/main/src/cgeo/geocaching/sorting/GeocodeComparator.java index 1377cce..fb93c0e 100644 --- a/main/src/cgeo/geocaching/sorting/GeocodeComparator.java +++ b/main/src/cgeo/geocaching/sorting/GeocodeComparator.java @@ -17,12 +17,8 @@ public class GeocodeComparator extends AbstractCacheComparator { } @Override - protected int compareCaches(cgCache cache1, cgCache cache2) { - if (cache1.getGeocode().length() > cache2.getGeocode().length()) { - return 1; - } else if (cache2.getGeocode().length() > cache1.getGeocode().length()) { - return -1; - } - return cache1.getGeocode().compareToIgnoreCase(cache2.getGeocode()); + protected int compareCaches(final cgCache cache1, final cgCache cache2) { + final int lengthDiff = cache1.getGeocode().length() - cache2.getGeocode().length(); + return lengthDiff != 0 ? lengthDiff : cache1.getGeocode().compareToIgnoreCase(cache2.getGeocode()); } } |
