Ignore:
Timestamp:
2016-11-28T21:18:55+01:00 (3 years ago)
Author:
simon04
Message:

Loop can be replaced with Collection.removeIf()

Location:
trunk/src/org/openstreetmap/josm/actions
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/actions/AbstractInfoAction.java

    r10383 r11339  
    88import java.util.ArrayList;
    99import java.util.Collection;
    10 import java.util.Iterator;
    1110import java.util.List;
    1211
     
    1514import org.openstreetmap.josm.Main;
    1615import org.openstreetmap.josm.data.notes.Note;
     16import org.openstreetmap.josm.data.osm.AbstractPrimitive;
    1717import org.openstreetmap.josm.data.osm.DataSet;
    1818import org.openstreetmap.josm.data.osm.OsmPrimitive;
     
    109109        // filter out new primitives which are not yet uploaded to the server
    110110        //
    111         Iterator<OsmPrimitive> it = primitivesToShow.iterator();
    112         while (it.hasNext()) {
    113             if (it.next().isNew()) {
    114                 it.remove();
    115             }
    116         }
     111        primitivesToShow.removeIf(AbstractPrimitive::isNew);
    117112
    118113        if (primitivesToShow.isEmpty() && noteToShow == null) {
  • trunk/src/org/openstreetmap/josm/actions/UnJoinNodeWayAction.java

    r10467 r11339  
    1010import java.util.ArrayList;
    1111import java.util.Collection;
    12 import java.util.Iterator;
    1312import java.util.LinkedList;
    1413import java.util.List;
     
    156155        for (int i = 1; i < selectedNodes.size(); i++) {
    157156            List<OsmPrimitive> ref = selectedNodes.get(i).getReferrers();
    158             for (Iterator<Way> it = result.iterator(); it.hasNext();) {
    159                 if (!ref.contains(it.next())) {
    160                     it.remove();
    161                 }
    162             }
     157            result.removeIf(way -> !ref.contains(way));
    163158        }
    164159
    165160        // Remove broken ways
    166         for (Iterator<Way> it = result.iterator(); it.hasNext();) {
    167             if (it.next().getNodesCount() <= 2) {
    168                 it.remove();
    169             }
    170         }
     161        result.removeIf(way -> way.getNodesCount() <= 2);
    171162
    172163        if (selectedWays.isEmpty())
     
    174165        else {
    175166            // Return only selected ways
    176             for (Iterator<Way> it = result.iterator(); it.hasNext();) {
    177                 if (!selectedWays.contains(it.next())) {
    178                     it.remove();
    179                 }
    180             }
     167            result.removeIf(way -> !selectedWays.contains(way));
    181168            return result;
    182169        }
Note: See TracChangeset for help on using the changeset viewer.