Ignore:
Timestamp:
2017-09-15T01:20:41+02:00 (2 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/data/validation
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/validation/tests/CrossingWays.java

    r12620 r12865  
    315315     */
    316316    public static List<List<WaySegment>> getSegments(Map<Point2D, List<WaySegment>> cellSegments, EastNorth n1, EastNorth n2) {
    317 
    318317        List<List<WaySegment>> cells = new ArrayList<>();
    319318        for (Point2D cell : ValUtil.getSegmentCells(n1, n2, OsmValidator.getGridDetail())) {
    320             List<WaySegment> segments = cellSegments.get(cell);
    321             if (segments == null) {
    322                 segments = new ArrayList<>();
    323                 cellSegments.put(cell, segments);
    324             }
    325             cells.add(segments);
     319            cells.add(cellSegments.computeIfAbsent(cell, k -> new ArrayList<>()));
    326320        }
    327321        return cells;
  • trunk/src/org/openstreetmap/josm/data/validation/tests/RelationChecker.java

    r12006 r12865  
    129129        Map<String, RoleInfo> map = new HashMap<>();
    130130        for (RelationMember m : n.getMembers()) {
    131             String role = m.getRole();
    132             RoleInfo ri = map.get(role);
    133             if (ri == null) {
    134                 ri = new RoleInfo();
    135                 map.put(role, ri);
    136             }
    137             ri.total++;
     131            map.computeIfAbsent(m.getRole(), k -> new RoleInfo()).total++;
    138132        }
    139133        return map;
  • 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.