Swing getValueAt(int,int) bug der JTable

[Holycrap]

Hallo, mein Problem ist, dass bei meinem TableModel bei

getValueAt(int rowIndex, int columnIndex);

immer als columnIndex 0 zurückgegeben wird, was eigentlich meiner meinung nach irgendwie unlogisch :bahnhof: ist mein Code sieht wie folgt aus:

private JTable table;
private MyTableModel model = new MyTableModel();

//Methode zum erzeugen der Table
private JScrollPane createTable() {
        JScrollPane jsp = new JScrollPane();
        table = new JTable(model);

//ColumnModel hat den richtigen ColumnCount, 
//da der TableHeader richtig gezeichnet wird.
        table.setColumnModel(MyTableModel.getColumnModel());

        table.getTableHeader().setReorderingAllowed(false);
        table.getTableHeader().setUpdateTableInRealTime(true);

        jsp.setViewportView(table);
        return jsp;
    }

//In der Collection sind bereits Elemente hab es hier nur abgekürzt
private Vector vector = new Vector();

    public int getRowCount() {
        return vector.size();
    }

    public int getColumnCount() {
        return getColumnModel().getColumnCount();
    }

    public Object getValueAt(int rowIndex, int columnIndex) {
        DataClass d = vector.get(rowIndex);
        //Und hier liegt mein Problem der columnIndex ist immer auf 0
        switch (columnIndex) {
            ...
        }
    }

//Gibt ein ColumnModel mit 9 TableColumns zurück
    public static DefaultTableColumnModel getColumnModel() {
        ...
    }

Hoffe ihr könnt mir helfen, bin schon am verzweifeln...;(
achja falls es relevant ist:
IDE: NetBeans 6.7
JDK: 6 Update 14

 

[SlaterB]

dass nur Spalte 0 vorhanden ist, könnte am
DefaultTableColumnModel getColumnModel()
liegen, welches du geschickt komplett ausblendest

gib doch mal getColumnCount() in die Konsole aus

 

[Holycrap]

das hab ich aus dem grund ausgeblendet...weil dort kein fehler liegt, denn da kommt nicht 0 sondern 9 zurück bei getColumnCount();

 

[Ebenius]

Du machst da was ziemlich schlechtes. Das ColumnModel sollte niemals mit dem TabellenModell verbunden sein. Das ColumnModel ist eine Sicht auf das Modell, das Modell selbst sollte aber eine feste Anzahl von Spalten haben, die unabhängig vom ColumnModel ist.

Die JTable verändert das ColumnModel auf Basis des Tabellenmodells. Wenn das Tabellenmodell abhängig vom ColumnModel ist, das in Abhängigkeit vom Tabellenmodell durch die JTable geändert wird, wodurch das Tabellenmodell anders aussieht, was das ColumnModell verändert, ... dann hast Du ein Henne-Ei-Problem.

Ebenius

 

[SlaterB]

na gut, dann verzichte ich mal auf stundenlanges nachfragen 'bitte vollständigen Code posten' usw.
und komme direkt zu einem Testprogramm:

public class TestGUI
    extends JFrame
{

    private JTable table;
    private MyTableModel model = new MyTableModel();

    // Methode zum erzeugen der Table
    private JScrollPane createTable()
    {
        JScrollPane jsp = new JScrollPane();
        table = new JTable(model);

        // ColumnModel hat den richtigen ColumnCount,
        // da der TableHeader richtig gezeichnet wird.
        table.setColumnModel(MyTableModel.getColumnModel());

        table.getTableHeader().setReorderingAllowed(false);
        table.getTableHeader().setUpdateTableInRealTime(true);

        jsp.setViewportView(table);
        return jsp;
    }

    public TestGUI()
        throws Exception
    {

        add(createTable());

        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(300, 300);
        setVisible(true);
    }

    public static void main(String[] args)
        throws Exception
    {

        new TestGUI();
    }
}


class MyTableModel
    extends AbstractTableModel
{
    private static DefaultTableColumnModel cm;
    // In der Collection sind bereits Elemente hab es hier nur abgekürzt
    private Vector vector = new Vector();

    public MyTableModel()
    {
        vector.add("");
        vector.add("");
        vector.add("");
        fireTableDataChanged();
    }

    public int getRowCount()
    {
        return vector.size();
    }

    public int getColumnCount()
    {
        return getColumnModel().getColumnCount();
    }

    public Object getValueAt(int rowIndex, int columnIndex)
    {
        System.out.println("getValueAt: " + rowIndex + ", " + columnIndex);
        return "test";
    }

    // Gibt ein ColumnModel mit 9 TableColumns zurück
    public static DefaultTableColumnModel getColumnModel()
    {
        if (cm == null)
        {
            cm = new DefaultTableColumnModel();
            cm.addColumn(new TableColumn());
            cm.addColumn(new TableColumn());
        }
        return cm;
    }
}

Ausgabe:

getValueAt: 0, 0
getValueAt: 0, 0
getValueAt: 1, 0
getValueAt: 1, 0
getValueAt: 2, 0
getValueAt: 2, 0
getValueAt: 0, 0
getValueAt: 0, 0
getValueAt: 1, 0
getValueAt: 1, 0
getValueAt: 2, 0
getValueAt: 2, 0

wie du es auch beschreibst,

Lösung:

.
            cm = new DefaultTableColumnModel();
            cm.addColumn(new TableColumn(0));
            cm.addColumn(new TableColumn(1));

es liegt also (wahrscheinlich) genau am TableColumnModel

 

[Holycrap]

danke vielmals leute, es lag tatsächlich daran das ich den modelIndex bei den TableColumns nicht übergeben hab, ihr seit die besten :toll::applaus::toll::applaus: