Changeset 2077 in josm for trunk/src/org/openstreetmap/josm/data/osm/DataSet.java

Timestamp:

2009-09-07T23:06:19+02:00 (15 years ago)

Author:

Gubaer

Message:

Had to replace DataSet:getPrimitiveById(id) with DataSet:getPrimitiveById(id,type). Primitive ids are not globally unique, only per type of primitive.
Fixed problems in unit test, available unit tests passing again.

File:

• trunk/src/org/openstreetmap/josm/data/osm/DataSet.java (modified) (2 diffs)

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

@@ -313 +313 @@
      *
      * @param id  the id, > 0 required
+     * @param type the type of  the primitive. Must not be null.
      * @return the primitive
      * @exception IllegalArgumentException thrown, if id <= 0
-     */
-    public OsmPrimitive getPrimitiveById(long id) {
+     * @exception IllegalArgumentException thrown, if type is null
+     * @exception IllegalArgumentException thrown, if type is neither NODE, or WAY or RELATION
+     */
+    public OsmPrimitive getPrimitiveById(long id, OsmPrimitiveType type) {
         if (id <= 0)
             throw new IllegalArgumentException(tr("parameter {0} > 0 required. Got {1}.", "id", id));
-        for (OsmPrimitive primitive : nodes) {
-            if (primitive.getId() == id) return primitive;
-        }
-        for (OsmPrimitive primitive : ways) {
-            if (primitive.getId() == id) return primitive;
-        }
-        for (OsmPrimitive primitive : relations) {
+        if (id <= 0)
+            throw new IllegalArgumentException(tr("paramete''{0}'' must not be null", "type"));
+        Collection<? extends OsmPrimitive> primitives = null;
+        switch(type) {
+        case NODE: primitives = nodes; break;
+        case WAY: primitives = ways; break;
+        case RELATION: primitives = relations; break;
+        case CHANGESET: throw new IllegalArgumentException(tr("unsupported value ''{0}'' or parameter ''{1}''", type, "type"));
+        }
+        for (OsmPrimitive primitive : primitives) {
             if (primitive.getId() == id) return primitive;
         }
@@ -341 +347 @@
         for (OsmPrimitive primitive : relations) {
             ret.add(primitive.getId());
-        }
-        return ret;
-    }
-
-    /**
-     * Replies the set of ids of all complete primitives (i.e. those with
-     * ! primitive.incomplete)
-     *
-     * @return the set of ids of all complete primitives
-     */
-    public Set<Long> getCompletePrimitiveIds() {
-        HashSet<Long> ret = new HashSet<Long>();
-        for (OsmPrimitive primitive : nodes) {
-            if (!primitive.incomplete) {
-                ret.add(primitive.getId());
-            }
-        }
-        for (OsmPrimitive primitive : ways) {
-            if (! primitive.incomplete) {
-                ret.add(primitive.getId());
-            }
-        }
-        for (OsmPrimitive primitive : relations) {
-            if (! primitive.incomplete) {
-                ret.add(primitive.getId());
-            }
         }
         return ret;