New feature: SmartGraphPanel.updateAndWait()

Author: brunomnsilva

releases/JavaFXSmartGraph-0.9.jar

@@ -40,6 +40,7 @@
 import com.brunomnsilva.smartgraph.graph.DigraphEdgeList;
 import com.brunomnsilva.smartgraph.graph.Edge;
 import com.brunomnsilva.smartgraph.graphview.SmartCircularSortedPlacementStrategy;
+import com.brunomnsilva.smartgraph.graphview.SmartStylableNode;
 
 /**
  *
@@ -232,13 +233,25 @@ private void continuously_test_adding_elements(Graph<String, String> g, SmartGra
                     Vertex<String> existing = get_random_vertex(g);
                     Vertex<String> vertexId = g.insertVertex(("V" + id));
                     g.insertEdge(existing, vertexId, ("E" + id));
+                    
+                    //this variant must be called to ensure the view has reflected the
+                    //underlying graph before styling a node immediately after.
+                    graphView.updateAndWait();
+                    
+                    //color new vertices
+                    SmartStylableNode stylableVertex = graphView.getStylableVertex(vertexId);
+                    if(stylableVertex != null) {
+                        stylableVertex.setStyle("-fx-fill: orange;");
+                    }
                 } else {
                     Vertex<String> existing1 = get_random_vertex(g);
                     Vertex<String> existing2 = get_random_vertex(g);
                     g.insertEdge(existing1, existing2, ("E" + id));
+                    
+                    graphView.update();
                 }
 
-                graphView.update();
+                
             }
         };
 

releases/JavaFXSmartGraph-0.9.zip

@@ -58,6 +58,9 @@
 import static com.brunomnsilva.smartgraph.graphview.UtilitiesJavaFX.pick;
 import static com.brunomnsilva.smartgraph.graphview.UtilitiesPoint2D.attractiveForce;
 import static com.brunomnsilva.smartgraph.graphview.UtilitiesPoint2D.repellingForce;
+import java.util.concurrent.Callable;
+import java.util.concurrent.ExecutionException;
+import java.util.concurrent.FutureTask;
 
 /**
  * JavaFX {@link Pane} that is capable of plotting a {@link Graph} or {@link Digraph}.
@@ -271,7 +274,7 @@ public void setAutomaticLayout(boolean value) {
 
     /**
      * Forces a refresh of the visualization based on current state of the
-     * underlying graph. 
+     * underlying graph, immediately returning to the caller.
      * 
      * This method invokes the refresh in the graphical
      * thread through Platform.runLater(), so its not guaranteed that the visualization is in sync
@@ -297,6 +300,46 @@ public void update() {
         });
 
     }
+    
+    /**
+     * Forces a refresh of the visualization based on current state of the
+     * underlying graph and waits for completion of the update.
+     * 
+     * Use this variant only when necessary, e.g., need to style an element
+     * immediately after adding it to the underlying graph. Otherwise, use
+     * {@link #update() } instead for performance sake.
+     * <p>
+     * New vertices will be added close to adjacent ones or randomly for
+     * isolated vertices.
+     */
+    public void updateAndWait() {
+        if (this.getScene() == null) {
+            throw new IllegalStateException("You must call this method after the instance was added to a scene.");
+        }
+
+        if (!this.initialized) {
+            throw new IllegalStateException("You must call init() method before any updates.");
+        }
+        
+        final FutureTask update = new FutureTask(new Callable<Boolean>() {
+            @Override
+            public Boolean call() throws Exception {
+                updateNodes();
+                return true;
+            }
+        });
+
+        //this will be called from a non-javafx thread, so this must be guaranteed to run of the graphics thread
+        Platform.runLater(update);
+        
+        try {
+            //wait for completion
+            update.get();
+        } catch (InterruptedException | ExecutionException ex) {
+            Logger.getLogger(SmartGraphPanel.class.getName()).log(Level.SEVERE, null, ex);
+        }
+
+    }
 
     private synchronized void updateNodes() {
         removeNodes();