Changeset 6316 in josm for trunk/src/org/openstreetmap/josm/data/projection/datum

Timestamp:

2013-10-07T20:18:17+02:00 (11 years ago)

Author:

Don-vip

Message:

Sonar/FindBugs - Loose coupling

File:

• trunk/src/org/openstreetmap/josm/data/projection/datum/NTV2GridShiftFile.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/data/projection/datum/NTV2GridShiftFile.java

@@ -25 +25 @@
 import java.util.ArrayList;
 import java.util.HashMap;
+import java.util.List;
 
 import org.openstreetmap.josm.tools.Utils;
@@ -166 +167 @@
     private NTV2SubGrid[] createSubGridTree(NTV2SubGrid[] subGrid) {
         int topLevelCount = 0;
-        HashMap<String, ArrayList<NTV2SubGrid>> subGridMap = new HashMap<String, ArrayList<NTV2SubGrid>>();
+        HashMap<String, List<NTV2SubGrid>> subGridMap = new HashMap<String, List<NTV2SubGrid>>();
         for (int i = 0; i < subGrid.length; i++) {
             if (subGrid[i].getParentSubGridName().equalsIgnoreCase("NONE")) {
@@ -179 +180 @@
                 topLevelSubGrid[topLevelCount++] = subGrid[i];
             } else {
-                ArrayList<NTV2SubGrid> parent = subGridMap.get(subGrid[i].getParentSubGridName());
+                List<NTV2SubGrid> parent = subGridMap.get(subGrid[i].getParentSubGridName());
                 parent.add(subGrid[i]);
             }
@@ -185 +186 @@
         NTV2SubGrid[] nullArray = new NTV2SubGrid[0];
         for (int i = 0; i < subGrid.length; i++) {
-            ArrayList<NTV2SubGrid> subSubGrids = subGridMap.get(subGrid[i].getSubGridName());
+            List<NTV2SubGrid> subSubGrids = subGridMap.get(subGrid[i].getSubGridName());
             if (!subSubGrids.isEmpty()) {
                 NTV2SubGrid[] subGridArray = subSubGrids.toArray(nullArray);