Threads Probleme mit Thread

[Kakulukiam21]

Einen wunderschönen guten Morgen,

ich setz jetzt schon eine Weile an einer Problematik und finde dafür einfach keine Lösung, daher frage ich mal hier nach in der Hoffnung das eine Lösung gefunden wird.

Ich habe ein Programm, was an sich relativ Simple ist. Ich habe eine GUI auf der was dargestellt wird. Alles nicht so schwer. Jetzt soll aber in der GUI immer ein aktueller Wert aus der Datenbank geholt und dargestellt werden. Das aktualisieren habe ich jetzt in einem neuen Thread gestartet, falls es falsch war korrigiert mich bitte.

Jedoch funktioniert es ja auch soweit. Jede Sekunde wird ein der aktuelle Wert angezeigt. Wenn ich jedoch dann die Seite verlasse um mir eine andere anzuschauen (der Thread läuft in der Zeit weiter, da ich ihn nur einmal starten möchte) und komme dann auf die Seite wo ich den Wert anzeigen lass, wird er einfach nicht mehr angezeigt. Und ich weiß nicht wieso. (Da find ich das Wort Quälcode wieder passend)

public class Uebersicht {
	JPanel tmp;
	JTextArea temperatur_AT1;

	public void anzeige() {

		final JFrame window = new JFrame();
		window.setUndecorated(true);
		// Dimension screenSize = Toolkit.getDefaultToolkit().getScreenSize();
		// window.setSize(screenSize);
		window.setSize(1024, 768);
		window.setLayout(null);

		tmp = new JPanel();
		tmp.setLayout(null);
		tmp.setBounds(375, 360, 120, 53);
		if (!DemoThread.isRunning) {
			DemoThread refreshTMP = new DemoThread();
			refreshTMP.textArea(tmp);
			refreshTMP.start();
		} else {
			// Hier soll einfach nur noch mal das refreshTMP.textArea(temp)
			// angezeigt werden,
			// ohne das ich noch mal ein Objekt von DemoThread erstelle. Meine
			// Idee war:
			temperatur_AT1 = new JTextArea();
			temperatur_AT1
					.setText(Double.toString(cn.getSollTemperaturMilch()));
			temperatur_AT1.setOpaque(true);
			temperatur_AT1.setBackground(Color.LIGHT_GRAY);
			temperatur_AT1.setEditable(false);
			temperatur_AT1.setHighlighter(null);
			temperatur_AT1.setFocusable(false);
			temperatur_AT1.setBorder(null);
			temperatur_AT1.setBounds(40, 10, 75, 40);
			temperatur_AT1.setFont(new Font("Arial", Font.PLAIN, 30));
			tmp.setVisible(true);

		}

	}

}

public class DemoThread extends Thread {
	DBConnect cn = new DBConnect();
	static DemoThread other;
	JTextArea temperatur_AT1;
	JFrame display;
	int thread = 0;
	static JPanel bg;
	static boolean isRunning = false;

	public void textArea(final JPanel tmp) {
		temperatur_AT1 = new JTextArea();
		temperatur_AT1.setText(Double.toString(cn.getSollTemperaturMilch()));
		temperatur_AT1.setOpaque(true);
		temperatur_AT1.setBackground(Color.LIGHT_GRAY);
		temperatur_AT1.setEditable(false);
		temperatur_AT1.setHighlighter(null);
		temperatur_AT1.setFocusable(false);
		temperatur_AT1.setBorder(null);
		temperatur_AT1.setBounds(40 ,10, 75, 40);
		temperatur_AT1.setFont(new Font("Arial", Font.PLAIN, 30));
		isRunning = true;
		bg = tmp;
	}

	public void run() {
		do {
			bg.add(temperatur_AT1);
			try {
				sleep(1000);
			} catch (InterruptedException e) {
			}
			temperatur_AT1
					.setText(Double.toString(cn.getSollTemperaturMilch()));
			bg.remove(temperatur_AT1);
			System.out.println();
		} while (thread == 0);
	}
	
}

Ich hoffe ihr könntet mir dabei helfen

LG Kaku

 

[Michael...]

Grundsätzlich: Merkwürdiger Aufbau. Was hat es mit dem if else in der Methode anzeige() auf sich?

Ausserdem sollte in dem Thread nur der anzuzeigende Wert geändert werden. Du allerdings nutzt hier eine zusätzliche TextArea und fügst und entfernst diese aus dem Panel. Warum?

Wenn der Thread eine Referenz auf die entsprechende Textkomponente besitzt, dann reicht ein einfaches setText("neuerText2") in der run aus.

 

[Kakulukiam21]

Hey,

Eigentlich ist es nur eine if() Abfrage ob der Thread schon gestartet ist. Wenn nicht soll er dies dann machen

Zum Thema anzeigen:
Wie erstelle ich bitte eine Referenz zur TextArea? Ich kann in der run() Methode doch nix übergeben oder irre ich mich da?

EDIT

Ansich würde das ja reichen nur, woher soll ich jetzt genau diese eine TextArea hernehmen

public class DemoThread extends Thread {
	DBConnect cn = new DBConnect();
	int thread = 0;
	static boolean isRunning = false;

	public void run() {
		do {
			try {
				sleep(1000);
			} catch (InterruptedException e) {
			}
			temperatur_AT1
					.setText(Double.toString(cn.getSollTemperaturMilch()));
			System.out.println();
		} while (thread == 0);
	}

}

 

[Michael...]

Ansich würde das ja reichen nur, woher soll ich jetzt genau diese eine TextArea hernehmen

public class DemoThread extends Thread {
	DBConnect cn = new DBConnect();
	int thread = 0;
	static boolean isRunning = false;

	public void run() {
		do {
			try {
				sleep(1000);
			} catch (InterruptedException e) {
			}
			temperatur_AT1
					.setText(Double.toString(cn.getSollTemperaturMilch()));
			System.out.println();
		} while (thread == 0);
	}

}

Da zu kannst du ja in der Klasse einen Konstruktor implementieren, dem eine Referenz auf die TextArea übergeben werden kann.

 

[Kakulukiam21]

Da zu kannst du ja in der Klasse einen Konstruktor implementieren, dem eine Referenz auf die TextArea übergeben werden kann.

Dann würde das ja so aussehen:

//DemoThread Klasse
public DemoThread(JTextArea temperatur_AT1, JTextArea temperatur_AT2){
	temperatur1 = temperatur_AT1;
	temperatur2 = temperatur_AT2;
}

//------------------------------------
//Uebersicht Klasse
if (!DemoThread.isRunning) {
	DemoThread refreshTMP = new DemoThread(temperatur_AT1, temperatur_AT2);
	refreshTMP.start();
}

Und das wird nur einmal aufgerufen. Da ich aber auch noch eine andere Seite besuchen will und wieder auf diese Seite zurück komme wird wiederum nix angezeigt.

 

[Michael...]

was meinst Du mit "andere Seite besuchen" und warum sollte "nach dem Zurückkommen" nichts mehr angzeigt werden.

 

[Kakulukiam21]

was meinst Du mit "andere Seite besuchen" und warum sollte "nach dem Zurückkommen" nichts mehr angzeigt werden.

Ich meine damit das ich nicht nur die Seite "Uebersicht" habe, sondern zum Beispiel auch die Seite "Warnung"... Wenn ich jetzt auf die Seite "Uebersicht" gehe funktioniert alles super. Dann bin ich aber auch fertig damit und gehe zur Seite "Warnung", dort schaue ich mich dann ein bisschen um und entscheide dann wieder auf Seite "Uebersicht" zu gehen um die Temperatur im Blick zu haben, sie wird DANN aber nicht angezeigt.

 

[Michael...]

Ich meine damit das ich nicht nur die Seite "Uebersicht" habe, sondern zum Beispiel auch die Seite "Warnung"...

OK, z.B. mittels CardLayout zwischen den Ansichten wechseln oder verwendest Du ein JTabbedPane?

Wenn ich jetzt auf die Seite "Uebersicht" gehe funktioniert alles super. Dann bin ich aber auch fertig damit und gehe zur Seite "Warnung", dort schaue ich mich dann ein bisschen um und entscheide dann wieder auf Seite "Uebersicht" zu gehen um die Temperatur im Blick zu haben, sie wird DANN aber nicht angezeigt.

Da scheint ein grundlegender Fehler im Design/ der Konzeption vorzuliegen. Kann es sein, dass beim Wechsel, z.B. neue Objekte / Komponenten erzeugt werden?

 

[Kakulukiam21]

OK, z.B. mittels CardLayout zwischen den Ansichten wechseln oder verwendest Du ein JTabbedPane?

das Böse null-Layout. Ich muss Buttons spezifisch anpassen und ein Layout ist in dem Fall noch zu statisch Überlegungen auf JTabbedPane zu wechseln laufen.

Da scheint ein grundlegender Fehler im Design/ der Konzeption vorzuliegen. Kann es sein, dass beim Wechsel, z.B. neue Objekte / Komponenten erzeugt werden?

Das wird gemacht, da ich die Seite ja jedes mal neu aufbauen muss wenn ich sie verlassen habe oder etwa nicht? Ich häng da grad fest ;(

 

[Michael...]

und ein Layout ist in dem Fall noch zu statisch

Da habe ich eher gegenteilige Erfahrungen gemacht

Das wird gemacht, da ich die Seite ja jedes mal neu aufbauen muss wenn ich sie verlassen habe oder etwa nicht? Ich häng da grad fest ;(

Wenn eine "Seite" bereits existiert bzw. aufgebaut ist, dann kann man diese doch wieder "verwenden"

In Deinem Fall "verlierst" Du daher wohl die Referenzen bzw. hälst Referenzen auf "falsche" Objekte.

 

[Kakulukiam21]

Und wie kann ich diese dann wieder verwenden?

 

[Michael...]

Und wie kann ich diese dann wieder verwenden?

Schwierig zu sagen, da man Dein Applikationsdesign und Klassenaufbau nicht kennt. Vermutlich lässt Dein Aufbau das nur schwer oder gar nicht zu.
Mit einem CardLayout/JTabbedPane (auch hier kann man Komponenten mit Null Layout reinstecken) erzeugt man das Layout bzw. die Komponente und steckt diese ins Layout bzw. in die Pane. Beim Ansichtenwechsel werden diese Komponenten dann "automatisch" wieder verwendet.

Aber auch wenn Du Dein Layout selbst aufbaust, hast Du doch für jede Ansicht ein JPanel, die Du abwechselnd anzeigen könntest. Mir ist nicht klar warum Du die Komponenten jedes Mal neu erstellst. Das ist doch ein Vorteil der OOP, dass man Objekte erstellt, verwendet und "wieder"verwendet.

 

[Kakulukiam21]

Aber auch wenn Du Dein Layout selbst aufbaust, hast Du doch für jede Ansicht ein JPanel, die Du abwechselnd anzeigen könntest. Mir ist nicht klar warum Du die Komponenten jedes Mal neu erstellst. Das ist doch ein Vorteil der OOP, dass man Objekte erstellt, verwendet und "wieder"verwendet.

Klar habe ich ein Panel, ich denke mein Fehler war es das Panel wieder zu löschen anstatt es einfach unsichtbar zu machen

 

[Michael...]

Mal eine Demo zu Threads und CardLayout:

import java.awt.BorderLayout;
import java.awt.CardLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;

public class CardLayoutThreadDemo extends JPanel {
	
	public static void main(String[] s) {
		JFrame frame = new JFrame();
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setBounds(0, 0, 300, 200);
		frame.setLocationRelativeTo(null);
		
		frame.add(new CardLayoutThreadDemo());
		frame.setVisible(true);
	}
	
	private CardLayout layout;
	
	public CardLayoutThreadDemo() {
		layout = new CardLayout();
		this.setLayout(layout);
		
		ActionListener listener = new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				layout.show(CardLayoutThreadDemo.this, e.getActionCommand());
			}
		};
		
		this.add(new AnyView(listener, "UPDATE_VIEW"), "ANY_VIEW");
		this.add(new UpdateView(listener, "ANY_VIEW"), "UPDATE_VIEW");
	}
	
	class UpdateThread implements Runnable {
		private JLabel label;
		
		public UpdateThread(JLabel label) {
			this.label = label;
		}
		
		public void run() {
			while(true) {
				try {
					Thread.sleep(1000);
				} catch (InterruptedException e) {
					e.printStackTrace();
				}
				if (label.isShowing()) {
					label.setText("" + System.currentTimeMillis()/1000);
					System.out.println("Label aktualisiert: " + System.currentTimeMillis()/1000);
				}
			}
		}
	}
	
	class UpdateView extends JPanel {
		public UpdateView(ActionListener listener, String command) {
			this.setLayout(new BorderLayout());
			JButton button = new JButton("zum AnyPanel");
			button.setActionCommand(command);
			this.add(button, BorderLayout.SOUTH);
			button.addActionListener(listener);
			
			JLabel label = new JLabel("ToBeUpdated", JLabel.CENTER);
			this.add(label, BorderLayout.CENTER);
			
			new Thread(new UpdateThread(label)).start();
		}
	}
	
	class AnyView extends JPanel {
		public AnyView(ActionListener listener, String command) {
			this.setLayout(new BorderLayout());
			JButton button = new JButton("zum UpdatePanel");
			button.setActionCommand(command);
			this.add(button, BorderLayout.SOUTH);
			button.addActionListener(listener);
		}
	}
}