unglaubwürdige NullPointerException

[Schnapsfahne]

Hallo zusammen,

hoffe es kann mir jemand helfen.
Es ist so, eine GUI soll alle 10 Sekunden refresht werden. Also lasse ich einen Thread laufen, der dann weitere Klassen /Methoden aufruft, welche dann die Werte erneuern.
Wenn ein Event vom User ausgelöst wird, dann soll eine andere Ausgabe alle 10 Sekunden erneuert werden.

Hier nun zu den Details:
Thread:

 	System.out.println("lm: " + listmich);
				 	System.out.println("container: " + c);
				 	System.out.println("posit: " + posit_Layout);
				 	stati.trennen(listmich, c, posit_Layout, true); // <- Ort der Fehlermeldung

aufgerufene Methode:

public void trennen(ArrayList serverRueckgabe_AList, Container c, GridBagConstraints posit, boolean ausfuehren_model){

Konsolenausgabe:

lm: [2:falscher_test:1:1:1:1:1:1:0:1:1:1:1, 3:sid_vkv:0:0:0:0:0:0:0:0:0:0:0, 1:testsache:0:0:1:1:1:0:0:1:1:1:1]
container: javax.swing.JPanel[null.contentPane,0,0,193x52,layout=java.awt.GridBagLayout,alignmentX=null,alignmentY=null,border=,flags=9,maximumSize=,minimumSize=,preferredSize=]
posit: java.awt.GridBagConstraints@6fa9fc
java.lang.NullPointerException
at test.Thread_senden.run(Thread_senden.java:60)
at java.lang.Thread.run(Thread.java:536)

Wie man in der Ausgabe erkennen kann, ist kein Wert NULL. Es werden auch alle parameter übernommen,
bin daher jetzt Ratlos, wodran das liegen könnte.
Dürfen Threads überhaupt Methoden in anderen Klassen aufrufen?

 

[Beni]

Swing ist nicht threadsicher, du verwirrst Swing vielleicht...

Du kannst mit den Methoden "invokeAndWait" und "invokeLater" Swing-Methoden sicher aufrufen. In der API hat es ein kleines Beispiel.

mfg Beni

 

[Mick]

container: javax.swing.JPanel[null.contentPane,0,0,193x52,layout=java.awt.GridBagLayout,alignmentX=null,alignmentY=null,border=,flags=9,maximumSize=,minimumSize=,preferredSize=]

Das sieht aus, als wenn Dein Panel null ist. Man kann aus Deinem Code jetzt nicht soviel sehen. Du solltest mal die relevanten Stellen posten.

Ist 'stati' denn initialisiert?

Grüße,
Mick

 

[Schnapsfahne]

Mick hatte recht,
Hab zwar stati zugewiesen, aber vergessen zu initialisieren.
Aber weitere Fehler bleiben leider nicht aus.

Hier erst einmal eine Klasse von mir, bzw. der Thread.

stati darf nur einmal initialisiert werden, da in den anderen Klassen,
manche Objekte nicht mehrfach erzeugt werden dürfen.

import java.util.*;
import java.awt.*;

public class Thread_senden extends Thread implements Runnable { 
	
	private int buttonauswahl;
	private Status stati;
	private Container c;
	private GridBagConstraints posit_Layout;
	private volatile boolean nicht_aufhoeren;
	
	public Thread_senden(int buttonauswahl, Container c, GridBagConstraints posit_Layout){
		this.buttonauswahl = buttonauswahl;
		this.c = c;
		this.posit_Layout = posit_Layout;
	}

	public void setNicht_aufhoeren(boolean nicht_aufhoeren){
		this.nicht_aufhoeren = nicht_aufhoeren;
	}

	public boolean getNicht_aufhoeren(){
		return this.nicht_aufhoeren;
	}

	public void run(){
		int auswahl_server;
		nicht_aufhoeren = true;
		while (nicht_aufhoeren){
			try {
				System.out.println("Button: " + buttonauswahl);
				if (buttonauswahl == 0) {
					auswahl_server = 1;
				} else {
					auswahl_server = buttonauswahl;
				}
				ArrayList rueckgabeArrayServer;						
				rueckgabeArrayServer = Senden.send(auswahl_server);
				// Methode "trennen" vom Objekt Status (Referenz "stati") aufrufen, Ausgabe von der Statustabelle aufrufen
				 if (buttonauswahl == 0) {
				 	System.out.println("lm: " + rueckgabeArrayServer);
				 	System.out.println("container: " + c);
				 	System.out.println("posit: " + posit_Layout);
					stati = new Status(rueckgabeArrayServer, c, posit_Layout);
					stati.trennen(rueckgabeArrayServer, c, posit_Layout, true);
				 } else if (buttonauswahl == 1){ 
					stati.trennen(rueckgabeArrayServer, c, posit_Layout, false);
				 }
				 else{
					stati.oplog(rueckgabeArrayServer, c, posit_Layout);
				 }
				 
				sleep(10000);
			} catch (InterruptedException e1) {
				e1.printStackTrace();
			}
		}
	}
}

In der Klasse, wo der Thread erzeugt wird, sieht es wie folgt aus:

		buttonauswahl = 0;
		thread_klasse = new Thread_senden (buttonauswahl,c, posit_Layout);
		th= new Thread(thread_klasse);
		th.start();
		posit_Layout.gridx = 0;
		posit_Layout.gridy = 2;
		c.add(empty, posit_Layout);	// Leeres Label dem Container hinzufügen

		posit_Layout.gridx = 1;
		posit_Layout.gridy = 2;
		c.add(empty, posit_Layout);

		JButton exitbutton = new JButton("Exit"); 
		posit_Layout.gridx = 2;
		posit_Layout.gridy = 2;
		c.add(exitbutton, posit_Layout);

		statiButton.addActionListener(new ActionListener() {

			public void actionPerformed(ActionEvent event) {
				thread_klasse.setNicht_aufhoeren(false);
				int buttonauswahl = 1;
				thread_klasse = new Thread_senden(buttonauswahl,c, posit_Layout);
				th= new Thread(thread_klasse);
				th.start();   // <<<<<<--- NullPointerException <<<<<<
			}
		});

		oplogButton.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent event) {
				int buttonauswahl = 2;
				thread_klasse.setNicht_aufhoeren(false);
				thread_klasse = new Thread_senden(buttonauswahl,c, posit_Layout);
				th.start();    // <<<<<<--- NullPointerException <<<<<<
			}
		});

[Edit by Beni: Code-tag benutzt, ist übersichtlicher]

 

[Schnapsfahne]

Hat sich erledigt.

musste nur immer wieder einen neuen Thread erstellen.
Hatte im Geiste vergessen, das ich den Thread ja beende und nicht anhalte und deswegen ein neuer erstellt werden muss,weil der andere ja gar nicht mehr vorhanden ist. :roll:

aber Danke für die rasche Hilfe. Da fallen manchen die simpelsten Fehler eher auf als einem selber.