Changeset 9706 in josm


Ignore:
Timestamp:
2016-02-01T17:10:12+01:00 (6 years ago)
Author:
simon04
Message:

Refactoring (Java 7 compare functions)

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/DefaultNameFormatter.java

    r9473 r9706  
    332332    }
    333333
    334     private void formatRelationNameAndType(Relation relation, StringBuilder result, TaggingPreset preset) {
     334    private StringBuilder formatRelationNameAndType(Relation relation, StringBuilder result, TaggingPreset preset) {
    335335        if (preset == null) {
    336336            result.append(getRelationTypeName(relation));
     
    346346            result.append('(');
    347347        }
     348        return result;
    348349    }
    349350
     
    357358
    358359            if (preset1 != null || preset2 != null) {
    359                 StringBuilder name1 = new StringBuilder();
    360                 formatRelationNameAndType(r1, name1, preset1);
    361                 StringBuilder name2 = new StringBuilder();
    362                 formatRelationNameAndType(r2, name2, preset2);
    363 
    364                 int comp = AlphanumComparator.getInstance().compare(name1.toString(), name2.toString());
     360                String name1 = formatRelationNameAndType(r1, new StringBuilder(), preset1).toString();
     361                String name2 = formatRelationNameAndType(r2, new StringBuilder(), preset2).toString();
     362
     363                int comp = AlphanumComparator.getInstance().compare(name1, name2);
    365364                if (comp != 0)
    366365                    return comp;
     
    382381            }
    383382
    384             if (r1.getMembersCount() != r2.getMembersCount())
    385                 return (r1.getMembersCount() > r2.getMembersCount()) ? 1 : -1;
    386 
    387             int comp = Boolean.valueOf(r1.hasIncompleteMembers()).compareTo(Boolean.valueOf(r2.hasIncompleteMembers()));
     383            int comp = Integer.compare(r1.getMembersCount(), r2.getMembersCount());
    388384            if (comp != 0)
    389385                return comp;
    390386
    391             if (r1.getUniqueId() > r2.getUniqueId())
    392                 return 1;
    393             else if (r1.getUniqueId() < r2.getUniqueId())
    394                 return -1;
    395             else
    396                 return 0;
     387
     388            comp = Boolean.compare(r1.hasIncompleteMembers(), r2.hasIncompleteMembers());
     389            if (comp != 0)
     390                return comp;
     391
     392            return Long.compare(r1.getUniqueId(), r2.getUniqueId());
    397393        }
    398394    };
Note: See TracChangeset for help on using the changeset viewer.