Timer Konzept-Frage

[WIng2005]

Hallo werte Gemeinde,

ich stehe vor folgendem Problem:
Ich möchte einen Timer starten, der in einem eigenen Thread gewisse Aufgaben erfüllen soll. Diesen starte ich per Checkbox und möchte ihn über diese wieder deaktivieren. Er soll nach Deaktivieren jedoch genau noch 1x zu Ende laufen.

Der CheckBox habe ich einen ActionListener zugewiesen, der den Timer in einem neuen Thread startet. Dieser soll später periodisch per Socken Daten von einem Arduino holen und in eine Datenbank schreiben. Das Programm selbst merkt davon nichts, da es losgelöst in einem eigenen Thread läuft.

       checkBoxTimerActive.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {

                if (checkBoxTimerActive.isSelected()==true) {
                    Timer timer = new Timer();
                    timer.schedule(new MyTimerTask(), 0,20000);

                } else {
                    //How to stop?
                }
                
            }
        });

Der Timer selbst:

import java.util.TimerTask;

public class MyTimerTask  extends TimerTask {

    @Override
    public void run() {
        // do something

    }
}

Ich müsste doch eigentlich innerhalb des Timers prüfen, ob er von außen abgebrochen wurde. Wenn ja, einmal alles zu Ende bearbeiten und dann das Objekt löschen, oder? Oder wäre es hier besser, keinen Timer zu nehmen, sondern einen Thread und diesen dann zu pausieren?

VG
Steffen

 

[Jw456]

Halo etwa so

checkBoxTimerActive.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                Timer timer = new Timer();
                MyTimerTask myTimerTask = new MyTimerTask();
                if (checkBoxTimerActive.isSelected() == true) {
                    timer.schedule(myTimerTask, 0, 20000);

                } else {
                    timer.cancel();
                    timer.schedule( myTimerTask, 20000);
                    //How to stop?
                }

            }
        });

 

[Oneixee5]

per Socken 🤣

Warum nicht einfach:

        timer.cancel();
        timer.purge();

Der Task wird weiterlaufen bis er zu Ende ist und nicht wieder gestartet.

 

[Jw456]

Warum nicht einfach:

        timer.cancel();
        timer.purge();

Der Task wird weiterlaufen bis er zu Ende ist und nicht wieder gestartet.

Er will ihn ja nach erneuten Button Klick und deaktivier Checkbox noch einmal laufen lassen. Deshalb erst cancel und einmal starten mit einer Verzögerung seiner Wahl. Ja es könnten und werden wohl zwei werden. Kommt halt auf dem Zeitpunkt an.

 

[Oneixee5]

Er soll nach Deaktivieren jedoch genau noch 1x zu Ende laufen.

Ich denke da steht nicht das der Task noch einmal laufen soll, sondern das der Task bis zum Ende ausgeführt werden soll. Der Task verschwindet nicht einfach, wenn der Timer gecancelt wird. Der Task läuft weiter bis er endet.

 

[WIng2005]

Moin Moin zusammen,
Socken...jaja die Autokorrektur 😂 -> Socket war gemeint.
Also: Der Timer soll nur zu Ende laufen...sonst bleiben u.U. Sockets offen.
Aber: irgendwie ist ein Bock im Timer: Habe das die Nacht gegen 5 gestartet, 06:26 ist er abgestorben. Aber der Reihe nach:
Werden mit:

Timer timer = new Timer();
timer.schedule(new MyTimerTask(), 0,period);

..2 Objekte gebildet? Müssten BEIDE danach manuell gelöscht werden oder macht das der GC?

Den Timer kennt er im Else-Zeig nicht, ist dann folgendes "Russisch", weil der Timer immer erstellt und im Else gleich wieder gelöscht wird?

checkBoxTimerActive.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                long period= 1000*(Long.parseLong(spinnerTimerVal.getValue().toString()));
                Timer timer = new Timer();
                if (checkBoxTimerActive.isSelected()==true) {
                    radioBtnTimerOn.setSelected(true);
                    timer.schedule(new MyTimerTask(), 0,period);
                } else {
                    timer.cancel();
                    timer.purge();
                }

            }
        });

EDIT: das bewirkt nichts.... der Timer läuft auch nach Chancen / pure munter weiter


Vielen Dank für eure Hilfe!

VG
Steffen

 

[Robert Zenz]

Du willst vermutlich eine Referenz auf die eine Timer Instanz halten, im Moment erzeugst du bei jedem Klick einen neuen, da ist dann stoppen recht schwierig. Desweiteren koennte es schon sein dass der Garbage Collector die eine Timer Instanz einsammelt und vernichtet da du keine Referenz darauf haeltst, muesste ich aber nachschauen ob dem wirklich so ist oder ob es irgendetwas globales gibt dass den Timer am Leben haelt.

 

[WIng2005]

Was wäre denn, wenn der Timer "von innen" beendet wird

public boolean getCheckBoxTimer(){
        return checkBoxTimerActive.isSelected();
    }

.
.
.
    
checkBoxTimerActive.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                
                if (checkBoxTimerActive.isSelected()==true) {
                    long period= 1000*(Long.parseLong(spinnerTimerVal.getValue().toString()));
                    Timer timer = new Timer();
                    radioBtnTimerOn.setSelected(checkBoxTimerActive.isSelected());
                    timer.schedule(new MyTimerTask(), 0,period);
                }
                

            }
        });




//Am Ende vom Timer:

if (Main.mainFrame.getCheckBoxTimer()==false) {
            Main.mainFrame.setRadioTimerActive(false);
            this.cancel();
        }

 

[Robert Zenz]

Ich weisz jetzt gerade nicht was this.cancel() macht, aber Nein, eher nicht.

Du musst eine Instanz auf den Timer halten damit du spaeter diesen Timer auch beenden kannst.

 

[Jw456]

Erstelle den Timer als Instanzvariable. Nicht im Listener.

 

[WIng2005]

Hatte es zwischenzeitlich trotzdem mal probiert...geht aber. Ich habe nur keine Ahnung, ob mein Timer-Objekt danach im Orbit versauert.
Lasse mein Programm grad mal im Dauerlauf.... läuft, und das ohne, wie für mich üblich 😂, den RAM zu füllen...

Deine Variante würde mich aber trotzdem mal interessieren.... wie genau mache ich das? Den Timer ausserhalb der IF anlegen und prüfen, ob !=Null?

@Override
            public void actionPerformed(ActionEvent e) {
                Timer timer;
                if (checkBoxTimerActive.isSelected()==true) {
                    long period= 1000*(Long.parseLong(spinnerTimerVal.getValue().toString()));
                    timer = new Timer();
                    radioBtnTimerOn.setSelected(checkBoxTimerActive.isSelected());
                    timer.schedule(new MyTimerTask(), 0,period);
                } else {
                    radioBtnTimerOn.setSelected(false);
                    if (timer != null){
                        timer.cancel();    [B]//IDE meckert, dass Timer ggf. nicht initialisiert wurde[/B]
                        timer.purge();
                    }   
                }
                }

VG
Steffen

 

[WIng2005]

Erstelle den Timer als Instanzvariable. Nicht im Listener.

jetzt haben wir uns überschnitten. Du meinst das Timer direkt als "public Timer timer" im Kopf?

 

[Jw456]

jetzt haben wir uns überschnitten. Du meinst das Timer direkt als "public Timer timer" im Kopf?

JA nicht in dem ActionListener im Konstruktor machst du das new Timer

 

[WIng2005]

OK...habe beides mal versucht.... die Variante mit Erstellung im Listender funktioniert nicht (wie oben schon angemerkt). Die Variante als Instanzvariable und dann:

  public Timer timer;
  .
  .
  .
        checkBoxTimerActive.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                //Timer timer;
                if (checkBoxTimerActive.isSelected()==true) {
                    long period= 1000*(Long.parseLong(spinnerTimerVal.getValue().toString()));
                    timer = new Timer();
                    radioBtnTimerOn.setSelected(checkBoxTimerActive.isSelected());
                    timer.schedule(new MyTimerTask(), 0,period);
                } else {
                    radioBtnTimerOn.setSelected(false);
                    if (timer != null){
                        timer.cancel();
                        timer.purge();
                    }
                }
            }
        });

...läuft.

 

[Jw456]

timer = new Timer();

sollte nicht im listener sein. oder vorher prüfen.

 

[Oneixee5]

    private static class CheckBoxListener implements ActionListener {

        private final Timer timer = new Timer();
        private final SpinnerNumberModel spinnerModel;

        CheckBoxListener(final SpinnerNumberModel spinnerModel) {
            this.spinnerModel = spinnerModel;
        }

        @Override
        public void actionPerformed(final ActionEvent event) {
            final JCheckBox checkBox = (JCheckBox) event.getSource();
            if (checkBox.isSelected()) {
                final long period = 1000 * ((Number) this.spinnerModel.getValue()).longValue();
                this.timer.schedule(new MyTimerTask(), 0, period);
            } else {
                this.timer.cancel();
            }
        }

    }

    ...
      
        final SpinnerNumberModel spinnerModel = new SpinnerNumberModel(...);
        final JSpinner spinnerTimerVal = new JSpinner(spinnerModel);
      
        final JRadioButton radioBtnTimerOn = new JRadioButton(...);

        final JCheckBox checkBoxTimerActive = new JCheckBox(...);
        checkBoxTimerActive.addActionListener(new CheckBoxListener(spinnerModel));
        checkBoxTimerActive.addActionListener(
                event -> radioBtnTimerOn.setSelected(checkBoxTimerActive.isSelected()));

    ...

 

[WIng2005]

uff.....davon verstehe ich die Hälfte nicht....(-> ich sollte dringend meinen Udemy-Kurs weitermachen) aber danke für deine Mühe!!!

Die Variante von oben funktioniert erstmal....vllt. nicht ganz so sauber programmiert. Für einen ersten Wurf und im Rahmen meiner Fähigkeiten bin ich ganz zufrieden. Für den Rest muss ich mir dringend mehr Wissen aneignen

Ich denke, wir können dann hier zumachen. Danke an alle !!
VG
Steffen