Added the ability to process coordinates before drawing. This will make as little requests to the api as possible to not overload the API or your computer

Author: CodingPupper3033

src/main/java/com/codingpupper3033/codebtekml/gui/screens/GuiDrawKML.java

@@ -5,6 +5,7 @@
 import com.codingpupper3033.codebtekml.gui.widgets.BlockPreview;
 import com.codingpupper3033.codebtekml.helpers.block.BlockNameConverter;
 import com.codingpupper3033.codebtekml.helpers.kmlfile.KMLParser;
+import com.codingpupper3033.codebtekml.helpers.map.Placemark;
 import com.codingpupper3033.codebtekml.helpers.map.altitude.AltitudeProcessor;
 import com.codingpupper3033.codebtekml.helpers.map.placemark.PlacemarkFactory;
 import net.minecraft.client.gui.GuiButton;
@@ -266,25 +267,29 @@ protected void actionPerformed(GuiButton button) throws IOException {
      * Using the values from the text boxes, try parsing and building the file.
      */
     public void build() {
-        Document[] documents;
-        try {
-            documents = KMLParser.parse(new File(fileNameTextBox.getText())); // TODO Add loading/parsing text (maybe below build button) to the screen to signify it is doing stuff even if it looks frozen
-        } catch (IOException e) {
-            errorFileText = ERROR_FINDING_FILE_TEXT;
-            return;
-        } catch (ParserConfigurationException e) {
-            errorFileText = ERROR_READING_FILE_TEXT;
-            return;
-        } catch (SAXException e) {
-            errorFileText = ERROR_READING_FILE_TEXT;
-            return;
-        }
+        new Thread(() -> { // New thread as to allow for loading screen TODO Make loading screen
+            Document[] documents;
+
+            try {
+                documents = KMLParser.parse(new File(fileNameTextBox.getText())); // TODO Add loading/parsing text (maybe below build button) to the screen to signify it is doing stuff even if it looks frozen
+            } catch (IOException e) {
+                errorFileText = ERROR_FINDING_FILE_TEXT;
+                return;
+            } catch (ParserConfigurationException e) {
+                errorFileText = ERROR_READING_FILE_TEXT;
+                return;
+            } catch (SAXException e) {
+                errorFileText = ERROR_READING_FILE_TEXT;
+                return;
+            }
+
+            Placemark[] placemarks = PlacemarkFactory.getPlacemarks(documents);
 
-        close(); // Should have successfully processed the file, gui is not needed now
+            // Process Altitudes
+            PlacemarkFactory.proccessPlacemarks(placemarks);
 
-        // Draw! (new thread, so you can see it doing its thing)
-        new Thread(() -> {
-            PlacemarkFactory.drawPlacemarks(documents, blockNameTextBox.getText());
+            close(); // Should have successfully processed the file, gui is not needed now
+            PlacemarkFactory.drawPlacemarks(placemarks, blockNameTextBox.getText()); // Draw!
         }).start();
     }
 

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

@@ -3,6 +3,7 @@
 import com.codingpupper3033.codebtekml.helpers.map.Coordinate;
 
 import java.io.IOException;
+import java.util.Arrays;
 import java.util.LinkedList;
 import java.util.Queue;
 
@@ -58,6 +59,10 @@ public void addCoordinateToProcessQueue(Coordinate coordinate) {
         coordinateProcessorQueue.add(coordinate);
     }
 
+    public void addCoordinatesToProcessQueue(Coordinate[] coordinates) {
+        coordinateProcessorQueue.addAll(Arrays.asList(coordinates));
+    }
+
     public Queue<Coordinate> getCoordinateProcessorQueue() {
         return coordinateProcessorQueue;
     }

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

@@ -2,6 +2,7 @@
 
 import com.codingpupper3033.codebtekml.helpers.kmlfile.KMLParser;
 import com.codingpupper3033.codebtekml.helpers.map.Placemark;
+import com.codingpupper3033.codebtekml.helpers.map.altitude.AltitudeProcessor;
 import org.w3c.dom.Document;
 import org.w3c.dom.Element;
 import org.w3c.dom.Node;
@@ -89,15 +90,24 @@ public static Placemark[] getPlacemarks(File f) throws ParserConfigurationExcept
         return getPlacemarks(KMLParser.parse(f));
     }
 
+    /**
+     * Draws all placemarks from the array
+     * @param placemarks Placemarks to draw
+     * @param blockName block name to draw with
+     */
+    public static void drawPlacemarks(Placemark[] placemarks, String blockName) {
+        for (Placemark placemark : placemarks) {
+            placemark.draw(blockName);
+        }
+    }
+
     /**
      * Draw all placemarks in the document with specified block name
      * @param documents documents to draw
      * @param blockName block name to draw with
      */
     public static void drawPlacemarks(Document[] documents, String blockName) {
-        for (Placemark placemark : getPlacemarks(documents)) {
-            placemark.draw(blockName);
-        }
+         drawPlacemarks(getPlacemarks(documents), blockName);
     }
 
     /**
@@ -111,4 +121,11 @@ public static void drawPlacemarks(Document[] documents, String blockName) {
     public static void drawPlacemarks(File f, String blockName) throws ParserConfigurationException, IOException, SAXException {
         drawPlacemarks(KMLParser.parse(f), blockName);
     }
+
+    public static void proccessPlacemarks(Placemark[] placemarks) {
+        for (Placemark placemark: placemarks) {
+            AltitudeProcessor.defaultProcessor.addCoordinatesToProcessQueue(placemark.getCoordinates());
+        }
+        AltitudeProcessor.defaultProcessor.processCoordinateQueue();
+    }
 }