Update von TimerTask

[DieDreiJavas]

Hallo!
Wir programmieren ein Spiel und möchten die Zeit messen. Dafür haben wir die TimerTask-Klasse benutzt.

package projekt.control;

import java.util.Timer;
import java.util.TimerTask;

import javax.swing.JLabel;


public class Task extends TimerTask
{
	JLabel uhrlabel;
	

        int zeit = 1;
	
     public void run()  
     {
      System.out.println(zeit + "Sekunden verstrichen" );
      zeit++;
    
      }
	
      public void fuehreTimerAus()
      {
		Timer timer = new Timer();

		timer.schedule ( new Task(), 2000 );

		timer.schedule ( new Task(), 1000, 1000 );

	}
	
	public JLabel Uhranzeigen()
	{
		JLabel uhrlabel = new JLabel (zeit+"Sekunden verstrichen");
		uhrlabel.setText(zeit + "Verstrichene Zeit");
		return uhrlabel;
	}
}

In der GUI wird dann das JLabel "uhrlabel" in den JFrame eingefügt

	public void updateZeitlabel()
	{
		
		
		Task uhr = new Task();
		contentPane.add(BorderLayout.SOUTH, uhr.Uhranzeigen() );
	}

Das JLabel wird auch angezeigt, allerdings läuft die Uhr nicht weiter. In der Konsole wird die Zeit richtig angezeigt. Gibt es eine Möglichkeit, die Uhrzeit im JLabel zu updaten?
Vielen Dank schonmal!

 

[Michael...]

Gibt es eine Möglichkeit, die Uhrzeit im JLabel zu updaten?

Ja per setText(....) ;-) Nur weil sich irgendwo ein int Wert ändert, kommt das JLabel es ja nicht unbedingt mit, man muss ihm schon "sagen", dass etwas neues angezeigt werden soll.

 

[DieDreiJavas]

Also nochmal per setText? Wo muss das denn dann hin?

 

[Michael...]

in die run, jedes mal wenn sich der Wert

zeit

ändert.

 

[DieDreiJavas]

hm ,nee, irgendwie haben wirs anscheinend doch noch nicht so ganz verstanden^^...
unsere Methode run() sieht jetzt so aus:

	public void run()  
  {
    System.out.println(zeit + "Sekunden verstrichen" );
    JLabel uhrlabel = new JLabel (zeit+"Sekunden verstrichen");
	uhrlabel.setText(zeit + "Verstrichene Zeit");
    zeit++;
    
  }

und es passiert nichts ! Was muss denn in den anderen Methoden geändert werden bzw. ist es so wie wirs gemacht haben überhaupt der richtige Anfang ???:L

 

[mjdv]

Vielleicht solltest du nicht jedes mal ein neues JLabel erstellen, sondern das nutzen was du auch auf dem JFrame anzeigst.

 

[DieDreiJavas]

Genau, das ist dann das JLabel uhrlabel.
Die Methode Uhranzeigen() wird von der GUI aufgerufen und dann wird das JLabel im JFrame angezeigt. Allerdings steht dann dort eben nur "1 Sekunde verstrichen", die Zeit läuft nur in der Konsole weiter. Wenn wir setText in die run()-Methode schreiben geht garnichts mehr.
Also wo genau wird denn die Zeit auf dem JLabel hochgezählt?:bahnhof:

 

[Michael...]

Also wo genau wird denn die Zeit auf dem JLabel hochgezählt?:bahnhof:

Nirgends, da Du in der run - wie bereits von mjdv erwähnt - jedes mal ein neues JLabel erstellst. Das ist natürlich Schmarrn.
Wie ich gerade gesehen habe hast Du in dem ersten Code zwei Variablen mit dem Namen uhrlabel. Eine Instanzvariable, die nicht initialisiert wird und lokale Variable die nur in der Methode "Uhranzeigen" (Methodennamen beginnen in Java üblicherweise mit einem kleinBuchstaben).
Du benötigst aber nur die Instanzvariable und solltest das

JLabel

vor uhrlabel in der Methode entfernen.

 

[DieDreiJavas]

Ah, so ist das gemeint, ok!
Also wir haben den Quellcode jetzt geändert:

package projekt.control;

import java.util.Timer;
import java.util.TimerTask;

import javax.swing.JLabel;



public class Task extends TimerTask
{
	

    int zeit = 1;
    JLabel uhrlabel = new JLabel();
    
	public void run()  
  {
    System.out.println(zeit + "Sekunden verstrichen" );
    uhrlabel.setText(zeit + "Sekunden  verstrichen");
    zeit++;
  }
	
	public void fuehreTimerAus()
	{
	
		Timer timer = new Timer();

		timer.schedule ( new Task(), 2000 );

		timer.schedule ( new Task(), 1000, 1000 );

	}
	
	
	public JLabel uhranzeigen()
	{
		uhrlabel.setText(zeit + "Sekunden verstrichen");
		zeit++;
		return uhrlabel;
	}
	

}

allerdings mit demselben Ergebnis !
Was wir auch noch überlegt haben, in die Methode

 uhranzeigen()

nur

 return uhrlabel;

zu schreiben, weil so der Code ja doppelt dasteht, aber dabei ist dann eine NullPointerException aufgetreten. Die Methode

 run()

kann man auch nicht in der Gui-Klasse aufrufen... :bahnhof:

 

[Michael...]

Schwer zu sagen, was woran liegt, da ich ja den restlichen Code nicht kenne. Bei der NullPointerException steht aber dabei in welcher Zeile sie auftritt, ab da kann man ja analysieren woran es liegt.

Die run() Methode wir nicht aufgerufen, das passiert beim Aufruf von Timer.schedule(...) (warum wird das bei Euch zweimal aufgerufen?)
Hier mal ein Beispiel:
http://www.java-forum.org/java-basics-anfaenger-themen/94269-jlabel-aktualisiert.html#post598677

 

[DieDreiJavas]

vielen Dank für deine Hilfe! Irgendwie schaffen wir aber trotzdem es nicht, das Beispiel für unsere Zwecke umzubauen... unser Quellcode sieht jetzt so aus, die NullPointerException tritt allerdings jetzt wieder auf (haben die Stelle im Quellcode kommentiert):

package projekt.control;

import java.util.Timer;
import java.util.TimerTask;

import javax.swing.JLabel;

import projekt.view.Gui;



public class Task extends TimerTask
{
	

	Gui g;
	int zeit = 1;
    public static JLabel uhrlabel;

    
	public void run()  
  {
    System.out.println(zeit + "Sekunden verstrichen" );
   
    //hier tritt die NullPointerException auf:
    uhrlabel.setText(zeit + "Sekunden  verstrichen");
    
    uhrlabel.setVisible(true);
    zeit++;
    g.updateZeitlabel();
  }
	
	public void fuehreTimerAus()
	{
	
		Timer timer = new Timer();

		timer.schedule ( new Task(), 2000 );

		timer.schedule ( new Task(), 1000, 1000 );

	}

}

der relevante Quellcode in der Gui sieht so aus:

	public void updateZeitlabel()
	{
		contentPane.add(BorderLayout.SOUTH, Task.uhrlabel );
	}

und das Ganze wird so aufgerufen:

		if (kommando.equals("Neues Spiel"))
		{
			
		gui.starteSpiel();
			//das Deckblatt soll verschwinden (mit setVisible(false)), das Spielfeld soll aufgebaut werden
			
			//Objekt der Klasse Task erzeugen, mit dem Objekt die Methode fuehreTimerAus aufrufen
			Task uhr = new Task();
			uhr.fuehreTimerAus();
			gui.updateZeitlabel();
			
		}

Vielen Dank

 

[discipulo]

Ich vermute das liegt daran das dein Task.uhrlabel gar nicht auf dein Panel gesetzt wird. Daher NullPointer

 contentPane.add(Task.uhrlabel, BorderLayout.SOUTH);

Erst die Komponente dann die Borderkonstante

 

[Michael...]

Die NPE kommt weil vermutliche nirgends

uhrlabel = new JLabel()

steht und bitte die Variable nicht static machen!
Dann sind da immernoch diese zweimalig Aufrufe von schedule in der Methode fuehreTimerAus mit jeweils new Task(), d.h. es exitieren dann mindestens drei Task Objekte (ist sicher nicht so gewünscht) Warum nicht einfach nur:

timer.schedule(this, 1000, 100);

Des weiteren wird in dieser

updateZeitlabel()

das Label in irgend einer GUI platziert. Ist das Label nicht von Anfang an in der GUI sichtbar? Dann muss ein validate() auf den Container ausgeführt werden und die Methode sollte wohl eher set/showZeitlabel heißen.
Der Aufruf dieser Methode innerhalb der run() scheint wohl auch eher ein verzweifelter Versuch gewesen zu sein - raus damit, hat nichts darin verloren ;-)

 

[Michael...]

Ich vermute das liegt daran das dein Task.uhrlabel gar nicht auf dein Panel gesetzt wird. Daher NullPointer

 contentPane.add(Task.uhrlabel, BorderLayout.SOUTH);

Erst die Komponente dann die Borderkonstante

Es funktionieren beide Varianten, sonst liese sich das ja garnicht kompilieren und ausführen und die NPE kommt erst beim Ausführen.