Timer und Date

[loony]

Hallo Leute,

ich fange gerade an mich mit Java und SWT unter Linux zu beschäftigen und habe da natürlich auch gleich ein kleines Problem.

Und zwar will ich eine Grafik in einem Label alle X Minuten aktualisieren.
Zum Testen des Timers wollte ich dazu erstmal alle 10 Sekunden die aktuelle Zeit in dem Label darstellen.

Allerdings bekomme ich immer folgenden Fehler beim Aufruf des Timers:

Exception in thread "Timer-0" org.eclipse.swt.SWTException: Invalid thread access
at org.eclipse.swt.SWT.error(Unknown Source)
at org.eclipse.swt.SWT.error(Unknown Source)
at org.eclipse.swt.SWT.error(Unknown Source)
at org.eclipse.swt.widgets.Widget.error(Unknown Source)
at org.eclipse.swt.widgets.Widget.checkWidget(Unknown Source)
at org.eclipse.swt.widgets.Label.setText(Unknown Source)
at WetterSpy$aktTask.run(WetterSpy.java:61)
at java.util.TimerThread.mainLoop(Timer.java:534)
at java.util.TimerThread.run(Timer.java:484)

Hier mein Quelltext:

import org.eclipse.swt.SWT;
import org.eclipse.swt.widgets.Display;
import org.eclipse.swt.widgets.Label;
import org.eclipse.swt.widgets.Shell;
import java.util.*;


public class WetterSpy {

	protected Shell shell;
	public Timer timer = new Timer();
	public Label lblWarnImage;

	/**
	 * Launch the application
	 * @param args
	 */
	public static void main(String[] args) {
		try {
			WetterSpy window = new WetterSpy();
			window.open();
		} catch (Exception e) {
			e.printStackTrace();
		}
	}

	/**
	 * Open the window
	 */
	public void open() {
		final Display display = Display.getDefault();
		createContents();
		shell.open();
		shell.layout();
		while (!shell.isDisposed()) {
			if (!display.readAndDispatch())
				display.sleep();
		}
	}

	/**
	 * Create contents of the window
	 */
	protected void createContents() {
		shell = new Shell();
		shell.setSize(874, 375);
		shell.setText("WetterSpy");

		lblWarnImage = new Label(shell, SWT.BORDER);
		lblWarnImage.setBounds(86, 53, 713, 63);

	    timer.schedule( new aktTask(), 1000); 
	}
	
	class aktTask extends TimerTask {
	    public void run() {
	      System.out.println("Aktualisiere Daten");

	      Date dat = new Date();
	      String uhrzeit = dat.getHours()+":"+dat.getMinutes()+":"+dat.getSeconds();
	      lblWarnImage.setText(uhrzeit);
	      
	      timer.schedule(new aktTask(), 10000);
	    }
	  }

}

Und im Quelltext in Eclipse streicht mir der Editor getHours, getMinutes und getSeconds durch.

Kann mir dabei jemand helfen?

Liebe Grüße,
loony

 

[Guest]

Allerdings bekomme ich immer folgenden Fehler beim Aufruf des Timers:

Exception in thread "Timer-0" org.eclipse.swt.SWTException: Invalid thread access
...

Änderungen an GUI müssen im GUI-Thread erfolgen, daher der Fehler. Lösung, siehe unten.

Und im Quelltext in Eclipse streicht mir der Editor getHours, getMinutes und getSeconds durch.

Die Methoden sind deprecated (veraltet; nicht mehr gültig) und sollten nicht mehr verwendet werden.
verwende einen Formatter (SimpleDateFormat) oder String.format(...). Lösung, siehe unten.

Hier eine korrigierte Version

protected void createContents() {
   shell = new Shell();
   shell.setSize(874, 375);
   shell.setText("WetterSpy");
   lblWarnImage = new Label(shell, SWT.BORDER);
   lblWarnImage.setBounds(86, 53, 713, 63);

   Calendar c = Calendar.getInstance(Locale.GERMANY);
   c.set(Calendar.SECOND, 10 + c.get(Calendar.SECOND)/10); // Zeit synchronisieren (light ;))
   timer.schedule( new aktTask(), c.getTime(), 10000L); // Alle 10 Sekunden aufrufen
}

class aktTask extends TimerTask {
   private Runnable updateAction;

   public void run() {
      // Aktualisierung immer im Thread der GUI ausführen
      Display.getDefault().asyncExec(getUpdateAction());
   }

   private Runnable getUpdateAction() {
      if(updateAction == null) {
         updateAction = new Runnable() {
            private final Format formatter = new SimpleDateFormat("HH:mm:ss");
            public void run() {
               lblWarnImage.setText(formatter.format(new Date()));
               // oder einfach lblWarnImage.setText(String.format("%tT", new Date()));
            }
         };
      }
      return updateAction;
   }
}

 

[loony]

Vielen Dank für die Info!
Funktioniert super.

Liebe Grüße,
loony