Changeset 17524 in josm for trunk/src/org/openstreetmap/josm/data/conflict/ConflictCollection.java

Timestamp:

2021-02-22T13:59:25+01:00 (3 years ago)

Author:

Don-vip

Message:

fix #20440 - don't create SubclassFilteredCollection just for counting instances in conflict dialog title

File:

• trunk/src/org/openstreetmap/josm/data/conflict/ConflictCollection.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/data/conflict/ConflictCollection.java

@@ -11 +11 @@
 import java.util.Set;
 import java.util.concurrent.CopyOnWriteArrayList;
+import java.util.function.Predicate;
 import java.util.stream.Collectors;
 
@@ -344 +345 @@
     }
 
+    /**
+     * Returns the number of conflicts involving nodes.
+     * @return The number of conflicts involving nodes.
+     * @since 17524
+     */
+    public final long getNumberOfNodeConflicts() {
+        return getNumberOfConflicts(c -> c.getMy() instanceof Node);
+    }
+
+    /**
+     * Returns the number of conflicts involving nodes.
+     * @return The number of conflicts involving nodes.
+     * @since 17524
+     */
+    public final long getNumberOfWayConflicts() {
+        return getNumberOfConflicts(c -> c.getMy() instanceof Way);
+    }
+
+    /**
+     * Returns the number of conflicts involving nodes.
+     * @return The number of conflicts involving nodes.
+     * @since 17524
+     */
+    public final long getNumberOfRelationConflicts() {
+        return getNumberOfConflicts(c -> c.getMy() instanceof Relation);
+    }
+
+    private long getNumberOfConflicts(Predicate<Conflict<?>> predicate) {
+        return conflicts.isEmpty() ? 0 : conflicts.stream().filter(Objects::nonNull).filter(predicate).count();
+    }
+
     @Override
     public int hashCode() {