Swing Timer Probleme

[hoernchen]

Hallo Leute,

Ich brauche bei einem Programm von mir einen Start Button der einen Timer aktiviert, allerdings bekomme ich ein Paar fehlermeldungen.

 private void startButtonActionPerformed(java.awt.event.ActionEvent evt) {
       
       startButton.setEnabled( false );

  final long start = System.currentTimeMillis();
  final long end   = start + minutesSlider.getValue() * 60 * 1000;

  final Timer timer = new Timer( 100, null );
  timer.addActionListener( new ActionListener() {
    public void actionPerformed( ActionEvent e ) {
      long now = System.currentTimeMillis();
      if ( now >= end )
      {
        remainingMinLabel.setText( "" );
        startButton.setEnabled( true );
        JOptionPane.showMessageDialog( null, "BING!" );
        timer.stop();
      }
      else
        remainingMinLabel.setText( (end – now) / 1000 + " Sekunden" );
    }
  } );
  timer.start;
 
        
    }

Wenn jemand eine Idee hat wäre ich sehr dankbar
Grüße
Michael

 

[Murray]

Ich brauche bei einem Programm von mir einen Start Button der einen Timer aktiviert, allerdings bekomme ich ein Paar fehlermeldungen.

Verrätst Du uns auch, was da für Fehlermeldungen kommen?

 

[tuttle64]

einen solchen Konstruktor

new Timer( 100, null );

gibt es für den Timer in package java.util nicht, ebenso die Methode timer.start nicht. Und last-but-not-least: Die Berechnung (end-now) enthält kein Minuszeichen, sondern ein Zeichen, welches so aussieht wie ein Minuszeichen. Man teste:

		int c1 = '–' + 0;
		int c2 = '-' + 0;
		System.out.println("Wert von c1 " + c1); // pseudo Minuszeichen mit Wert: 8211
		System.out.println("Wert von c2 " + c2); // echtes Minuszeichen mit Wert: 45

 

[eRaaaa]

Und wer sagt dir dass er nicht den Swing-Timer meint (schließlich befinden wir uns ja im Swing-UForum)? Allerdings muss dann beim Methodenaufruf die Klammern hinzugefügt werden --> timer.start()
Die Frage wäre dann aber, wieso du erst null übergibst und danach den Listener addest? Wieso nicht schon beim Konstruktor ?

 

[André Uhres]

Dass man im Konstruktor keinen Listener übergibt, ist ja kein Problem und kann auch hier zu keiner Fehlermeldung führen. Dagegen finde ich es stilistisch schöner, wenn wir die anonyme Klasse als Member konvertieren:

        final Timer timer = new Timer(100, null);
        timer.addActionListener(new TimerAction(end, timer));
        timer.start();
...

    private class TimerAction implements ActionListener {

        private final long end;
        private final Timer timer;

        public TimerAction(final long end, final Timer timer) {
            this.end = end;
            this.timer = timer;
        }

        public void actionPerformed(final ActionEvent e) {
            long now = System.currentTimeMillis();
            if (now >= end) {
                remainingMinLabel.setText("");
                startButton.setEnabled(true);
                JOptionPane.showMessageDialog(null, "BING!");
                timer.stop();
            } else {
                remainingMinLabel.setText((end - now) / 1000 + " Sekunden");
            }
        }
    }
}

Gruß,
André

 

[hoernchen]

Danke André dei Code hat jetzt fehlerfrei funktioniert