javafx メソッドの引数としてのstage

Question

ある参考書のコードです。

import javafx.application.Application;
import javafx.geometry.Insets;
import javafx.geometry.Pos;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.control.Label;
import javafx.scene.control.TextField;
import javafx.scene.layout.HBox;
import javafx.scene.layout.VBox;
import javafx.stage.Stage;

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

    TextField textFieldH = new TextField();
    TextField textFieldW = new TextField();
    Label lblBMI = new Label("BMI");

    @Override
    public void start(Stage stage) throws Exception {

        stage.setTitle("BMI");
        stage.setWidth(200);
        stage.setHeight(155);

        Label lblH = new Label("　身長(cm)：");
        textFieldH.setPrefWidth(80);

        HBox boxH = new HBox();
        boxH.getChildren().addAll(lblH, textFieldH);


        Label lblW = new Label("　体重(kg)：");
        textFieldW.setPrefWidth(80);

        HBox boxW = new HBox();
        boxW.getChildren().addAll(lblW, textFieldW);


        Button btnGo = new Button("計算");
        btnGo.setOnMouseClicked(event -> updateValue(stage));

        VBox root = new VBox();
        root.setAlignment(Pos.TOP_CENTER);
        root.setPadding(new Insets(5, 5, 5, 5));
        root.setSpacing(5.0);
        root.getChildren().addAll(boxH, boxW, btnGo, lblBMI);

        stage.setScene(new Scene(root));
        stage.show();
    }

    void updateValue(Stage stage)
    {
        String sh = textFieldH.getText();
        String sw = textFieldW.getText();
        double h = Double.parseDouble( sh.trim() );
        double w = Double.parseDouble( sw.trim() );
        double bmi = 10000.0 * w / (h * h);
        lblBMI.setText(String.format("BMI=%5.2f", bmi));
    }
}

上記が全体のコードなのですが

Button btnGo = new Button("計算");
　　　　btnGo.setOnMouseClicked(event -> updateValue(stage));

と

void updateValue(Stage stage)
   {
       String sh = textFieldH.getText();
       String sw = textFieldW.getText();
       double h = Double.parseDouble( sh.trim() );
       double w = Double.parseDouble( sw.trim() );
       double bmi = 10000.0 * w / (h * h);
       lblBMI.setText(String.format("BMI=%5.2f", bmi));

のコードでstageが引数として使われている理由がわかりません。
一応、

Button btnGo = new Button("計算");
　　　　btnGo.setOnMouseClicked(event -> updateValue());

void updateValue()
   {
       String sh = textFieldH.getText();
       String sw = textFieldW.getText();
       double h = Double.parseDouble( sh.trim() );
       double w = Double.parseDouble( sw.trim() );
       double bmi = 10000.0 * w / (h * h);
       lblBMI.setText(String.format("BMI=%5.2f", bmi));

としてコンパイル、実行してみたのですが、問題なく意図した動きをしてくれました。
なぜstageが引数として使われていたのか、その意図は何かがわかりませんでした。

Answer 1

あなたの察しの通り、そのコードでstageを引数に持つことは**「なんの意味もない」**です。

なぜその引数があるかというのは、その参考書を編集した人にしか意図はわからないですが、予想で書きます。

• もともとはstageを使った処理を書いていたが、編集段階で削除され、引数だけ消し忘れた
• 後々のページでstageを使った処理を書くために、最初から書いてある

の2つが考えられます。

Comments

[vetom]

やっぱりそうですよね。
このコードはこれで完成形みたいなので後々にこれを発展させていくみたいなものはありませんでした。
細かいところですが、参考書なのですべてに意図があると思い深く悩んでいました。
あってもいいけどあえてなぜ書くのかと。
分かりやすく教えていただきありがとうございます。

[masaya_ohashi]

もし気になるならその参考書の出版に誤植の指摘として報告してあげると次の版で直されるかもしれません。合理的な理由は見当たらないので、おそらくただの誤植でしょう。

Answer 2

解決済みへのコメントで恐縮ですが・・・

自分もmasaya_ohashiさんコメントと同様の印象を持ちましたが、参考書の他の部分と関連がない単独の完成コードだとして、著者があえてこのようなインターフェースにした（そうなのかどうかは分かりませんが）と仮定してその理由を考えてみました。

JavaFXの参考書の対象読者はJavaFXだけでなくJavaの学習中の人も含まれるだろうと思います。そうした読者がサンプルをベースに「ちょっと違うものを作ってみる」ということはよくあるだろうと思います。

そういう場面ではこのコードは「何かのコントロールを操作した際に、画面を更新するための一般形」として利用できます。

読者がコードを変形する際に、「サンプルとは違うコントロール（あるいはノード）」を追加する際には、サンプルに書かれているようにaaaaクラスにコントロール用のフィールドを追記するというやりかたが簡単に想像つくでしょう。

一方、「画面を動かす」などのことをやろうとするとき、Stageのインスタンスが必要になりますね。Java言語の理解が一定以上の読者なら「startメソッドの引数に初期画面のStageが渡されているのだから、ハンドラーのラムダ式の本体で引数のstageを参照できる＝＞ハンドラーメソッドの引数としてわたせばいい」ということは容易に考えられますが、Java自体の初学者の場合は「どうやってupdateValueの中でStageを求めたらいいのだろう」なんて迷うかも知れません。

自分は「そういう読者に向けてのちょっとした配慮」なのかも知れないと想像しました。

(尤も、これは考えすぎかも知れません。こういう配慮を意図しているなら参考書本文にそれらしい補足をするでしょうし。)

Comments

[vetom]

KSwordOfHaste先生
いつもありがとうございます。
確かにこのサンプルベースに違うものを作ったときに引数stageは必要でしたが、そのためにあるとまでは想像つきませんでした。

>>一方、「画面を動かす」などのことをやろうとするとき、Stageのインスタンスが必要になりますね。Java言語の理解が一定以上の読者なら「startメソッドの引数に初期画面のStageが渡されているのだから、ハンドラーのラムダ式の本体で引数のstageを参照できる＝＞ハンドラーメソッドの引数としてわたせばいい」ということは容易に考えられますが、Java自体の初学者の場合は「どうやってupdateValueの中でStageを求めたらいいのだろう」なんて迷うかも知れません。>>
この部分は正直完璧に理解できませんが、ある程度勉強が追い付いたらまた見てみます。

[vetom]

ちなみに前回の質問で体系的に勉強したほうがいいとアドバイスしてくださったので、pc-brainという会社のjavaを受けてみることにしました。勉強するなら書きながらすぐ上級者に添削してもらった方が近道というのネットで見かけたので教材＋いつでも質問okのこの教材を選びました。
本を買って、分からないことをこのようなサイトで聞けばいいのかとも思いましたが、何となくこのようなサイトの意図される使い方ではないと思ったので（レベルが低すぎて）。
それでも自作のコードなど、いろんな意見を聞きたいときはまた質問させてください。

[KSwordOfHaste]

漠然とした質問は本サイトでは歓迎されませんが、「自分なりに調べる」「疑問点が具体的で、かつ何を知りたいかが明記されている」といったものなら歓迎されると思います。質問の内容のレベルが高いか低いかよりも前述のようなことが肝心だろうと思います。という意味では本件は悪くない質問と思います。

ちなみに「～先生」は勘弁してください。自分だって本サイトで勉強させてもらっている一人であって、ただの一アマチュアに過ぎません。普通に「～さん」でお願いします。

[vetom]

配慮なくすみません。気分を害されていたら申し訳ないです。
教えてもらっている人に、先生を呼ぶのが自分の中で当たり前だったもので。
また宜しくお願いします。

[KSwordOfHaste]

気分を害したわけではないですのでご心配なく。わかっていただければＯＫです。

[masaya_ohashi]

余談ですが、本当は引数でstageを渡して操作するのが設計として美しいとはわかっていても、私はついつい((Stage)node.getScene().getWindow())を使ってしまう駄目なやつです…

[KSwordOfHaste]

うーん・・・実のところ、JavaFXのより一般的なクラス構成(普通のInitializableなコントローラークラスがある)であれば、自分もmasaya_ohashiさんのおっしゃる手段がベターと思います。コントローラークラスのメソッド引数にビューのウィンドウ(=Stage)があるのは「簡単に求まるものをわざわざ渡している」感がすごくしますよね。

そう考えればStageをstartメソッド以外で積極的に引数として渡す機会は少なそうなので、自分の回答のような意図を著者さんが想定している可能性は薄そうですね・・・