selbstupdatendes JLabel?

[Basti91]

Hallo,
Ich möchte einen Counter mittels Thread implementieren.
Den Counter habe ich soweit geschrieben und wollte nun die Ausgabe durch ein JLabel tätigen.
Leider habe ich Probleme damit, dass es sich updatet bzw. es bleibt beim ersten Wert stehen.
Wie bekomme ich es hin, dass sich nur das Label periodisch updatet?

Hier der Counter-Thread:

public class Counter extends Thread {
	public int minutes=30;
	public int seconds=00;
	public String time =minutes+" : "+seconds;
	public JLabel test = new JLabel(time);
	
	public void run(){
		while((minutes+seconds)!=0){
			try{
				sleep(1000);
			}
			catch(Exception e){
				
			}
			if (seconds==0){ 
				seconds = 59;
				minutes --;
			}
			else{seconds --;}
			time =minutes+" : "+seconds;
			test = new JLabel(time);
		}
	}

 

[turtle]

Swing ist nicht multi-Thread fähig.

Dies bedeutet, dass Änderungen nur über den Event Dispatch Thread (EDT) durchgeführt werden dürfen. Daher zeichnet Dein Code während der Laufzeit das GUI nicht neu und zeigt "nur" den ersten Wert an.

Zur Lösung gibt es eine Fülle von Hinweisen im Netz, die alle darauf hinauslaufen, dass Du den EDT schnell wieder freigeben musst und länger dauerende Operationen in einen separaten Thread auslagern solltest. Mit der Klasse SwingWorker kannst Du einfach Dinge vor, während einer Background-Operation, oder danach das GUI ändern lassen.

Hier mal ein kleines Beispiel:

public class JLabelTimer {
	public static void main(String[] args) {
		JFrame frame = new JFrame("Label");
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		JPanel panel = new JPanel();
		final JLabel test = new JLabel("Time");
		final SimpleDateFormat sdf = new SimpleDateFormat("HH:mm:ss");
		SwingWorker<Void, Void> worker = new SwingWorker<Void, Void>() {
			public Void doInBackground() {
				while (true) {
					try {
						Thread.sleep(1000);
						String text = sdf.format(new Date());
						test.setText(text);
					} catch (InterruptedException e) {
						e.printStackTrace();
					}
				}
			}

			public void done() {
			}
		};
		worker.execute();
		panel.add(test);
		frame.add(panel);
		frame.pack();
		frame.setVisible(true);
	}
}

 

[c_sidi90]

edit

 

[tuttle64]

ungeachtet dessen was bereits geschrieben worden ist, finde ich den timer nicht gerade optimal umgesetzt. abgesehen davon, solltest Du die variable test nicht mit new, sondern mit test.setText(time) setzen. und es ist nicht ersichtlich, wo der thread gestartet wird.

 

[Basti91]

Hmm der SwingWorker erscheint mir sehr umständlich.
kann ich nicht in meinem Thread das JPanel zusammenbauen und der Main-GUI übergeben?

 

[hansmueller]

Hallo,

du solltest dein

test = new JLabel(time);

wie oben schon beschrieben mit

test.setText(text);

ersetzten und das ganze innerhalb eines invokeandWait bzw. invokeLater ablaufen lassen.

SwingUtilities.invokeLater(new Runnable()
{
	public void run()
	{
		test.setText(text);
	}
});

//bzw.

try
{
	SwingUtilities.invokeAndWait(new Runnable()
	{				
		@Override
		public void run()
		{
			test.setText(text);					
		}
	});
}
catch (Exception e)
{	
	e.printStackTrace();
}

Schau in der Api nach, was der Unterschied zwischen den beiden ist. Je nachdem wie dein Programm aufgebaut ist, mußt du dich für eines entscheiden.

MfG
hansmueller