Changeset 9711 in josm for trunk/src/org/openstreetmap/josm/gui/datatransfer

Timestamp:

2016-02-01T23:47:27+01:00 (8 years ago)

Author:

simon04

Message:

see #12300 - Provide wrapper classes for transfer data to avoid unchecked casts

Location:

trunk/src/org/openstreetmap/josm/gui/datatransfer

Files:

• PrimitiveTransferable.java (modified) (3 diffs)
• RelationMemberTransferable.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/datatransfer/PrimitiveTransferable.java

@@ -19 +19 @@
 
     /**
-     * Data flavor for {@link PrimitiveData}.
+     * A wrapper for a collection of {@link PrimitiveData}.
      */
-    public static final DataFlavor PRIMITIVE_DATA = new DataFlavor(PrimitiveData.class, PrimitiveData.class.getName());
+    public static final class Data {
+        private final Collection<PrimitiveData> primitiveData;
+
+        private Data(Collection<PrimitiveData> primitiveData) {
+            this.primitiveData = primitiveData;
+        }
+
+        /**
+         * Returns the contained {@link PrimitiveData}
+         * @return the contained {@link PrimitiveData}
+         */
+        public Collection<PrimitiveData> getPrimitiveData() {
+            return primitiveData;
+        }
+    }
+
+    /**
+     * Data flavor for {@link PrimitiveData} which is wrapped in {@link Data}.
+     */
+    public static final DataFlavor PRIMITIVE_DATA = new DataFlavor(Data.class, Data.class.getName());
     private final Collection<? extends OsmPrimitive> primitives;
 
@@ -47 +66 @@
             return getStringData();
         } else if (PRIMITIVE_DATA.equals(flavor)) {
-            return getRelationMemberData();
+            return getPrimitiveData();
         }
         throw new UnsupportedFlavorException(flavor);
@@ -63 +82 @@
     }
 
-    protected Collection<PrimitiveData> getRelationMemberData() {
+    protected Data getPrimitiveData() {
         final Collection<PrimitiveData> r = new ArrayList<>(primitives.size());
         for (OsmPrimitive primitive : primitives) {
             r.add(primitive.save());
         }
-        return r;
+        return new Data(r);
     }
 }

trunk/src/org/openstreetmap/josm/gui/datatransfer/RelationMemberTransferable.java

@@ -19 +19 @@
 
     /**
-     * Data flavor for {@link RelationMemberData}.
+     * A wrapper for a collection of {@link RelationMemberData}.
      */
-    public static final DataFlavor RELATION_MEMBER_DATA = new DataFlavor(RelationMemberData.class, RelationMemberData.class.getName());
+    public static final class Data {
+        private final Collection<RelationMemberData> relationMemberDatas;
+
+        private Data(Collection<RelationMemberData> primitiveData) {
+            this.relationMemberDatas = primitiveData;
+        }
+
+        /**
+         * Returns the contained {@link RelationMemberData}
+         * @return the contained {@link RelationMemberData}
+         */
+        public Collection<RelationMemberData> getRelationMemberData() {
+            return relationMemberDatas;
+        }
+    }
+
+    /**
+     * Data flavor for {@link RelationMemberData} which is wrapped in {@link Data}.
+     */
+    public static final DataFlavor RELATION_MEMBER_DATA = new DataFlavor(Data.class, Data.class.getName());
     private final Collection<RelationMember> members;
 
@@ -64 +83 @@
     }
 
-    protected Collection<RelationMemberData> getRelationMemberData() {
+    protected Data getRelationMemberData() {
         final Collection<RelationMemberData> r = new ArrayList<>(members.size());
         for (RelationMember member : members) {
             r.add(new RelationMemberData(member.getRole(), member.getType(), member.getUniqueId()));
         }
-        return r;
+        return new Data(r);
     }
 }