ボタンで行う加算演算の修正

Question

前提・実現したいこと

※先程の記載の説明が不足していたため、改めて補足致します。

下記のソースコードの通り、+= を用いて加算演算を行えるようにしたのですが、iが0の状態で、１回押しても数字が反映されず２回目で反映される形になります。

１回目で、反映させたいのですが、この場合の改善策を教えて頂くことをお願いすることは出来ますでしょうか？

よろしくお願いします。

発生している問題・エラーメッセージ

１回目で押すと、数値が反映されず、２回目で反映されます。

該当のソースコード

import java.io.IOException;

import javafx.application.Application;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.control.Label;
import javafx.scene.layout.BorderPane;
import javafx.scene.layout.HBox;
import javafx.stage.Stage;

public class aaa extends Application{
    private Label lb;
    private Button bt,bt1,bt2,bt3,bt4;
    private int i;
    public static void main(String[]args) {
    	launch(args);
    }
    public void start(Stage stage)throws IOException{
    	lb = new Label("数値;"+i);
    	bt = new Button("1up");
    	bt1 = new Button("2up");
    	bt2 = new Button("3up");
    	bt3 = new Button("4up");
    	bt4 = new Button("5up");
    	HBox hb = new HBox();
        hb.getChildren().add(bt);
        hb.getChildren().add(bt1);
        hb.getChildren().add(bt2);
        hb.getChildren().add(bt3);
        hb.getChildren().add(bt4);
    	BorderPane bp = new BorderPane();

    	bp.setCenter(lb);
    	bp.setBottom(hb);

    	Scene sc = new Scene(bp,300,300);
    	stage.setScene(sc);
    	stage.show();
        bt.setOnAction(ae -> {
        	lb.setText("数値;"+(i));
        	i += 1;
            });
        bt1.setOnAction(ae1 -> {
        	lb.setText("数値;"+(i));
        	i += 2;
            });
        bt2.setOnAction(ae2 -> {
        	lb.setText("数値;"+(i));
        	i += 3;
            });
        bt3.setOnAction(ae3 -> {
        	lb.setText("数値;"+(i));
        	i += 4;
            });
        bt4.setOnAction(ae4 -> {
        	lb.setText("数値;"+(i));
        	i += 5;
            });
    }
}

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

jdk14

Comments

[退会済みユーザー]

聞きたい事が良く分からないです。

ソース自体は各ボタンの押下イベントで、ボタン事に設定した加算値の増加量が+1なんだから、
どう頑張っても+1ずつしか増えない。
加算した後の値をiで維持していないのだから、常に【0 + 加算値】なんだけど、

回答者側はこの辺はどう整理すればいいですか？要件のお話ですね。

[退会済みユーザー]

具体的な話。ここをはっきりさせて欲しい。

初期状態から1upを押し続けたら、どう画面に出るのが正しいのか
初期状態から2upを押し続けたら、どう画面に出るのが正しいのか
1upを何回か押した後に、2upを押したらどう画面に出るのが正しいのか

[nakagen]

すみませんでした、上記のコードを改めて改正しますので、そちらを改めて見て頂けませんでしょうか･･･？

[nakagen]

初期状態から1upを押し続けたら、どう画面に出るのが正しいのか
→常に、iに１が加算されていく状態が正しいです
初期状態から2upを押し続けたら、どう画面に出るのが正しいのか
→常に、iに２が加算されていく状態が正しいです
1upを何回か押した後に、2upを押したらどう画面に出るのが正しいのか
→１×n+2=1n+2といった答えが欲しいです

Answer 1

勝手にこのように要件を解釈します。
初期値0として、1～5upボタン押下したら、初期値に加算する。

1up を押したら、1、2、3、…
2up を押したら、2、4、6、…
1up を2回、2up を2回押したら、1、2、4、6、…
みたいな動きをさせたいのだろうと予測する。

問題点1. 加算後の値を保持していない

値を保持したいのはiと予測するが、各ボタン押下イベント内でiに何かを代入している形跡はない

問題点2. 用途不明な増加量の加算をしている

各ボタンの増加値を収めたであろう変数。i1～i5になぜか+1している理由がよく分からない。
増加量なんだから、変えちゃダメだろう。

ちなみに++はインクリメント。
1しか増えないので、加算処理はi += （増加量）;みたいな書き方をしないとダメだろう。

上記問題点を解消すると、質問者さんの理想の形に近づくと思う。
頑張ってください。

修正ソースに対する指摘

１回押しても数字が反映されず２回目で反映される形になります。

画面への出力setText(～)前後でiには何が入っているでしょうか？
デバッグすれば分かると思います。

順番が重要です。そこをクリアするだけでこうなりますよ。
頑張ってください。

最後のコメントが怪しいのでたとえ話

金がないので日当1万円の仕事に友人と一緒にすることにしました。

仕事前日
友「なー、お前この仕事でいくらもらった？」
私「まだ仕事してないし、0円に決まってるじゃないか！」

※画面初期化時の例え

初日朝（1）
友「なー、お前この仕事でいくらもらった？」
私「ん、0円だけど」

その日の夕方に1万円もらって帰宅した。

※1up一回押下した時の例え

2日目朝
友「なー、お前この仕事でいくらもらった？」
私「ん、1万円だけど」

その日の夕方に1万円もらって帰宅した。合計2万円である。

※1upをもう一回押下した時の例え

3日目朝
友「なー、お前この仕事でいくらもらった？」
私「ん、2万円だけど」

その日の夕方に1万円もらって帰宅した。合計3万円である。

※1upを更にもう一回押下した時の例え

友人が聞きに来るのが朝じゃなくて日当を貰った夕方以降だったら、その日に稼いだ金額を加味した回答が出来るよね

Comments

[nakagen]

ご回答をありがとうございます！すみません、修正してしまったので、改めて読んで頂くことは出来ませんでしょうか･･･？よろしくお願いします。

[nakagen]

すみません、デバックとは何をすれば良いのか教えて頂くことは可能でしょうか･･･？重ね重ねすみません･･･

[退会済みユーザー]

回答修正しました。

[退会済みユーザー]

PCやスマホは調べものが得意なので自力で調べるようにした方が良いでしょう。
教えてクンと言われるのは心外でしょうから。その上で質問した方が心証が良いです。

https://www.google.com/search?q=java+%E3%83%87%E3%83%90%E3%83%83%E3%82%B0+%E6%96%B9%E6%B3%95&oq=java+%E3%83%87%E3%83%90%E3%83%83%E3%82%B0+%E6%96%B9%E6%B3%95&aqs=chrome..69i57.5577j0j7&sourceid=chrome&ie=UTF-8

開発環境に何を使っているのかが分からないので何とも言えないですが、
どんな環境でも出来るとしたら【System.out.println(i)】あたりでで変数【i】の中身を
画面出力前後で確認する事でしょう。

[nakagen]

今のところ、思い浮かぶのが「()」か「+」なのですが、そのいずれかを消すと言うことでしょうか･･･？

[退会済みユーザー]

だから、手元にソースがあるんですからやってみましょうよ。
で、想定する動きなら良し。ダメならその旨再質問してください。

[nakagen]

すみません、デバックモードを起動したのですが、特に変化が無かったのですが、この場合どのようなことが考えられますでしょうか･･･？
以下、System.outで試した結果です

0 //1up
1 //1up
2 //1up
3 // 1up
4 // 2up
6 // 2up
8 //2up
10 //2up

[退会済みユーザー]

デバッグモードが正しく扱えているかどうかは、そちらの開発環境が全く分からないので答えられません。
私はエスパーじゃないので。
正直デバッグモードについてまで書きたくないので、printlnを書いてラベルにsetTextする前後でiの値の変化を見てみて下さい。
気付けると思うんで。

[nakagen]

lb.setText("数値;"+(i += 1));
これで、なんとか加算演算ができました･･･
つまり、iが２つあったため、１回目が反映されなかったということでしょうか･･･？

[退会済みユーザー]

よーく考えて下さいね。
例えば最初の1up押下時に1回目は【0】が出るんですよ。つまり、【i】には【0】が入っている。
でも、nakagenさんが出したいのは【1】なんですよね？
では、【i】の値がどうなってたら思い通りの結果になりますか？

[退会済みユーザー]

解決した様で何よりです。

> つまり、iが２つあったため、１回目が反映されなかったということでしょうか･･･？

ちがいます。
単純に加算処理をする前に画面に出してたからです。

[nakagen]

最後まで、付き合って頂き本当にありがとうございました･･･

なるほどです･･･、最初にあったiが画面処理をしたのが原因でしたか･･･