Probleme mit einem Timer

[dweiner]

Hi,

ich habe eine Netzwerkanwendung bei mir laufen, in der ich einen Timer programmiert habe (also den Timer hab ich hier als Codeschnipsel im Forum gefunden). Das Programm läuft so ab, dass der Server an die verschiedenen Clients eine Nachricht schickt - haben die Clients die Nachricht erhalten, so startet der Timer. Der Code des Timers sieht folgendermaßen aus:

		count = 60; 
		countdown = count;
		timer1 = new Timer(1000, new ActionListener(){
			public void actionPerformed(final ActionEvent e){
			
				
			}
		});
		
		  timer1.setInitialDelay(0); 
	        timer2 = new Timer(count*1000, new ActionListener() { 
	            public void actionPerformed(final ActionEvent e) {

	            	 //sobald actionPerformed hier aufgerufen wird, erfolgen weitere Ereignisse

	            	
	                timer1.stop();
	                timer2.stop();
	            } 
	        });

timer1.start();
timer2.start();

Das Problem ist, dass der Timer auf verschiedenen Client-Rechnern unterschiedlich schnell herunterläuft (Countdown)!!! Wie gesagt, die Clients bekommen gleichzeitig das Startsignal, der Timer beginnt daraufhin von 60 herunterzuzählen (im Sekundentakt); aber irgendwann ist der eine Client schon bei 40 Sekunden während der andere schon bei 35 Sekunden ist!!!
Der Timer ist in eine relativ komplexe GUI eingebettet, auf der nebenbei Aktionen ausgeführt werden (z.B. werden Werte in Textfeldern angepasst etc.). Manchmal kommt es auch vor, dass der Timer dann ganz zum Stillstand kommt (was dann besonders ärgerlich ist weil dann meine gesamte Netzwerkkommunikation betroffen ist).
Irgendwie muss an der Implementierung des Timers was faul sein - kann man den besser implementieren (z.B. soll er besser in einen Thread gepackt werden, weil er ja im Prinzip nebenher laufen soll - parallel zu den Tätigkeiten in der GUI)?

Bitte um Hilfe - ist ziemlich dringend!

Danke schon mal!

Gruß
Dominik

 

[Murray]

Interessant wäre der Code des Timers (oder der Link auf den Thread, aus dem Du ihn kopiert hast)

 

[dweiner]

Also den kompletten Code des Timers habe ich hier:

package thread; 
/* 
* SwingTimerDemo.java 
*/ 
import java.awt.*; 
import java.awt.event.*; 
import javax.swing.*; 
public class SwingTimerDemo extends JFrame { 
    private Timer timer1, timer2; 
    private MyPanel panel; 
    private int count, countDown, countTimes; 
    private JButton btCancel, btStart; 
    public SwingTimerDemo() { 
        super("SwingTimerDemo"); 
        setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE); 
        setSize(250,150); 
        setLocationRelativeTo(null); 
        panel = new MyPanel(); 
        add(panel); 
        count = 9; 
        countDown = count; 
        timer1 = new Timer(1000, new ActionListener() { 
            public void actionPerformed(final ActionEvent e) { 
                panel.getLabel1().setText("Countdown: " + (countDown--) + " seconds "); 
            } 
        }); 
        timer1.setInitialDelay(0); 
        timer2 = new Timer(count*1000, new ActionListener() { 
            public void actionPerformed(final ActionEvent e) { 
                countDown = count; 
                panel.getLabel2().setText(" Count: " + (++countTimes) + " times"); 
                timer1.restart(); 
            } 
        }); 
        timer1.start(); 
        timer2.start(); 
        btCancel = new JButton("Cancel"); 
        btCancel.addActionListener(new ActionListener() { 
            public void actionPerformed(ActionEvent e) { 
                timer1.stop(); 
                timer2.stop(); 
            } 
        }); 
        add(btCancel, BorderLayout.NORTH); 
        btStart = new JButton("Start"); 
        btStart.addActionListener(new ActionListener() { 
            public void actionPerformed(ActionEvent e) { 
                countDown = count; 
                timer1.start(); 
                timer2.start(); 
            } 
        }); 
        add(btStart, BorderLayout.SOUTH); 
    } 
    public static void main(final String args[]) { 
        SwingUtilities.invokeLater(new Runnable() { 
            public void run() { 
                new SwingTimerDemo().setVisible(true); 
            } 
        }); 
    } 
} 
class MyPanel extends JPanel{ 
    private JLabel label1, label2; 
    public MyPanel(){ 
        super(new BorderLayout()); 
        label1 = new JLabel(); 
        label2 = new JLabel(); 
        label2.setForeground(Color.RED); 
        add(label1, BorderLayout.NORTH); 
        add(label2, BorderLayout.SOUTH); 
    } 
    public JLabel getLabel1() {return label1;} 
    public JLabel getLabel2() {return label2;} 
}

Mein Timer ist wie gesagt daraus übernommen (siehe oben), wobei ich meinen nicht in einen Thread gepackt habe (also "new Runnable" etc. weggelassen).

 

[Murray]

OK, es geht also um einen javax.swing.Timer - ich dachte, das wäre eine eigene (bzw. irgendwoher kopierte) Klasse.

 

[dweiner]

Kann mein Problem denn tatsächlich darin bestehen, dass ich den Timer nicht in einen Thread gepackt habe? Wie gesagt, er ist in eine GUI eingebettet, auf der jede Menge Aktionen ausgeführt werden. Nebenbei soll der Timer laufen. Und in den meisten Fällen hängt er sich dann irgendwann auf oder er funktioniert auf verschiedenen Rechnern nicht synchron....

 

[LordLuzifer]

Der Swing-Timer funktioniert eben wegen der GUI-Abhängigkeit nicht wirklich toll; teilweise sind die Unterschiede so gravierend, dass man sie mit einer Uhr nachmessen kann...
Probier mal java.util.Timer aus, der startet einen eigenen Thread und sollte daher relativ exakt sein.

 

[dweiner]

Ok, danke dir - das ist ein guter Punkt wenn das so stimmt was du sagst. Gibt es da irgendwas besonderes zu beachten wenn man den util.Timer in eine GUI einbettet oder funktioniert das von der Implementierung genauso wie beim SwingTimer. Ist es auch möglich den Swing-Timer in einem Thread laufen zu lassen, also würde dies evtl. auch zu Verbesserungen führen?

Gruß
Dominik