ボタンアクション２

Question

for文で作成したボタンにアクションを付けたのですが、このプログラムだと一番最後に作成されたボタンしかアクションが反映されません。どのように改善するとよいでしょうか。
プログラミング初心者でほとんど手探りで学習しているため初歩的なことを聞いて申し訳ございません。

 private void createButton() {

        for (int i = 0; i < a; i++) {
         index=i;
         Button[i].setOnAction(actionHandler);
        }
    
    Container.getChildren().add(vbox);    
      
  }
   EventHandler<ActionEvent>actionHandler = new EventHandler<ActionEvent>(){
       @Override
       public void handle(ActionEvent e){
     
         for (int i = 0; i < a; i++){
            if(Button[index].equals(Button[i])){
             画面遷移;
            }
         }
      }
   }; 

Comments

[KSwordOfHaste]

字下げがおかしいと閲覧者がコードを見るのに手間どります。適切な字下げをするようご注意ください。

[GFC]

とりあえず字下げしてみましたが、このような感じでいいのでしょうか？

[KSwordOfHaste]

元よりは見やすくなりました。ただ一定の字下げ幅になっていないのでベストではありませんね。字下げの文字数を4文字でも2文字でもよいので統一するのがベストだと思います。

Answer 1

質問者さんのハンドラーの中身についてコメントします。ハンドラーの中でnameButtons[index]とnameButtons[i]をループで比較していることから質問者さんの意図はハンドラーの中で最初のボタンならindex=0、次のボタンならindex=1となることを期待しているはずです。しかしおそらくindexの宣言の位置が不適切なため意図通りに動作しないと思います。

意図通りにするには次のようにforループの中でindexを「finalなローカル変数として宣言」する必要があります。またハンドラーの実体はこのローカル変数indexを参照できるスコープ内(つまりループの内側)でlambda式として参照するか無名クラスとしてnewする必要があります。（おそらく質問者さんのコードはindexをフィールドにしているかforループの外にあるローカル変数として宣言していると思われます。)

for (int i = 0; i < a; i++) {
  nameButtons[i] = new Button();
  ...
  int index = i; // ここでローカル変数として「宣言」することが必須
  // (1) lambda式でハンドラーを記述する例(Java1.8以降でしか使えません)
  nameButtons[i].setOnAction(ev -> {
    Button actionButton = nameButtons[index]; //<-最初のボタンはindex=0,次はindex=1になります
    ...
  });
  // (2) 無名クラスでハンドラーを記述する例。(1)と同じ動作になります。
  // Java1.8より前のバージョンではこちらの書き方になります。でもJavaFXを使っているということは
  // 多分Java1.8をお使いだと思うので(1)を推奨します。
  nameButtons[i].setOnAction(new EventHandler<ActionEvent>() {
    @Override
    public void actionPerformed(ActionEvent ev) {
      Button actionButton = nameButtons[index];
      ...
    }
  });
}

さらに、ご質問のコードのハンドラー内でのループによる比較には意味がありません。なぜならループ内のif文はどのボタンでも必ずtrueになり実行されてしまうからです。ループによる比較を行わずとも何番目のボタンなのかは最初からindexに入っているはずです。

Comments

[GFC]

ありがとうございます無事解決できました。また、プログラムの字下げなどの指摘も勉強になりました。

Answer 2

nameButton[i].setOnAction(actionHandler);

この部分を

nameButton[i].setOnAction(new EventHandler<ActionEvent>(){
    @Override
    public void handle(ActionEvent e){
        for (int i = 0; i < a; i++)
        {
            if(nameButton[index].equals(nameButton[i])){
                画面遷移;
            }
        }
     }
   });

とすれば良いと思います。

質問の回答では、毎回同じオブジェクトが登録されています。
違うオブジェクトを登録する必要があります。
それぞれ、newして違うオブジェクトを作らないといけません。

Comments

[GFC]

やっぱり一番最後のボタンが参照されます・・・