Ignore:
Timestamp:
2017-09-15T01:20:41+02:00 (3 years ago)
Author:
Don-vip
Message:

see #11390 - SonarQube - squid:S3824 - "Map.get" and value test should be replaced with single method call

Location:
trunk/src/org/openstreetmap/josm/gui
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/sort/RelationNodeMap.java

    r10043 r12865  
    108108
    109109    private void addPair(Node n, int i) {
    110         Set<Integer> ts = map.nodes.get(n);
    111         if (ts == null) {
    112             ts = new TreeSet<>();
    113             map.nodes.put(n, ts);
    114         }
    115         ts.add(i);
    116 
    117         Set<Node> ts2 = map.ways.get(i);
    118         if (ts2 == null) {
    119             ts2 = new TreeSet<>();
    120             map.ways.put(i, ts2);
    121         }
    122         ts2.add(n);
     110        map.nodes.computeIfAbsent(n, k -> new TreeSet<>()).add(i);
     111        map.ways.computeIfAbsent(i, k -> new TreeSet<>()).add(n);
    123112    }
    124113
    125114    private void addNodeWayMap(Node n, int i) {
    126         Set<Integer> ts = onewayMap.nodes.get(n);
    127         if (ts == null) {
    128             ts = new TreeSet<>();
    129             onewayMap.nodes.put(n, ts);
    130         }
    131         ts.add(i);
     115        onewayMap.nodes.computeIfAbsent(n, k -> new TreeSet<>()).add(i);
    132116    }
    133117
    134118    private void addWayNodeMap(Node n, int i) {
    135         Set<Node> ts2 = onewayMap.ways.get(i);
    136         if (ts2 == null) {
    137             ts2 = new TreeSet<>();
    138             onewayMap.ways.put(i, ts2);
    139         }
    140         ts2.add(n);
     119        onewayMap.ways.computeIfAbsent(i, k -> new TreeSet<>()).add(n);
    141120    }
    142121
    143122    private void addNodeWayMapReverse(Node n, int i) {
    144         Set<Integer> ts = onewayReverseMap.nodes.get(n);
    145         if (ts == null) {
    146             ts = new TreeSet<>();
    147             onewayReverseMap.nodes.put(n, ts);
    148         }
    149         ts.add(i);
     123        onewayReverseMap.nodes.computeIfAbsent(n, k -> new TreeSet<>()).add(i);
    150124    }
    151125
    152126    private void addWayNodeMapReverse(Node n, int i) {
    153         Set<Node> ts2 = onewayReverseMap.ways.get(i);
    154         if (ts2 == null) {
    155             ts2 = new TreeSet<>();
    156             onewayReverseMap.ways.put(i, ts2);
    157         }
    158         ts2.add(n);
     127        onewayReverseMap.ways.computeIfAbsent(i, k -> new TreeSet<>()).add(n);
    159128    }
    160129
    161130    private void addRemainingForward(Node n, int i) {
    162         Set<Node> ts2 = remainingOneway.get(i);
    163         if (ts2 == null) {
    164             ts2 = new TreeSet<>();
    165             remainingOneway.put(i, ts2);
    166         }
    167         ts2.add(n);
     131        remainingOneway.computeIfAbsent(i, k -> new TreeSet<>()).add(n);
    168132    }
    169133
  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/sort/RelationSorter.java

    r12663 r12865  
    151151            for (AdditionalSorter sorter : ADDITIONAL_SORTERS) {
    152152                if (sorter.acceptsMember(m)) {
    153                     List<RelationMember> list;
    154                     list = customMap.get(sorter);
    155                     if (list == null) {
    156                         list = new LinkedList<>();
    157                         customMap.put(sorter, list);
    158                     }
    159                     list.add(m);
    160                     wasAdded = true;
     153                    wasAdded = customMap.computeIfAbsent(sorter, k -> new LinkedList<>()).add(m);
    161154                    break;
    162155                }
  • trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletionManager.java

    r12859 r12865  
    506506    }
    507507
    508     private static void registerListeners(AutoCompletionManager autoCompletionManager) {
    509         autoCompletionManager.ds.addDataSetListener(autoCompletionManager);
     508    private AutoCompletionManager registerListeners() {
     509        ds.addDataSetListener(this);
    510510        MainApplication.getLayerManager().addLayerChangeListener(new LayerChangeListener() {
    511511            @Override
    512512            public void layerRemoving(LayerRemoveEvent e) {
    513513                if (e.getRemovedLayer() instanceof OsmDataLayer
    514                         && ((OsmDataLayer) e.getRemovedLayer()).data == autoCompletionManager.ds) {
    515                     INSTANCES.remove(autoCompletionManager.ds);
    516                     autoCompletionManager.ds.removeDataSetListener(autoCompletionManager);
     514                        && ((OsmDataLayer) e.getRemovedLayer()).data == ds) {
     515                    INSTANCES.remove(ds);
     516                    ds.removeDataSetListener(AutoCompletionManager.this);
    517517                    MainApplication.getLayerManager().removeLayerChangeListener(this);
    518518                }
     
    529529            }
    530530        });
     531        return this;
    531532    }
    532533
     
    538539     */
    539540    public static AutoCompletionManager of(DataSet dataSet) {
    540         AutoCompletionManager result = INSTANCES.get(dataSet);
    541         if (result == null) {
    542             result = new AutoCompletionManager(dataSet);
    543             INSTANCES.put(dataSet, result);
    544             registerListeners(result);
    545         }
    546         return result;
     541        return INSTANCES.computeIfAbsent(dataSet, ds -> new AutoCompletionManager(ds).registerListeners());
    547542    }
    548543}
Note: See TracChangeset for help on using the changeset viewer.