Alles jetzt direkt im Forum geschrieben - Tippfehler und so bitte daher eigenständig ausbessern:
Wie genau soll so etwas aussehen?
Also die erste Variante, die ich angesprochen habe mit Konstruktor und Übergabe der Instanz:
[CODE lang="java" highlight="12,35-37"]public class MausMalenFX extends Application{
private Label lab;
@Override
public void start(Stage meinStage) throws Exception {
FlowPane root = new FlowPane();
lab = new Label("Hello");
root.getChildren().add(lab);
Canvas meinCanvas = new Canvas(400, 400);
root.getChildren().add(meinCanvas);
Scene meineScene = new Scene(root, 400, 400);
Maus maus = new Maus(this);
meinCanvas.setOnMouseClicked(maus);
meinStage.setScene(meineScene);
meinStage.setTitle("mein Titel");
meinStage.show();
}
public void labelsetzen(String x) {
lab.setText(x);
}
public static void main(String[] args) {
launch(args);
}
}
class Maus implements EventHandler<MouseEvent>{
private MausMalenFX neu;
public Maus(MausMalenFX neu) {
this.neu = neu;
}
@Override
public void handle(MouseEvent e) {
neu.labelsetzen("ändern");
}
}[/CODE]
Die Variante mit der Methodenreferenz (Zu der ich immer raten würde! Oder alternative mit Lambda Expression):
[CODE lang="java" title="Lambda Expression"]public class MausMalenFX extends Application{
private Label lab;
@Override
public void start(Stage meinStage) throws Exception {
FlowPane root = new FlowPane();
lab = new Label("Hello");
root.getChildren().add(lab);
Canvas meinCanvas = new Canvas(400, 400);
root.getChildren().add(meinCanvas);
Scene meineScene = new Scene(root, 400, 400);
meinCanvas.setOnMouseClicked(e -> labelsetzen("ändern"));
meinStage.setScene(meineScene);
meinStage.setTitle("mein Titel");
meinStage.show();
}
public void labelsetzen(String x) {
lab.setText(x);
}
public static void main(String[] args) {
launch(args);
}
}[/CODE]
[CODE lang="java" title="Methoden Referenz"]public class MausMalenFX extends Application{
private Label lab;
@Override
public void start(Stage meinStage) throws Exception {
FlowPane root = new FlowPane();
lab = new Label("Hello");
root.getChildren().add(lab);
Canvas meinCanvas = new Canvas(400, 400);
root.getChildren().add(meinCanvas);
Scene meineScene = new Scene(root, 400, 400);
meinCanvas.setOnMouseClicked(this::setLabelToAendern);
meinStage.setScene(meineScene);
meinStage.setTitle("mein Titel");
meinStage.show();
}
public void labelsetzen(String x) {
lab.setText(x);
}
public void setLabelToAendern(MouseEvent e) {
labelsetzen("aendern");
}
public static void main(String[] args) {
launch(args);
}
}[/CODE]
Lambda Ausdruck ist halt sehr gut, wenn man nur einen einzigen Aufruf hat und die Parameter der Methode nicht dem entsprechen, was man braucht. Hier wird MouseEvent ja nicht benutzt...
Aber LambdaAudrücke sollten nicht ganze Blöcke sein. Das geht thoretisch zwar auch, aber das ist unleserlich. Da ist dann eine eigenständige Methode besser. So wie im letzten Beispiel die Methode setlabelToAendern. Die Methode kann genau so heissen, dass klar ist, was sie macht. Und dann sieht man z.B. auch direkt an einer Zeile, was passiert:
meinCanvas.setOnMouseClicked(this::setLabelToAendern);
Da steht ja gut lesbar: Wenn Du auf meinCanvas clickst, dann wird das Label auf aendern gesetzt.
