Adding ability to process coordinates before drawing. Adding the ability to Placemark and all subclasses

Author: CodingPupper3033

src/main/java/com/codingpupper3033/codebtekml/events/KeyPressed.java

@@ -1,7 +1,7 @@
 package com.codingpupper3033.codebtekml.events;
 
 import com.codingpupper3033.codebtekml.KeyInit;
-import com.codingpupper3033.codebtekml.helpers.map.altitude.AltitudeProcessor;
+import com.codingpupper3033.codebtekml.gui.screens.GuiDrawKML;
 import net.minecraft.client.Minecraft;
 import net.minecraftforge.fml.common.eventhandler.SubscribeEvent;
 import net.minecraftforge.fml.common.gameevent.TickEvent.ClientTickEvent;
@@ -15,8 +15,7 @@ public class KeyPressed {
     public void clientTick(ClientTickEvent event) {
         if (Minecraft.getMinecraft().currentScreen == null) { // Keystrokes that only work on the default game menu
             // Open KML Menu
-            //if (KeyInit.openKMLMenuKeybind.isPressed()) Minecraft.getMinecraft().displayGuiScreen(new GuiDrawKML(null));
-            if (KeyInit.openKMLMenuKeybind.isPressed()) AltitudeProcessor.defaultProcessor.processCoordinateQueue();
+            if (KeyInit.openKMLMenuKeybind.isPressed()) Minecraft.getMinecraft().displayGuiScreen(new GuiDrawKML(null));
         }
     }
 }

src/main/java/com/codingpupper3033/codebtekml/helpers/map/Placemark.java

@@ -16,7 +16,11 @@ public Placemark(Element element) {
 
     }
 
-    //public abstract Coordinate[] getCoordinates();
+    /**
+     * Gets all coordinates that this placemark uses, so we can pre-process them
+     * @return all the Coordinates in this placemark
+     */
+    public abstract Coordinate[] getCoordinates();
 
     /**
      * Should draw the placemark into the game

src/main/java/com/codingpupper3033/codebtekml/helpers/map/altitude/AltitudeProcessor.java

@@ -28,7 +28,7 @@ public class AltitudeProcessor {
     /**
      * List of coordinates to process
      */
-    public Queue<Coordinate> coordinateProcessorQueue = new LinkedList<>();
+    private Queue<Coordinate> coordinateProcessorQueue = new LinkedList<>();
 
     /**
      * Default altitude method. Should not be used.

src/main/java/com/codingpupper3033/codebtekml/helpers/map/altitude/OpenElevationAltitudeProcessor.java

@@ -96,19 +96,15 @@ public double[] getGroundLevels(Coordinate[] coordinates) throws IOException {
     public boolean processCoordinateQueue() {
         super.processCoordinateQueue();
 
-        for (int i = 0; i <17; i++) {
-            coordinateProcessorQueue.add(new Coordinate(42.7264177662084, -73.67230722692297,0,AltitudeMode.RELATIVE_TO_GROUND));
-        }
-
-        while (!coordinateProcessorQueue.isEmpty()) { // Keep working until all out
-            Coordinate[] setOfLocationsToProcess = new Coordinate[Math.min(MAX_COORDINATES_PER_REQUEST,coordinateProcessorQueue.size())]; // Make it only as big as needed
+        while (!getCoordinateProcessorQueue().isEmpty()) { // Keep working until all out
+            Coordinate[] setOfLocationsToProcess = new Coordinate[Math.min(MAX_COORDINATES_PER_REQUEST,getCoordinateProcessorQueue().size())]; // Make it only as big as needed
             int i = 0;
-            while (!coordinateProcessorQueue.isEmpty() && i < setOfLocationsToProcess.length) { // Only do the max size at a time
-                setOfLocationsToProcess[i] = coordinateProcessorQueue.poll();
+            while (!getCoordinateProcessorQueue().isEmpty() && i < setOfLocationsToProcess.length) { // Only do the max size at a time
+                setOfLocationsToProcess[i] = getCoordinateProcessorQueue().poll();
                 i++;
             }
 
-            // Process them
+            // Process this set
             try {
                 double[] groundLevels = getGroundLevels(setOfLocationsToProcess);
 

src/main/java/com/codingpupper3033/codebtekml/helpers/map/placemark/LineString.java

@@ -21,6 +21,11 @@ public LineString(Element element) {
         coordinates = KMLParser.getElementCoordinates(element);
     }
 
+    @Override
+    public Coordinate[] getCoordinates() {
+        return coordinates;
+    }
+
     @Override
     public void draw(String blockName) {
         for (int i = 1; i < coordinates.length; i++) {

src/main/java/com/codingpupper3033/codebtekml/helpers/map/placemark/Point.java

@@ -19,6 +19,11 @@ public Point(Element element) {
         coordinate = KMLParser.getElementCoordinates(element)[0]; // Only one coordinate as it is a point
     }
 
+    @Override
+    public Coordinate[] getCoordinates() {
+        return new Coordinate[]{coordinate};
+    }
+
     @Override
     public void draw(String blockName) {
         MinecraftCommands.set(coordinate, blockName); // Use World Edit set command as a point will just be a block