Replace a node at (row,col) in a JavaFX GridPane

Question

I am making a grid-style game/simulation based on bugs "sensing" and eating food. I am using a gridPane (called worldGrid) of labels to show the grid of bugs and food. This is obviously going to be constantly updated when a bug moves cells towards food etc.

I currently have a function updateGrid(int col, int row, String cellContent) which I want to replace the label at [row,col] with a label that has the new text in cellContent.

I have the follow which works

worldGrid.add(new Label(cellContent), row,col);

however im worried that that is just adding a label on top of the current label and obviously over 100 iterations of the simulation thats not ideal.

I have tried this before adding the label:

worldGrid.getChildren().remove(row,col);

However I then get an IllegalArgumentException when trying to do the add line.

Any ideas on how to do this? Or even better, any ideas on how best to show a constantly changing grid that will eventually use sprites instead of text?

score 18 · Accepted Answer · edited Sep 18 '16 at 04:56

18

The col/row provided by grid.add(node, col, row) (ATTENTION first comes col!) is only a layout constraint. This does not provide any means to access columns or rows like slots in a 2-dimensional array. So to replace a node, you have to know its object itself, e.g. remember them in a separate array.

Then you are able to call getChildren().remove(object)... e.g.:

GridPane grid = new GridPane();

Label first = new Label("first");
Label second = new Label("second");

grid.add(first,  1,  1);
grid.add(second,  2,  2);
second.setOnMouseClicked(e -> {
    grid.getChildren().remove(second);
    grid.add(new Label("last"), 2, 2);
});
box.getChildren().addAll(grid);

edited Sep 18 '16 at 04:56

sohnryang

725
2
13
27

answered Nov 26 '14 at 18:41

Jens-Peter Haack

1,887
13
18

Of course, using remove(object) has to search the object linearily in the whole children list, so on huge grids, it might be not very efficient. (If a game need freuqent and fast updates) – Jens-Peter Haack Nov 26 '14 at 18:43
If you have more than 1 column in your grid (which surely you do), you have to remove the whole row and add it again with the new Node and keeping the same as equal. – Joe Almore Dec 02 '16 at 15:19
1

Joe Almore that's absolutely not true. You can remove a single node and then add another at its position, just as the example shows – xeruf Dec 10 '17 at 13:59

score 2 · Answer 2 · edited Mar 26 '16 at 23:06

I agree with Jens-Peter but I would add that you can use GridPane's getColumnIndex and getRowIndex to obtain a particular node's location.

For example ( this is in a component which extends GridPane ):

// set the appropriate label
for (Node node : getChildren()) {
    if (node instanceof Label
     && getColumnIndex(node) == column
     && getRowIndex(node) == row) {
        ((Label)node).setTooltip(new Tooltip(tooltip));
    }
}

in other words, go through all the nodes in the GridPane and check for a match of the conditions you want, in this case row and column.

score 0 · Answer 3 · answered Mar 21 '17 at 18:25

0

You called getChildren().remove() method directly will cause the gridpane to go out of sync with the constraints. When you add, it also setup the constraint for the node. Add clearConstraints() method.

answered Mar 21 '17 at 18:25

PhucLy

43
7

Replace a node at (row,col) in a JavaFX GridPane

3 Answers3

Linked