Timer.scheduleAtFixedRate().

[Campino]

hi,

ich habe eine Uhr programmiert, die regelmäßig ihre anzeige an die Systemzeit anpasst. Dabei startet ein Timer einen TimerTask, der die Systemzeit ausließt und anzeigt. Das ganze braucht 97% meiner CPU-Leistung, wo seht ihr die Performance-Killer? Haltet ihr es für sinnvoller, die Zeit einmal auszulesen und dann bei jedem TimerTask-Aufruf eine Variable hochzuzählen? Oder wie würdet ihr das machen?

 

[Illuvatar]

Wie oft rufst du den TimerTask denn auf? wenn das so schnell wie möglich hintereinander geschiet, ist das ja klar, dann ist das ja kaum was anderes als ne while(true) Schleife.

Mach lieber nen Thread (oder auch TimerTask), der alle sagen wir 25 millisekunden repaint() aufruft. Und die Zeit ausllesen machst du am besten dann in der paint-Methode, weil dann wird die Zeit genau so oft ausgelesen, wie das auch benötigt wird.

 

[Sky]

die regelmäßig ihre anzeige an die Systemzeit anpasst.

Wie oft wird angepaßt?

Dabei startet ein Timer einen TimerTask, der die Systemzeit ausließt und anzeigt.

Wie und wo wird angezeigt?

Mach lieber nen Thread (oder auch TimerTask), der alle sagen wir 25 millisekunden repaint() aufruft.

TimerTask ist m.E. besser, weil der GUI-Thread dann nicht blockiert ist. TimerTask ist außerdem schon vorhanden. Alle 25 ms lesen ist zu oft, für eine Uhr, die sekundengenau geht. @Campino: Geht doch sekundengenau, oder ?

Und die Zeit ausllesen machst du am besten dann in der paint-Methode, weil dann wird die Zeit genau so oft ausgelesen, wie das auch benötigt wird.

Falsch. Dann wird die Zeit ausgelesen, wenn neu gemalt wird, was durchaus auch mehr als sagen wir einmal in der Sekunde sein kann.
Man könnte evtl. einen ComponentListener verwenden, um den Timer anzuhalten/zu starten, wenn die Uhr nicht sichtbar ist/aufgeschaltet wird.

 

[Illuvatar]

Dabei startet ein Timer einen TimerTask, der die Systemzeit ausließt und anzeigt.

Wie und wo wird angezeigt?

Mach lieber nen Thread (oder auch TimerTask), der alle sagen wir 25 millisekunden repaint() aufruft.

TimerTask ist m.E. besser, weil der GUI-Thread dann nicht blockiert ist. TimerTask ist außerdem schon vorhanden. Alle 25 ms lesen ist zu oft, für eine Uhr, die sekundengenau geht. @Campino: Geht doch sekundengenau, oder ?

Der wird doch net blockiert :? Aber gut, man könnte nen javax.swing.Timer nehmen, der is sogar explizit mit dem GUI-Thread synchronisiert.

Und die Zeit ausllesen machst du am besten dann in der paint-Methode, weil dann wird die Zeit genau so oft ausgelesen, wie das auch benötigt wird.

Falsch. Dann wird die Zeit ausgelesen, wenn neu gemalt wird, was durchaus auch mehr als sagen wir einmal in der Sekunde sein kann.
Man könnte evtl. einen ComponentListener verwenden, um den Timer anzuhalten/zu starten, wenn die Uhr nicht sichtbar ist/aufgeschaltet wird.

Naja und wie willst dus machen? (Wenn der Timer läuft) Einmal pro Sekunde abfragen? Aber das kann ja dann immer zu spät kommen und is somit ungenau.
Ich meinte: immer wenn gezeichnet wird muss die aktuelle Zeit dasein. Dann ist es so genau wie möglich und dabei wird nur so viel CPU wie nötig verbraucht.

 

[Sky]

Naja und wie willst dus machen? (Wenn der Timer läuft) Einmal pro Sekunde abfragen?

Einmal pro Sekunde ist natürlich zu wenig. 100,200,500 ms sind aber in Ordnung.

Ich meinte: immer wenn gezeichnet wird muss die aktuelle Zeit dasein.

Dann sind wir durchaus gleicher Meinung.

Dann ist es so genau wie möglich und dabei wird nur so viel CPU wie nötig verbraucht.

Prinzipiell auch hier Zustimmung. Nur kann es passieren, das paint mehr als einmal pro Sekunde aufgerufen wird. Da muss man nicht unbedingt jedes mal die Zeit auslesen (wie gesagt: alle 100/200/500 ms).

 

[Campino]

Also:
die uhr ist sogar nur Minutengenau. Derzeit rufe ich den TimerTask alle 30000ms auf (also alle halbe Minute) dann setze ich die Zeit als Text eines JMenu und rufe validate() auf.

EDIT: Problem gelöst. ich hatte das ganze erst als Thread und hab dann beim Umwandeln in TimerTask die while(true) Schleife drinngelassen, statt alle 30000ms die zeit zu aktualisieren wurde alle 30000ms eine neue Endlosschleife gestartet, die die Zeit aktualisierte.