Changeset 16387 in josm for trunk

Timestamp:

2020-04-30T20:53:59+02:00 (5 years ago)

Author:

simon04

Message:

Java 8: use Stream.anyMatch

File:

• trunk/src/org/openstreetmap/josm/data/osm/OsmPrimitive.java (modified) (6 diffs)

trunk/src/org/openstreetmap/josm/data/osm/OsmPrimitive.java

@@ -17 +17 @@
 import java.util.function.Consumer;
 import java.util.stream.Collectors;
+import java.util.stream.IntStream;
 import java.util.stream.Stream;
 
@@ -510 +511 @@
         if (referrers instanceof OsmPrimitive)
             return referrers instanceof Relation && ((OsmPrimitive) referrers).isSelected();
-        for (OsmPrimitive ref : (OsmPrimitive[]) referrers) {
-            if (ref instanceof Relation && ref.isSelected())
-                return true;
-        }
-        return false;
+        return Arrays.stream((OsmPrimitive[]) referrers)
+                .anyMatch(ref -> ref instanceof Relation && ref.isSelected());
     }
 
@@ -524 +522 @@
             return isOuterMemberOfMultipolygon((OsmPrimitive) referrers);
         }
-        for (OsmPrimitive ref : (OsmPrimitive[]) referrers) {
-            if (isOuterMemberOfMultipolygon(ref))
-                return true;
-        }
-        return false;
+        return Arrays.stream((OsmPrimitive[]) referrers)
+                .anyMatch(this::isOuterMemberOfMultipolygon);
     }
 
     private boolean isOuterMemberOfMultipolygon(OsmPrimitive ref) {
-        if (ref instanceof Relation && ref.isSelected() && ((Relation) ref).isMultipolygon()) {
-            for (RelationMember rm : ((Relation) ref).getMembersFor(Collections.singleton(this))) {
-                if ("outer".equals(rm.getRole())) {
-                    return true;
-                }
-            }
-        }
-        return false;
+        return ref instanceof Relation
+                && ref.isSelected()
+                && ((Relation) ref).isMultipolygon()
+                && ((Relation) ref).getMembersFor(Collections.singleton(this)).stream()
+                    .anyMatch(rm -> "outer".equals(rm.getRole()));
     }
 
@@ -576 +568 @@
 
     private void updateTagged() {
-        for (String key: keySet()) {
-            // 'area' is not really uninteresting (putting it in that list may have unpredictable side effects)
-            // but it's clearly not enough to consider an object as tagged (see #9261)
-            if (!isUninterestingKey(key) && !"area".equals(key)) {
-                updateFlagsNoLock(FLAG_TAGGED, true);
-                return;
-            }
-        }
-        updateFlagsNoLock(FLAG_TAGGED, false);
+        // 'area' is not really uninteresting (putting it in that list may have unpredictable side effects)
+        // but it's clearly not enough to consider an object as tagged (see #9261)
+        updateFlagsNoLock(FLAG_TAGGED, keySet().stream()
+                .anyMatch(key -> !isUninterestingKey(key) && !"area".equals(key)));
     }
 
     private void updateAnnotated() {
-        for (String key: keySet()) {
-            if (getWorkInProgressKeys().contains(key)) {
-                updateFlagsNoLock(FLAG_ANNOTATED, true);
-                return;
-            }
-        }
-        updateFlagsNoLock(FLAG_ANNOTATED, false);
+        updateFlagsNoLock(FLAG_ANNOTATED, keySet().stream()
+                .anyMatch(getWorkInProgressKeys()::contains));
     }
 
@@ -746 +728 @@
         } else if (referrers instanceof OsmPrimitive[]) {
             OsmPrimitive[] orig = (OsmPrimitive[]) referrers;
-            int idx = -1;
-            for (int i = 0; i < orig.length; i++) {
-                if (orig[i] == referrer) {
-                    idx = i;
-                    break;
-                }
-            }
+            int idx = IntStream.range(0, orig.length)
+                    .filter(i -> orig[i] == referrer)
+                    .findFirst().orElse(-1);
             if (idx == -1)
                 return;
@@ -904 +882 @@
 
     /**
-     * Merges the technical and semantical attributes from <code>other</code> onto this.
+     * Merges the technical and semantic attributes from <code>other</code> onto this.
      *
      * Both this and other must be new, or both must be assigned an OSM ID. If both this and <code>other</code>