Swing JTable während Laufzeit beschreiben

[Magublafix]

Hallo Leute,
Bin relativ neu in Java (habe früher C# programmiert) und schon nervt mich das ganze GUI Zeug....
Habe jetzt schon viele Forenbeiträge gelesen aber konnte mein Problem nicht lösen... Also mein letzter Ausweg

Also mein Problem JTable aktualisiert sich nicht und bleibt leer.

Die wichtigen Bestandteile meiner Hauptklasse mit der GUI:

import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.util.Vector;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.SwingUtilities;
import javax.swing.UIManager;
import javax.swing.table.DefaultTableModel;

import org.dyno.visual.swing.layouts.Constraints;
import org.dyno.visual.swing.layouts.GroupLayout;
import org.dyno.visual.swing.layouts.Leading;
import org.dyno.visual.swing.layouts.Trailing;



public class Main extends JFrame {
	private static final long serialVersionUID = 1L;
	private JButton Manuel;
	DefaultTableModel model = new DefaultTableModel();
	int row = 0;
	private JTable jTable0 = new JTable(model);
	private JScrollPane jScrollPane0;
	private static final String PREFERRED_LOOK_AND_FEEL = "javax.swing.plaf.metal.MetalLookAndFeel";
	[B]public Main() {
		createTable();
		initComponents();
		WriteSignals ws = new WriteSignals();
		ws.start();
	}
	
	public Main(String Bssid, String Ssid, int Signal)
	{
		addData(Bssid,Ssid,Signal);
	}
	
	private  void createTable()
	{
		model.addColumn("Bssid");
		model.addColumn("Ssid");
		model.addColumn("Signal");
	}
	
	public void addData(String Bssid, String Ssid, int Signal)
	{
		Vector<String> addData = new Vector<String>();
		addData.add(Bssid);
		addData.add(Ssid);
		addData.add(Signal);
		model.addRow(addData);
		model.fireTableStructureChanged();
		model.fireTableDataChanged();
		
	}[/B]
	
	private void initComponents() {
		setLayout(new GroupLayout());
		add(getManuel(), new Constraints(new Leading(9, 12, 12), new Leading(58, 10, 10)));
		add(getJScrollPane0(), new Constraints(new Trailing(12, 381, 10, 10), new Leading(7, 187, 10, 10)));
		setSize(541, 377);
	}


	private JScrollPane getJScrollPane0() {
		if (jScrollPane0 == null) {
			jScrollPane0 = new JScrollPane();
			jScrollPane0.setViewportView(getJTable0());
		}
		return jScrollPane0;
	}


	private JTable getJTable0() {
		if (jTable0 == null) {
			jTable0 = new JTable();
			jTable0.setModel(new DefaultTableModel(new Object[][] {}, new String[] {}) {
				private static final long serialVersionUID = 1L;
				Class<?>[] types = new Class<?>[] { Object.class, Object.class, };
	
				public Class<?> getColumnClass(int columnIndex) {
					return types[columnIndex];
				}
			});
			jTable0.setAutoCreateColumnsFromModel(false);
		}
		return jTable0;
	}


	private static void installLnF() {
		try {
			String lnfClassname = PREFERRED_LOOK_AND_FEEL;
			if (lnfClassname == null)
				lnfClassname = UIManager.getCrossPlatformLookAndFeelClassName();
			UIManager.setLookAndFeel(lnfClassname);
		} catch (Exception e) {
			System.err.println("Cannot install " + PREFERRED_LOOK_AND_FEEL
					+ " on this platform:" + e.getMessage());
		}
	}

	/**
	 * Main entry of the class.
	 * Note: This class is only created so that you can easily preview the result at runtime.
	 * It is not expected to be managed by the designer.
	 * You can modify it as you like.
	 */
	public static void main(String[] args) {
		installLnF();
		SwingUtilities.invokeLater(new Runnable() {
			@Override
			public void run() {
				Main frame = new Main();
				frame.setDefaultCloseOperation(Main.EXIT_ON_CLOSE);
				frame.setTitle("Main");
				frame.getContentPane().setPreferredSize(frame.getSize());
				frame.pack();
				frame.setLocationRelativeTo(null);
				frame.setVisible(true);
			}
		});
	}

}

Klasse WriteSignals... Daten werden an Main Klasse übergeben...

public class WriteSignals extends Thread {
	String[] Bssid = new String[100];
	String[] Ssid = new String[100];
	int[] SignalStrength = new int[100];
	int numberOfReadings = 0;
	
	public WriteSignals()
	{
		
	}
	
	public void getData()
	{
		Signal s = new Signal();
		Bssid = s.getBssid();
		Ssid = s.getSsid();
		SignalStrength = s.getSignalStrength();
		numberOfReadings = s.getNumberOfSignals();
	}
	
	public void run() {
		
		while(true)
		{
			getData();
		
			for(int i=0; i < numberOfReadings; i++)
			{
				System.out.println(Bssid[i]+"  "+Ssid[i]+"  "+SignalStrength[i]);
				[B]Main main = new Main(Bssid[i], Ssid[i], SignalStrength[i]);[/B]
				try{
					Thread.currentThread().sleep(1000);
				}catch(Exception ex){}
			}
			
		}
			/*try{
			Thread.currentThread().sleep(1000);
			}catch(Exception e){}*/
	}

}

Vielleicht hat ja jemand von euch Zeit mir zu helfen wäre ganz nett....
Falls sich jemand fragt, was das Ganze sein soll....
Wird ein Steurungprogramm für eine Wlan-Antenne, die vertikal und horizontal drehbar ist und sich automatisch den besten Empfang sucht...

lg Magublafix

PS: Ich entschuldige mich gleich für die hässliche Programmierung...

 

[Michael...]

Habs nur mal kurz überflogen und unterstelle mal, dass das nicht so gewünscht ist:

public class WriteSignals extends Thread {
...
	Main main = new Main(Bssid[i], Ssid[i], SignalStrength[i]);
...
}

Du erzeugst hier im Thread jedes Mal ein neues Fenster, das nicht einmal angezeigt wird.

 

[hiti]

ach, hast du ja. ...vergiss es. Hab mich verguckt.
Gruss, hiti

Hallo Magublafix,

setze die JTable in eine JScrollPane und diese dann in Deinen Panel/Fram mit:

JPanel p = new JPanel();
frame.setContentPane(p);
...
JScrollPane scrollpane = new JScrollPane(table);
p.add(scrollpane);

 

[Magublafix]

Jo stimmt...
Die übergabe der Variablen muss ich so und so noch umgestallten und einen Eventlistener einbauen....
Aber das dürfte nichts an dem Hauptproblem ändern, dass sich die JTable nicht aktualisieren lässt...
wenn ich einen Datensatz vor dem initComponents(); einfüge funktioniert es aber danach nicht mehr...

lg Magublafix

 

[hiti]

...
und mit nach dem update ein table.setVisible(true) oder frame.setVisible(true) ??

 

[Michael...]

Aber das dürfte nichts an dem Hauptproblem ändern, dass sich die JTable nicht aktualisieren lässt...

Doch, den mit jeden neuen Fenster erzeugst Du auch eine neue Tabelle in die Du dann die Daten einfügst. --> Die sichtbare Tabelle ändert sich nicht, das sich deren Daten nicht ändern.

 

[Magublafix]

hmmmmmm ^^
klingt logisch....
Ich danke mal vorerst werd ich dann gleich mal testen...

 

[Michael...]

Hier mal ein kurzes Bsp. auf dem Du vielleicht aufbauen kannst. Generell sind JTable und Threads Themen mit denen man sich erst einmal getrennt und intensiver beschäftigen sollte, bevor man beide kombiniert.

import javax.swing.*;
import javax.swing.table.*;

public class AutomaticActualizeTableDemo extends JFrame {
	private DefaultTableModel model;
	private JTable table;

	public AutomaticActualizeTableDemo() {
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		this.setBounds(0, 0, 300, 200);
		this.setLocationRelativeTo(null);
		
		model = new DefaultTableModel(new Object[][] {}, new Object[] {"Bssid", "Ssid", "Signal"});
		table = new JTable(model);
		
		this.getContentPane().add(new JScrollPane(table));
		
		new Thread(new ChangingThread(this)).start();
	}
	
	public void addRowToModel(Object[] o) {
		model.addRow(o);
	}
	
	public static void main(String[] args) {
		new AutomaticActualizeTableDemo().setVisible(true);
	}
}

class ChangingThread implements Runnable {
	private AutomaticActualizeTableDemo aatd;
	private int i;
	
	public ChangingThread(AutomaticActualizeTableDemo aatd) {
		this.aatd = aatd;
		i = 0;
	}
	
	public void run() {
		while (i<=20) {
			try {
				Thread.sleep(1000);
			} catch (InterruptedException e) {}
			aatd.addRowToModel(new Object[] {"Hurz", "Hurz", new Integer(i++)});
		}
	}
}

 

[Magublafix]

Hey danke für deine Hilfe...
Habs jetzt folgendermaßen gelöst....

Habe einfach das TableModel an eine Methode in der Klasse WriteSignal weitergegeben und kann sie nun dort in der Endlosschleife beschreiben....
Auf die Idee das TableModel zu übergeben bin ich vorher nicht gekommen

Danke sehr
lg Magublafix

 

[Magublafix]

Sodala hab ich auch gleich das nächste Problem...

Nun wird die Tabelle zwar beschrieben, aber leider werden alle Zeilen immer auf den letzten Wert gesetzt, sprich wenn in der ersten zeile bereits ("test", "test", "test") steht und ich eine neue zeile ("test2","test2","test2") einfügen will wird auch die erste zeile auf diesen Wert gesetzt...

Code sieht folgendermaßen aus....

Main (mit GUI):

import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.util.Vector;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.SwingUtilities;
import javax.swing.UIManager;
import javax.swing.table.DefaultTableModel;

import org.dyno.visual.swing.layouts.Constraints;
import org.dyno.visual.swing.layouts.GroupLayout;
import org.dyno.visual.swing.layouts.Leading;
import org.dyno.visual.swing.layouts.Trailing;


//VS4E -- DO NOT REMOVE THIS LINE!
public class Main extends JFrame {
	private static final long serialVersionUID = 1L;
	//Initialize button for manual control
	private JButton Manuel;
	//Initialize TableModel
	DefaultTableModel model = new DefaultTableModel();
	//Initialize JTable
	private JTable jTable0 = new JTable(model);
	//Initialize JScrollPane
	private JScrollPane jScrollPane0;
	private static final String PREFERRED_LOOK_AND_FEEL = "javax.swing.plaf.metal.MetalLookAndFeel";
	public Main() {
		createTable();
		initComponents();
		//Initialize WriteSignals, deliver TableModle
		WriteSignals ws = new WriteSignals();
		ws.getTable(model);
		ws.start();
	}
	
	private  void createTable()
	{
		//Create Columns
		model.addColumn("Bssid");
		model.addColumn("Ssid");
		model.addColumn("Signal");
	}
	
	private void initComponents() {
		setLayout(new GroupLayout());
		add(getManuel(), new Constraints(new Leading(9, 12, 12), new Leading(58, 10, 10)));
		add(getJScrollPane0(), new Constraints(new Trailing(12, 381, 10, 10), new Leading(7, 187, 10, 10)));
		setSize(541, 377);
	}


	private JScrollPane getJScrollPane0() {
		if (jScrollPane0 == null) {
			jScrollPane0 = new JScrollPane();
			jScrollPane0.setViewportView(getJTable0());
		}
		return jScrollPane0;
	}


	private JTable getJTable0() {
		if (jTable0 == null) {
			jTable0 = new JTable();
			jTable0.setModel(new DefaultTableModel(new Object[][] {}, new String[] {}) {
				private static final long serialVersionUID = 1L;
				Class<?>[] types = new Class<?>[] { Object.class, Object.class, };
	
				public Class<?> getColumnClass(int columnIndex) {
					return types[columnIndex];
				}
			});
			jTable0.setAutoCreateColumnsFromModel(false);
		}
		return jTable0;
	}


	private JButton getManuel() {
		if (Manuel == null) {
			Manuel = new JButton();
			Manuel.setText("ScanManuelly");
			Manuel.addMouseListener(new MouseAdapter() {
	
				public void mouseClicked(MouseEvent event) {
					//Initialize manual control
					WadarManuell wm = new WadarManuell();
					wm.setVisible(true);
					
				}
			});
		}
		return Manuel;
	}

	private static void installLnF() {
		try {
			String lnfClassname = PREFERRED_LOOK_AND_FEEL;
			if (lnfClassname == null)
				lnfClassname = UIManager.getCrossPlatformLookAndFeelClassName();
			UIManager.setLookAndFeel(lnfClassname);
		} catch (Exception e) {
			System.err.println("Cannot install " + PREFERRED_LOOK_AND_FEEL
					+ " on this platform:" + e.getMessage());
		}
	}

	/**
	 * Main entry of the class.
	 * Note: This class is only created so that you can easily preview the result at runtime.
	 * It is not expected to be managed by the designer.
	 * You can modify it as you like.
	 */
	public static void main(String[] args) {
		installLnF();
		SwingUtilities.invokeLater(new Runnable() {
			@Override
			public void run() {
				Main frame = new Main();
				frame.setDefaultCloseOperation(Main.EXIT_ON_CLOSE);
				frame.setTitle("Main");
				frame.getContentPane().setPreferredSize(frame.getSize());
				frame.pack();
				frame.setLocationRelativeTo(null);
				frame.setVisible(true);
			}
		});
	}

}

Klasse WriteSignals (Beschreiben der JTable):

import java.util.Vector;

import javax.swing.table.DefaultTableModel;


public class WriteSignals extends Thread {
	//Initialize Vector for Data of Table
	Vector addData = new Vector();
	String[] Bssid = new String[100];
	String[] Ssid = new String[100];
	int[] SignalStrength = new int[100];
	int numberOfReadings = 0;
	//Initialize TableModel
	DefaultTableModel model = new DefaultTableModel();
	
	public void getData()
	{	
		//Initialize Signal
		Signal s = new Signal();
		//Get Data from Signal
		Bssid = s.getBssid();
		Ssid = s.getSsid();
		SignalStrength = s.getSignalStrength();
		numberOfReadings = s.getNumberOfSignals();
	}
	
	public void getTable(DefaultTableModel model)
	{	
		//Set this.TableModel to Main.TableModel
		this.model = model;
	}
	
	
	
	public void run() {
		
		while(true)
		{
			getData();
			//run for-loop, write Signals into Table
			for(int i=0; i < numberOfReadings; i++)
			{
				addData.clear();
				System.out.println(Bssid[i]+"  "+Ssid[i]+"  "+SignalStrength[i]);
				addData.add(Bssid[i]);
				addData.add(Ssid[i]);
				addData.add(SignalStrength[i]);
				model.addRow(addData);
				System.out.println(addData);
			}
			try{
				//wait 1500 milliseconds
				Thread.currentThread().sleep(1500);
			}catch(Exception ex){}
			
		}
	}

}

 

[Michael...]

public class WriteSignals extends Thread {
...
	public void run() {
...
		addData.clear();
...
	}
}

Vergiss das mal mit dem leeren des Vectors und erzeuge hier einen neuen Vector. So wie Dein Code momentan läuft, wird jedesmal eine Referenz auf den selben Vector in die Tabelle eingefügt, dessen Inhalt zudem jedesmal überschrieben wird. Somit hast Du dann in jeder Zeile das drin stehen, was Du alles letztes in den Vector gesteckt hast.

 

[Magublafix]

Hi
So in die Richtung hab ich mir das auch gedacht mit dem Fehler,
Aber was meinst du erzeug hier einen neuen Vector, das is ja in ner endlosschleife, da kann ich doch nicht jedesmal einen neuen Vector erzeugen...

 

[Michael...]

Wenn Du etwas in der Tabelle anfügen willst, wird Dir nichts anderes übrig bleiben.
Du brauchst pro Zeile einen Vector.

 

[Magublafix]

ja ich kann mir das grad Codemäßig nicht vorstellen...

könntest mir vl kurz mal in Codeform aufschreiben wie du das meinst?..

lg Geri

 

[Magublafix]

Ok...Ok..

Habs jetzt verstanden
bin etwas schwer von Begriff

Danke auf jeden Fall