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

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

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/data/validation/util/ValUtil.java

    r11852 r12865  
    5656        cell = new Point2D.Double(x0, y0);
    5757        cellNodes.add(cell);
    58         List<Way> ways = cellWays.get(cell);
    59         if (ways == null) {
    60             ways = new ArrayList<>();
    61             cellWays.put(cell, ways);
    62         }
    63         cells.add(ways);
     58        cells.add(cellWays.computeIfAbsent(cell, k -> new ArrayList<>()));
    6459
    6560        // End of the way
    … …  
    6762        if (!cellNodes.contains(cell)) {
    6863            cellNodes.add(cell);
    69             ways = cellWays.get(cell);
    70             if (ways == null) {
    71                 ways = new ArrayList<>();
    72                 cellWays.put(cell, ways);
    73             }
    74             cells.add(ways);
     64            cells.add(cellWays.computeIfAbsent(cell, k -> new ArrayList<>()));
    7565        }
    7666
    … …  
    8777        if (!cellNodes.contains(cell)) {
    8878            cellNodes.add(cell);
    89             ways = cellWays.get(cell);
    90             if (ways == null) {
    91                 ways = new ArrayList<>();
    92                 cellWays.put(cell, ways);
    93             }
    94             cells.add(ways);
     79            cells.add(cellWays.computeIfAbsent(cell, k -> new ArrayList<>()));
    9580        }
    9681
    … …  
    9984        if (!cellNodes.contains(cell)) {
    10085            cellNodes.add(cell);
    101             ways = cellWays.get(cell);
    102             if (ways == null) {
    103                 ways = new ArrayList<>();
    104                 cellWays.put(cell, ways);
    105             }
    106             cells.add(ways);
     86            cells.add(cellWays.computeIfAbsent(cell, k -> new ArrayList<>()));
    10787        }
    10888        return cells;
Note: See TracChangeset for help on using the changeset viewer.