javaFX でnodeの配列を作ったときに、それぞれのnodeからのActionを区別して処理したい

Question

前提

JavaFXでButtonクラスの配列を作りました。それぞれのButtonをクリックしたときに、どのButtonからのEventか識別する方法がよくわからず、長々と識別するためのコードを書いています。

実現したいこと

Button配列のそれぞれの要素に対するEventHandlerが、もう少しスマートに書けないかと思っています。

自分で識別コードを書かないと、配列の範囲外の添え字になってしまいます。

該当のソースコード

import javafx.application.Application;
import javafx.event.ActionEvent;
import javafx.event.EventHandler;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.control.Label;
import javafx.scene.layout.VBox;
import javafx.stage.Stage;
import javafx.geometry.Pos;
 
public class HelloWorld extends Application {
  static int i;
  static int actionButton;
  static final int numberOfButton = 2;
  
  @Override
  public void start(Stage primaryStage) {
    Label lbl = new Label();
    Button[] btn = new Button[5];
    for(i=0; i<numberOfButton; i++){
			btn[i] = new Button();
			btn[i].setText(String.valueOf(i));
	    btn[i].setOnAction(new EventHandler<ActionEvent>() {
	      @Override
	      public void handle(ActionEvent event) {
	        String souce = event.toString();
	        actionButton = Integer.valueOf(souce.substring(souce.indexOf("button")+8,souce.length()-2));
	        
	        System.out.println(actionButton);
	        if(lbl.getText().equals("")){
		        lbl.setText("Hello World"+"/"+actionButton);
		      }else{
		        lbl.setText("");
		      }
	      }
	    });
		}
    
    
    VBox root = new VBox(20);
    root.setAlignment(Pos.CENTER);
    for (i=0; i< numberOfButton; i++){
			root.getChildren().add(btn[numberOfButton-i-1]);
		}
    root.getChildren().add(lbl);
		
		Scene scene = new Scene(root, 300, 500);
		
    primaryStage.setTitle("Hello World!");
    primaryStage.setScene(scene);
    primaryStage.show();
  }
  
  public static void main(String[] args) {
		launch(args);
  }
}

補足情報（FW/ツールのバージョンなど）

OpenJX 18.0.1 JDK18.0.2

Comments

[jimbe]

souce ・・・ source でしょうか。

わざわざボタン毎にイベントハンドラを生成しているのですから、イベントハンドラ自体に値を渡して置けば良いだけでは。

そもそも toString の結果を解析するようなコードは、基本的にしてはいけないことです。
toString がどのような文字列を返す(べき)かは決まっていません。

[UserOfJava]

回答ありがとうございます。

イベントハンドラ自体に値を渡して

は、具体的にはどのようなコードになるのか、ご教示頂ければありがたいです。

Answer 1

ハンドラをクラス化してコンストラクタで値を渡しておくというのも方法の一つと思います。つまり「 0 番のボタン」と「 "0" と表示するハンドラ」を結びつける感じ。

  class ButtonHandler implements EventHandler<ActionEvent> {
    private int actionButton;
    private Label lbl;
    ButtonHandler(int actionButton, Label lbl) {
      this.actionButton = actionButton;
      this.lbl = lbl;
    }
    @Override
    public void handle(ActionEvent event) {
      System.out.println(actionButton);
      if(lbl.getText().equals("")) {
        lbl.setText("Hello World" + "/" + actionButton);
      } else {
        lbl.setText("");
      }
    }
  }

(ハンドラの登録部分)

    btn[i].setOnAction(new ButtonHandler(i, lbl));

また、ハンドラ内で振り分ることに拘るのであれば、 ActionEvent の getSource() というメソッドが押されたボタンを返します。

    @Override
    public void handle(ActionEvent event) {
      Button source = (Button)event.getSource();
      int actionButton = Integer.parseInt(source.getText());

      System.out.println(actionButton);
      if(lbl.getText().equals("")) {
        lbl.setText("Hello World" + "/" + actionButton);
      } else {
        lbl.setText("");
      }
    }

Comments

[UserOfJava]

早速、コードを例示して頂きありがとうございます。
やつてみます。

Answer 2

イベントハンドラ自体に値を渡して

は、具体的にはどのようなコードになるのか、ご教示頂ければありがたいです。

は、こういう意味かと思ってました^^;

import javafx.application.Application;
import javafx.event.ActionEvent;
import javafx.event.EventHandler;
import javafx.geometry.Pos;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.control.Label;
import javafx.scene.layout.VBox;
import javafx.stage.Stage;


public class HelloWorld extends Application {
    public static void main(String[] args) { launch(args); }

    static final int numberOfButton = 2;

    @Override public void start(Stage primaryStage) {
        Label lbl = new Label();

        // 「区別する」というと単一ハンドラで動作を変えるようなイメージだが。。。
//        EventHandler<ActionEvent> handler = (e) -> {
//            if (lbl.getText().equals("")) {
//                Button b = (Button) e.getSource();
//                lbl.setText("Hello World/" + b.getText());
//            } else {
//                lbl.setText("");
//            }
//        };

        Button[] btn = new Button[numberOfButton];
        for (int i = 0; i < btn.length; i++) {
            btn[i] = new Button("" + i);
//            btn[i].setOnAction(handler);

            final int ii = i; // ラムダには実質finalしか渡せないので一旦変数に受ける
            btn[i].setOnAction(event -> {
                if (lbl.getText().equals("")) {
                    lbl.setText("Hello World/" + ii); // iは参照できないがiiならｏｋ！
                } else {
                    lbl.setText("");
                }
            });
        }

        VBox root = new VBox(20);
        root.setAlignment(Pos.CENTER);
        for (int i = btn.length - 1; 0 <= i; i--) {
            root.getChildren().add(btn[i]);
        }

        root.getChildren().add(lbl);

        primaryStage.setTitle("Hello World!");
        primaryStage.setScene(new Scene(root, 300, 500));
        primaryStage.show();
    }
}

---

static int i;
たまにループの変数を外に出しているコードを見ますが、何の意味があるんでしょうか。

Comments

[jimbe]

>>イベントハンドラ自体に値を渡して
>こういう意味かと思ってました

最初それが一番簡単かと思ったのですが、コード内コメントのようなことを理解されているか分からなかったので普通に生成するようにしました。

[UserOfJava]

TN8001さん、ありがとうごさいます。
int final ii = i;
が肝ですね。ラムダ表記と。

[TN8001]

> 最初それが一番簡単かと思ったのですが、コード内コメントのようなことを理解されているか分からなかったので普通に生成するようにしました。

なるほど。
「だけ」例が抜けているような気がしたので回答しましたが、確かにわかりにくいところですからね。
C#なんかはエラーが出ない分、かえって混乱する奴ですｗ

Answer 3

この件に関し、回答を頂いた皆様に感謝しております。

その後、ネットを色々な形で検索して、私のコードとの関連で解説している記事があり、理解しました。
https://happynow.hateblo.jp/entry/20120512/1336826316

もっと早くに調べておけば良かったのですが、検索の語句がまずかったようです。

結局のところ、for文の i をそのまま参照せずに、いったん final int ii = i; で受けて、このiiでEventHandler のメソッドを書けばよいということのようです。　したがって、static i も使う必要がなくなりました。

皆様方のアドバイスのおかげで、Java の仕様の知らなかったことを勉強できました。
改めて、お礼申し上げます。

Answer 4

jimbeさん、TN8001さん
時間を割いて、アドバイスしていただきありがとうございました。

なまけ者なので、自分のコードの最小限の修正で済むようにしました。
static int i;
はなんだとのお叱りで恐縮します。でも、for文の i を内部クラスから参照するには、この方法が楽です。したがって、これは外せません。

もとのコードとの違いは、
---java
btn[i].setOnAction(new EventHandler<ActionEvent>() {
final int ii = i;
@Override
public void handle(ActionEvent event) {

とfinal int ii =i; の一行をいれて、handle method では、 i の代わりに ii を用いることで、うまく行きそうです。

お示ししたのは検証用のプログラムで、開発中のものは長くなるので、エッセンスだけ検証しました。

ラムダ表記はらくなように見えますが、取り付けていません。すみません。古い人間なので。

JavaFXのコンパイラの問題ですかね。別にfinalを持ち出さなくても、コードとしてはそれぞれの配列要素に処理を for 文の i で定義しているのですけど、実際にeventが発生すると、なにやら、for 文の i を引きずって、最後のi +1 で実行しようとします。

OpenJX のjavadoc 読んでも、なんか薄いような気がして、この先、JavaFXはなくなるのではと言う予感もするぐらいです。

皆さん、面白くもない検証に協力して頂いて、本当に感謝しています。

Comments

[jimbe]

> for文の i を内部クラスから参照するには、この方法が楽です。したがって、これは外せません。

これには賛同できません。
「for 文の i を引きずって、最後のi +1 で実行しようとします」がこの問題です。コンパイラ云々ではありません。

[UserOfJava]

あまり分かってなくて言っていますので、申し訳ありません。

[UserOfJava]

shiketaさん

node にユーザー指定の色々を持たせる仕組みですね。有益なアドバイス、ありがとうございます。
handler１本で、すべてのeventを処理できそうに思えます。
貴重な時間を割いて、ありがとうございました。

Answer 5

# 無用そうなループとか、問題を複雑にしそうなグローバル変数とか、突っ込みどころはいろいろありますが、それはさておき。

もう少しスマートに書けないかと思っています。

set/getUserData()を使ってみては?

• https://docs.oracle.com/javase/jp/8/javafx/api/javafx/scene/Node.html#setUserData-java.lang.Object-

import javafx.application.Application;
import javafx.event.ActionEvent;
import javafx.event.EventHandler;
import javafx.geometry.Pos;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.control.Label;
import javafx.scene.layout.VBox;
import javafx.stage.Stage;

public class HelloWorld
    extends Application {
  static final int numberOfButton = 2;

  @Override
  public void start(Stage primaryStage) {
    final Label lbl = new Label();
    final Button[] btn = new Button[numberOfButton];

    final EventHandler<ActionEvent> handler = new EventHandler<ActionEvent>() {
      @Override
      public void handle(ActionEvent event) {
        final int actionButton = (Integer) ((Button) event.getSource()).getUserData();

        System.out.println(actionButton);
        if (lbl.getText().equals("")) {
          lbl.setText("Hello World" + "/" + actionButton);
        } else {
          lbl.setText("");
        }
      }
    };

    final VBox root = new VBox(20);
    root.setAlignment(Pos.CENTER);

    for (int i = 0; i < numberOfButton; i++) {
      final Button button = new Button();
      root.getChildren().add(button);
      button.setText(String.valueOf(i));
      button.setUserData(i);
      button.setOnAction(handler);
      btn[i] = button;
    }

    root.getChildren().add(lbl);

    final Scene scene = new Scene(root, 300, 500);

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

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