CODEWITHSUNDEEP
javajavafxjavafx-8
theAnon
theAnon

Reputation: 77

Handling nested events

I have a game that displays an archer and I am trying to set it up so that when my button is pressed, the user is then allowed to click anywhere on the screen and then a new archer would be set to where the click happen. My code currently allows me to click the screen and set a new archer regardless of whether the button was pressed or not. Could someone explain what is wrong becuase I though MouseEvent would occur on the scene once the button was pressed.

 myButton.setOnMouseClicked(
                    new EventHandler<MouseEvent>()
                    {
                        public void handle(MouseEvent e)
                        {
                            gc.setFill(color);
                            gc.fillRect(0,0,width,height);
                            scene.setOnMouseClicked(new EventHandler<MouseEvent>()
                            {
                                public void handle(MouseEvent e)
                                {
                                    archer.setX((int)e.getSceneX());
                                    archer.setY((int)e.getSceneY());
                                    archer.drawCharStand(gc);
                                }
                            });
                        }
                    });

Upvotes: 0

Views: 917

Answers (2)

James_D
James_D

Reputation: 209704

You could use a ToggleButton, so that you only place the archer when the toggle button is selected:

private ToggleButton myButton = new ToggleButton("Place Archer");

// ...

scene.setOnMouseClicked(e -> {
    if (myButton.isSelected()) {
         archer.setX((int)e.getSceneX());
         archer.setY((int)e.getSceneY());
         archer.drawCharStand(gc);
         myButton.setSelected(false);
    }
});

The last line will unselect the toggle button "automatically" after placing the archer. If you want the user to be able to place multiple archers easily (and have to manually switch off that mode), omit that line.

Upvotes: 1

MojoJojo
MojoJojo

Reputation: 4242

You will need to change your code slightly. Perhaps try with a variable that tracks if the button was clicked:

boolean archerPlacementMode = false;
....
myButton.setOnMouseClicked(new EventHandler<MouseEvent>(){
    public void handle(MouseEvent e)
            {
                if(!archerPlacementMode) {
                     archerPlacementMode = true;
                     gc.setFill(color);
                     gc.fillRect(0,0,width,height);
                     archerPlacementMode = true;
                      return;
                }
            }
        });

scene.setOnMouseClicked(new EventHandler<MouseEvent>() {
    public void handle(MouseEvent e)
    {
        if(archerPlacementMode) {
            archer.setX((int)e.getSceneX());
            archer.setY((int)e.getSceneY());
            archer.drawCharStand(gc);
            archerPlacementMode = false;
        }
    }
});

Upvotes: 0

Related Questions