javafxを用いたカウントダウンについて

Question

javafxの機能を用いて、ボタンを押したらカウントダウンが始まるというプログラムを作成しているのですが、ボタンを押すとlabelの文字が変化せずに秒数が経過してしまいます。どのようにプログラムを変更すればよいのでしょうか。

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

public class Test extends Application  {
    private Label label;

    @Override
    public void start(Stage stage) throws Exception {
	this.label = new Label();
	label.setText("Count Down");
	Button b = new Button("start");
	
	VBox vbox = new VBox();
	vbox.getChildren().addAll(b, label);

	b.setOnAction(event -> {
		for (int i = 3; i > 0; i--) {
		    label.setText(String.valueOf(i));
		    try {
			Thread.sleep(1000);
		    } catch (Exception e) {
		    }
		}
		label.setText("start!");
	    });

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

Comments

[退会済みユーザー]

うろ覚えだけど、JavaFXはmain関数と同じスレッドの画面アクセスと
同スレッド内での連続更新に制限（今回だとforループで1秒間隔更新処理）があって
別スレッドに分ける必要があったかと思います。
具体的には確か…別クラスとして画面を更新するコントローラクラスを置いて、さらにThreadクラスでタイマー管理をするのではなく、TimerTaskを使う感じで書いた覚えがあります。
これから仕事なんで夜にでもまた見に来ます。

Answer 1

単純に考えると別スレッドでの実行です。
コントロール類には触れないので、Platform.runLater()等がいります。

Timelineでやるのもわかりやすいかもしれません。

注意）連打した時の挙動が違います。

import javafx.animation.KeyFrame;
import javafx.animation.KeyValue;
import javafx.animation.Timeline;
import javafx.application.Application;
import javafx.application.Platform;
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.util.Duration;

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

    private Label label;
    private Timeline timeline;

    @Override
    public void start(Stage stage) {
        this.label = new Label();
        label.setText("Count Down");
        Button b = new Button("start");
        Button b2 = new Button("start");
        VBox vbox = new VBox();
        vbox.getChildren().addAll(b, b2, label);


        b.setOnAction(event -> new Thread(() -> {
            for (int i = 3; i > 0; i--) {
                final int ii = i;
                Platform.runLater(() -> label.setText(String.valueOf(ii)));
                try {
                    Thread.sleep(1000);
                } catch (Exception e) { }
            }
            Platform.runLater(() -> {
                label.setText("start!");
                // なんかやる
            });
        }).start());


        timeline = new Timeline(
                new KeyFrame(Duration.seconds(0), new KeyValue(label.textProperty(), "3")),
                new KeyFrame(Duration.seconds(1), new KeyValue(label.textProperty(), "2")),
                new KeyFrame(Duration.seconds(2), new KeyValue(label.textProperty(), "1")),
                new KeyFrame(Duration.seconds(3), new KeyValue(label.textProperty(), "start!")));
        timeline.setOnFinished(e -> {
            // なんかやる
        });
        b2.setOnAction(event -> timeline.play());

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

Comments

[submaru]

timelineの方を利用して実装しました！
ご回答ありがとうございました！

Answer 2

一応やりたいだろうことは出来ますね。
画面更新に対して色々面倒な制約がありますが、ここに書ききれるものでもないので参考リンクを張ります。

1. ポイント1. JavaFXメインスレッドで画面更新を行う

JavaFXにおけるスレッドの扱い - 軽Lab

メインスレッドを止めてしまうThread.sleep()により、メインスレッドが止まっている事、それから画面更新は処理を抜けてアイドル状態を作らないと起こらないらしい事。
つまりこの仕様のせいでラベルのテキストが更新されていなかったという事。
打開するには、空き時間にメインスレッドで遅延更新をするPlatform#runLater()を上手く使う事。

2. ポイント2. Threadの代わりにどうやってタイマー処理をするのか

ScheduledExecutorService 使い方メモ

まあ、色々あるんだけどScheduledExecutorService系のものを使うのが良いと思う。
実行タイミングなどが違ったりするので用途に応じて使い分ける感じですね。

3. ではどう直すか？

3.1. 問題点の整理

b.setOnAction(event -> {
    for (int i = 3; i > 0; i--) {
        label.setText(String.valueOf(i));
        try {
            Thread.sleep(1000);
        } catch (Exception e) {
        }
    }
    label.setText("start!");
});

label.setText(String.valueOf(i));
先述の通りメインスレッド処理中では更新できない。遅延更新させる必要がある。

Thread.sleep(1000);
こちらもメインスレッドを止めるだけなので使えない。ScheduledExecutorService等を使う必要がある。

for (int i = 3; i > 0; i--) { … }
前述2つの指摘により、そもそもforループは使えないので、発想を変える必要がある。
ScheduledExecutorServiceは実行する処理クラスにRunnableを使えるので、これをimplementsした実行クラスを作ればいい。（まあ、匿名インナークラスでもいいけど）

3.2. ScheduledExecutorServiceを使う場合

最初に断っておくが実現方法はこれ以外にもある。ここで書くつもりもないけど…。原稿料くれってなるし。
興味があれば他のScheduleServiceについても調べてみると良いと思う。

ScheduledExecutorService - JavaDoc11
Runnableインターフェースを使ったクラスを作って、それをどう実行するかというのがこのクラスの醍醐味。
一応、サボってこんな感じには書けるが、この場合は1秒間隔で延々とhelloと言い続けてしまう。
別途停止ボタンを置いて止める機構を作れればいいが今回の要件では使えない。
つまりカウントダウン終了時に自発的に止める工夫が必要となる。

ScheduledExecutorService service = Executors.newSingleThreadScheduledExecutor();
service.scheduleWithFixedDelay(() -> {
  System.out.println("hello!");
}, 1, 1, TimeUnit.SECONDS);

まあ、ぱっと浮かんだのはScheduledExecutorServiceのタスクを止めるには、ScheduledExecutorService#shutdown()を呼ぶ必要がある為、
サービスインスタンスを炊く制するタスクに持たせて、カウントダウン終了条件を満たしたらshutdown()を呼ぶようにすれば良いだろう。

そうすると呼び出し側はこんなイメージになって。

ScheduledExecutorService service = Executors.newSingleThreadScheduledExecutor();
service.scheduleWithFixedDelay(new MyTask(～), 1, 1, TimeUnit.SECONDS);

呼び出される側（ラベル更新を行う実行犯）はこんな感じになる。

class MyTask implements Runnable {
   // クラスメンバその1
   // クラスメンバその2
   // クラスメンバその3

   /**
    * ラベルを一定間隔でカウントダウン表示する為のタスク
    * @param service スケジュールサービス
    * @param count カウント開始値
    * @param label 更新するラベル
    */
    public MyTask(ScheduledExecutorService service, int count, Label label) {
        // クラスメンバの初期化処理を書く
    }

    @Override
    public void run() {
        // ラベルを更新する処理
        // カウントダウンする処理
        // 終了条件判定でshutdownさせるかどうかの処理
    }
}

以上です。

ちなみに、コードを書くためのヒントは出しますが
私の実装コードを現時点で出すつもりはありません。
悪しからず宜しくお願いします。