JTable sortiert ausgeben

[Tino124]

Hallo allerseits, es geht um folgendes: Ich habe ein Programmchen erstellt das mir Daten einer URL via DOM parser ausliest und diese in eine Tabelle schreibt. Soweit funktioniert auch alles, nur ist die ausgabe der Tabbelle komlett durcheinander ich hätte gerne einfach 5 Spalten die " Linie, Ziel, Platz, HF und Zeit" heißen und die jeweils dazugehörigen Tags auch in deren Spalte schreibt.

Hier einmal mein Quellcode zum Verständnis

import java.io.*;
import java.net.URL;

import org.w3c.dom.*;
import javax.xml.parsers.*;
import org.xml.sax.*;
import java.util.*;
import javax.swing.*;
import javax.swing.table.*;

public class Test extends AbstractTableModel {
	Vector data;
	Vector columns;

	public Test() {
		try {
			   DocumentBuilderFactory dbFactory = DocumentBuilderFactory.newInstance();
		        DocumentBuilder dBuilder = dbFactory.newDocumentBuilder();
		        Document doc = dBuilder.parse(new URL("xy.php")
		                        .openStream());
		        doc.getDocumentElement().normalize();
		       
		        
		        
	        NodeList nl = doc.getElementsByTagName("Linie");
			NodeList n2 = doc.getElementsByTagName("Ziel");
			NodeList n3 = doc.getElementsByTagName("Platz");
			NodeList n4 = doc.getElementsByTagName("HF");
			NodeList n5 = doc.getElementsByTagName("Zeit");

			
			NodeList DFIpublic = doc.getElementsByTagName("Abfahrt");
			String data1;
			String data2;
			String data3;
			String data4;
			String data5;
			data = new Vector();
			columns = new Vector();
			for (int i = 0; i < DFIpublic.getLength(); i++) {
				data1 = nl.item(i).getFirstChild().getNodeValue();
				data2 = n2.item(i).getFirstChild().getNodeValue();
				data3 = n3.item(i).getFirstChild().getNodeValue();
				data4 = n4.item(i).getFirstChild().getNodeValue();
				data5 = n5.item(i).getFirstChild().getNodeValue();
				String line = data1 + " " + data2 + " " + data3 + " " + data4 + " " + data5  ;
				StringTokenizer st2 = new StringTokenizer(line, " ");
				while (st2.hasMoreTokens())
					data.addElement(st2.nextToken());
			}
			columns.add("");
			columns.add("");
			columns.add("");
			columns.add("");
			columns.add("");

			

		} catch (Exception e) {
			e.printStackTrace();
		}
	}

	public int getRowCount() {
		return data.size() / getColumnCount();
	}

	public int getColumnCount() {
		return columns.size();
	}

	public Object getValueAt(int rowIndex, int columnIndex) {
		return (String) data.elementAt((rowIndex * getColumnCount())
				+ columnIndex);
	}

	public static void main(String argv[]) throws Exception {
		Test t = new Test();
		JTable table = new JTable();
		table.setModel(t);
		JScrollPane scrollpane = new JScrollPane(table);
		JPanel panel = new JPanel();
		panel.add(scrollpane);
		JFrame frame = new JFrame();
		frame.add(panel, "Center");
		frame.pack();
		frame.setVisible(true);
	}
}

Jemand ne Idee?

 

[KrokoDiehl]

Hallo. Zunächst einmal erzeugst du mit

            columns.add("");
            columns.add("");
            columns.add("");
            columns.add("");
            columns.add("");

lauter leere Spaltennamen. Hier gehören deine Spaltenüberschriften rein. Bzw. muss du ohnehin die Methode

getColumneName(int)

vom TableModel überschreiben und den entsprechenden Namen zurückgeben. Wenn du die Namen dort hardkodierst, kannst du dir den Vector columns auch sparen.

Ansonsten finde ich das Einlesen der Daten sehr untypisch. Eine Tabelle stellt zweidimensionale Daten dar, da ist es auch intuitiver wenn du z.B. ein

String[][]

-Array nehmen würdest und hättest dann auch wesentlich intuitivere Zugriffe (z.B. in

getValueAt()

).
Das Einlesen der Daten würde demnach auch etwa so aussehen:

data = new String[ DFIpublic.getLength() ][5];

for (int i = 0; i < DFIpublic.getLength(); i++) {
    data[i][0] = n1.item(i).getFirstChild().getNodeValue();
    data[i][1] = n2.item(i).getFirstChild().getNodeValue();
    data[i][2] = n3.item(i).getFirstChild().getNodeValue();
    data[i][3] = n4.item(i).getFirstChild().getNodeValue();
    data[i][4] = n5.item(i).getFirstChild().getNodeValue();
}

Und getValueAt():

@Override
public Object getValueAt(int row, int col) {
    return data[row][col];
}

 

[Tino124]

Naja obs untypisch ist oder nicht, ich sitze seid 2 Wochen an dem blöden Programm und habe kaum Java Erfahrung.. Ich bin schonmal ganz froh drum das so auf die Reihe bekommen zu haben. Und wie mir das jetzt bei meinem Problem mit der Sotierung helfen soll versteh ich nicht wirklich ???:L

 

[KrokoDiehl]

Ein Problem an der Stelle wird sein, dass die Datenstruktur des Vektors (1-dimensional) untypisch für eine Tabelle (2-dimensional) ist und du damit das Umrechnungsproblem hast. Daher ist mein Vorschlag eine passendere Datenstruktur zu wählen, und zwar das 2D-Stringarray. Du kannst natürlich auch

List<List<String>>

machen oder sonstwas ... aber für den Anfang ist das Stringarray wohl gut.

Wenn du das nicht magst müsstest du mal näher beschreiben was "durcheinander" heißt.

 

[Tino124]

Durcheinander heißt, dass ich verschiedene Tags abgreife( Linie Ziel Platz HF und Zeit).so sie sind auf der hinterlegten URL des öfteren hinterlegt mal 6 mal mal 7 mal mal 10 ma das variert, dass heißt meine tabelle sollte A dynamisch sein, das habe ich glaub ich ganz gut gelöst. Aber das ganze wird total verwurschtelt ausgegeben, das schreibt es wie es will in die Tabelle, und es sollte so sein das die Linien in der Spalte a sind das ziel in der spalte b usw.. am besten wäre natürlich, wenn die spalten sogar benannt wären.

 

[KrokoDiehl]

Okay. Man sollte nicht zuviel Kompliziertes auf einmal machen, daher zuerst einmal volle Konzentration auf die Tabelle und wir gehen davon aus du feste Datengrößen hast, sprich 5 Spalten und DFIpublic.getLenth() Zeilen.

Wenn du damit ein Stringarray wie folgt anlegst, dann hast du genau deine Tabellenstruktur als 2D-Array. Die TableModel-Methoden können also ohne Umrechnerei die Informationen holen:

public class Test extends AbstractTableModel {

    private String[][] data;

    public Text() {
        data = new String[ DFIpublic.getLength() ][ 5 ];
        // daten einlesen
    }

    @Override
    public int getColumnCount() {
        return 5;
    }

    @Override 
    public int getRowCount() {
        return data.length;
    }

    @Override
    public String getColumnName(int col) {
        switch (col) {
            case 0: return "Linie";
            ...
        }
        return "gibts nid";
    }

    @Override
    public Object getValueAt(int row, int col) {
        return data[row][col];
    }
}

 

[truesoul]

Hallo.

Naja, vieles hat schon KrokoDiehl erwähnt aber ich würde dir empfehlen auch die Grundlagen dafür anzueignen. Btw Eigentlich solltest du dir überhaupt die Grundlagen von Java aneignen bevor du auf biegen und brechen das fertig machst, wohl dann mit hilfe von jemanden der so gnädig ist.


Java ist auch eine Insel – 15.25 Tabellen mit JTable

 

[Tino124]

Der DOM Parser muss logischer weiße noch im code enthalten sein vorher, sonst komm ich ja garnicht an die Tags ran oder?

Ja also es sollten einfach 5 Feste reihen sein und die Zeilen unten müssten dynamisch sein, da auf der URL da immer akutellisiert und es immer unterschiedlich viele tag packe gibt.


Ja das hast du wohl recht, mit dem Programm hab ich mich ziehmlich übernommen.. aber sobald ich das auf die Reihe bekommen hab, werd ichs wieder langsamer angehen. Ohne die Grundkentnisse hat mann da kaum chancen wie ich die letzten 2 Wochen gemerkt habe..

 

[Tino124]

Ich kriegs tortz des Codes den du gepostet hast nicht hin :/ Egal wie ich es dreh und wende es kommt nichts bei raus

 

[parabool]

ersetze mal die for-Schleife aus deinen Angangsquelltext mit folgendem:

for (int i = 0; i < DFIpublic.getLength(); i++)            
{            	
 data.addElement(nl.item(i).getFirstChild().getNodeValue());
 data.addElement(n2.item(i).getFirstChild().getNodeValue());
 data.addElement(n3.item(i).getFirstChild().getNodeValue());
 data.addElement(n4.item(i).getFirstChild().getNodeValue());
 data.addElement(n5.item(i).getFirstChild().getNodeValue());            
}

 

[Tino124]

Funktioniert super, habe den Fehler auch gefunden. Die tabelle an sich wurde vorher auch sotiert ausgegeben, der Fehler wurde durch eine Leerstelle zwischen 2 wörtern verursacht. sobald ich einen anderen link verwedet habe der diese leerstelle nicht beinhaltet wurde die tabelle wieder formatiert ausgegeben. aber nun mit deiner methode funktioniert das mit beiden links 🙂

Jetzt müsste ich nur noch spalten namen vergeben, ne idee wie ich das hinbekomme?

 

[parabool]

table.getTableHeader().setDefaultRenderer(new DefaultTableCellRenderer()
{
	String[] cols = {"1","2","3","4","5"};
	public Component getTableCellRendererComponent(JTable arg0,Object arg1, boolean arg2, boolean  arg3, int arg4,int col) 
	{
		this.setText(cols[col]);										
		return this;
	}
});

 

[Tino124]

Die Frage klingt jetzt vielleicht dumm, aber wo muss ich das hinpacken egal wo ichs einsetzte kommt ne exception

 

[parabool]

direkt nach:

table.setModel(t);