Odliczanie w JavieFX - timer

pytanie zadane 5 sierpnia 2018 w Java przez periedynek Obywatel (1,320 p.)

Cześć. Chciałem stworzyć prosty timer w JavieFX.
Nie wiem czy mój sposób jest dobry.
Tworze sobie formatkę z przyciskiem START/STOP, oraz 3 Labelami: 1) Minuty 2) ":" 3) Sekundy, jakby połączyć te 3 labele wychodzi np: 3:34.

W wyniku czego mogę łatwo pobrać liczbe minut i sekund i przekonwertować to na sekundy. Wyświetlanie za pomocą np. minuty:

labelOfMinutes.setText(amountOfMinutes)

sekundy:

labelOfSeconds.setText(amountOfSeconds)

gdzie int amountOfMinutes = time/60;, amountOfSeconds = time%60.

Nie potrafię wymyślić innego dobrego pomysłu jak rozwiązać to odliczanie w dół.

Może ma ktoś inny pomysł?

1 odpowiedź

odpowiedź 5 sierpnia 2018 przez periedynek Obywatel (1,320 p.)

O czymś takim mówiłem:

 @FXML
    void startTimer() throws InterruptedException {

        int amountOfMinutes = Integer.parseInt(displayMinutes.getText());
        int amountOfSeconds = Integer.parseInt(displaySeconds.getText());

        int time = amountOfMinutes * 60 + amountOfSeconds;
        System.out.println(time);
        for (int i = 0; i < time; time--) {
            displayMinutes.setText(String.valueOf(amountOfMinutes / 60));
            displaySeconds.setText(String.valueOf(amountOfSeconds % 60));
            Thread.sleep(1000);
            displaySeconds.setText("");
            displayMinutes.setText("");
        }
    }

Program się wywala w pętli, nie doszedłem jeszcze do tego dlaczego.

Pokaż 6 poprzednich komentarzy

komentarz 6 sierpnia 2018 przez mbabane Szeryf (79,280 p.)

Można zdaje się uprościć dodawanie klatek do jednej operacji, oraz lepiej zmień nazwy displayMinutes i displaySeconds ponieważ nie jest dobrze jeśli zmienne zawierają czasownik. Czasownik stosuje się do metod, ponieważ one zasadniczo coś wykonują, natomiast zmienna jest tylko pewnym narzędziem. Co do zmiennej time (tej mojej) po prostu zmień nazwę jeśli Ci nie odpowiada - nikt się nie pogniewa ;). I zdaje się, że linie:

displaySeconds.setText("");
displayMinutes.setText("");

nie są potrzebne.

Całość po ewentualnych zmianach:

    public void showCountDown(int time) {
        Timeline timeline = new Timeline();

        int nextFrameTime = 0;

        for (int i = 0; i < time; time--) {
            int finalTime = time;

            timeline.getKeyFrames().add(new KeyFrame(
                            Duration.millis(nextFrameTime),
                            action -> 
                            {
                                minutesLabel.setText(String.valueOf(finalTime / 60));
                                secondsLabel.setText(String.valueOf(finalTime % 60));
                            }
                    )
            );
            nextFrameTime += 1000;
        }
        timeline.setCycleCount(1);
        timeline.play();
    }

komentarz 6 sierpnia 2018 przez periedynek Obywatel (1,320 p.)

Okej, załapałem wszystko.

Dzięki wielkie. Szkoda, ze pisałeś w kometarzach. Nie mam możliwosci dać najlepszej odpowiedzi.

komentarz 7 sierpnia 2018 przez periedynek Obywatel (1,320 p.)

Ostatnie pytanie, jak mogę zapobiec temu, że jak ktoś wciśnie "START", to jeżeli wciśnie ponownie start to nic się nie zrobi, tylko czas będzie lecieć dalej?

Chciałem to zrobić za pomocą warunku:

if (minutesLabel.getText().equals("25") && secondsLabel.getText().equals("00")) ||(minutesLabel.getText().equals("00") && secondsLabel.getText().equals("00"))

to wtedy się odpala. Nie wziąłem jednak pod uwagę tego, że jeżeli ktos wciśnie stop, czyli:

timeline.stop();

to gdy ponownie wciśnie start, aplikacja nie ruszy z odliczaniem ze względu na warunek powyżej.

Pytam przez to, ponieważ jeżeli klikne start, zacznie się odliczanie, klikne znów i jeszcze raz, to czas odliczania spada o jakies tam sekundy.

komentarz 7 sierpnia 2018 przez mbabane Szeryf (79,280 p.)
edycja 7 sierpnia 2018 przez mbabane

Można to zrobić np. wyłączając przycisk przed rozpoczęciem odliczania, a jako ostatnią klatkę w timeline ustawić dodatkowo, żeby przycisk się włączał - ewentualnie jakaś flaga i całą metodę objąć ifem (przy czym też chyba trzeba będzie ustawiać jako ostatnią klatkę zmianę flagi).

Istnieje jeszcze taka możliwość, chyba najbardziej czytelna:



button.setDisable(true);

//petla for z ustawianiem klatek


timeline.setOnFinish( action -> button.setDisable(false) );
//powyzsza linia musi być po petli for

timeline.setCycleCount(1);
timeline.play();