JavaFX - Event Handler auf verschiedenen Objekten, die von for loop erstellt werden

Question

Ich möchte eine Gruppe von zehn verschiedenen Kreisen mit einer for-Schleife erstellen und jede von ihnen die Farbe ändern, wenn der Mauszeiger über eine von ihnen bewegt wird und sie auch ändern eine dritte Farbe mit einem Mausklick. Jedoch hat nur einer der Kreise - der letzte, der in der Schleife erstellt wird - die Farbänderungen, unabhängig davon, auf welchen Kreis geklickt oder über den man sich bewegt. Kann mir jemand erklären warum und wie kann ich das beheben? Ich wäre sehr dankbar. Hier ist mein Code:

public class View extends Parent{ 
BorderPane gameScreen; 
Group hexaBlock; 
ArrayList<Circle> circleList = new ArrayList<>(); 
Circle circle; 
... 
public View(){ 
     gameScreen = new BorderPane(); 
     hexaBlock = new Group(); 
     ... 
     for(int y=0; y<2; y++){ 
      for(double x=0; x<5; x++){ 
       circle = new Circle(xPosition(hexagon width*x), yPosition(hexagon height*4*y), radius); 
       circleList.add(circle); 
       circle.setFill(Color.BLACK); 
       circle.setOnMousePressed(new EventHandler<MouseEvent>() { 
        @Override 
        public void handle(MouseEvent event) { 
         circle.setFill(Color.CYAN); 
        } 
       }); 
       circle.addEventHandler(MouseEvent.MOUSE_ENTERED, new EventHandler<MouseEvent>(){ 
        @Override 
        public void handle(MouseEvent t) { 
         circle.setFill(Color.RED); 
        } 
       }); 
       circle.addEventHandler(MouseEvent.MOUSE_EXITED, new EventHandler<MouseEvent>(){ 
        @Override 
         public void handle(MouseEvent t) { 
         circle.setFill(Color.BLACK); 
         } 
       }); 
} 
this.getChildren().add(gameScreen); 
... 
gameScreen.setCenter(hexaBlock); 
... 
hexaBlock.getChildren().addAll(circleList); 

.....

Answer 1

Hier ist eine Beispielanwendung. Diese App verwendet Lambdas für die Zuhörer.

import java.util.Random; 
import javafx.application.Application; 
import javafx.scene.Scene; 
import javafx.scene.layout.AnchorPane; 
import javafx.scene.paint.Color; 
import javafx.scene.shape.Circle; 
import javafx.stage.Stage; 

/** 
* 
* @author blj0011 
*/ 
public class JavaFXApplication54 extends Application 
{ 

    @Override 
    public void start(Stage primaryStage) 
    { 
     Random random = new Random(); 

     AnchorPane root = new AnchorPane(); 

     for (int i = 0; i < 5; i++) 
     { 
      int x1 = random.nextInt(300); 
      System.out.println("l: " + x1); 
      int y1 = random.nextInt(250); 
      int radius = random.nextInt(10) + 3; 
      root.getChildren().add(getCircle(x1, y1, radius)); 
     } 

     Scene scene = new Scene(root, 300, 250); 

     primaryStage.setTitle("Hello World!"); 
     primaryStage.setScene(scene); 
     primaryStage.show(); 
    } 

    /** 
    * @param args the command line arguments 
    */ 
    public static void main(String[] args) 
    { 
     launch(args); 
    } 

    Circle getCircle(int x1, int y1, int radius) 
    { 
     Circle tempCircle = new Circle(x1, y1, radius); 
     tempCircle.setFill(Color.BLACK); 

     tempCircle.setOnMousePressed(me -> tempCircle.setFill(Color.CYAN)); 
     tempCircle.setOnMouseEntered(me -> tempCircle.setFill(Color.RED)); 
     tempCircle.setOnMouseExited(me -> tempCircle.setFill(Color.BLACK)); 

     return tempCircle; 
    } 
} 

Answer 2

Dies ist mein nehmen auf sie, die setOnMouseExited Kommentar-, wenn Sie es wollen wieder schwarz machen.

import java.util.Random; 

import javafx.application.Application; 
import javafx.scene.Scene; 
import javafx.scene.layout.AnchorPane; 
import javafx.scene.paint.Color; 
import javafx.scene.shape.Circle; 
import javafx.stage.Stage; 

public class CircleColours extends Application { 

    private final Random random = new Random(); 

    public static void main(String[] args) { 
     launch(args); 
    } 

    @Override 
    public void start(Stage primaryStage) { 
     AnchorPane pane = new AnchorPane(); 
     Scene scene = new Scene(pane, 600, 400); 
     addCircles(pane, 10, 50); 
     primaryStage.setScene(scene); 
     primaryStage.show(); 
    } 

    public void addCircles(AnchorPane pane, int amount, int radius) { 
     for (int i = 0; i < amount; i++) { 
      Circle circle = new Circle(random.nextInt((int) pane.getWidth()), random.nextInt((int) pane.getHeight()), radius); 
      circle.setOnMouseEntered(event -> circle.setFill(Color.rgb(random.nextInt(255), random.nextInt(255), random.nextInt(255)))); 
      circle.setOnMouseClicked(event -> circle.setFill(Color.rgb(random.nextInt(255), random.nextInt(255), random.nextInt(255)))); 
      //circle.setOnMouseExited(event -> circle.setFill(Color.BLACK)); 
      pane.getChildren().add(circle); 
     } 
    } 

} 

Answer 3

circle ist ein Feld. Wenn die Ereignisbehandlungsroutinen ausgeführt werden, wird der Wert des Felds abgerufen, und in diesem Fall enthält er den zuletzt zugewiesenen Wert, d. H. Den zuletzt erstellten Kreis.

Beachten Sie, dass Sie auf final (effektiv endgültig für Java> = 8) lokale Variablen in umgebenden Bereichen von anonyme Klassen zugreifen können. Ich empfehle das Entfernen des circle Feld und erklärt circle, wo Sie den Wert zuweisen:

for(int y=0; y<2; y++){ 
    for(double x=0; x<5; x++){ 
     final Circle circle = new Circle(xPosition(hexagon width*x), yPosition(hexagon height*4*y), radius); 

     ... 
     // init circle handlers/properties 
    } 
}