Zeilen in Tabelle hinzufügen und löschen

[sofa-surfer]

Hallo,

man nehme eine JTable, angebracht auf einer von JPanel erbenden Klasse. Diese Klasse besitzt u.a. zwei Methoden:

public void addFile(File file)
{
    Vector content = new Vector();
    
    // alles mögliche (für hier nicht wichtige)

    content.add(tree);
    content.add(size);

    ((DefaultTableModel)table.getModel()).addRow(content);
    // table ist die eben die oben genannte Tablle
}

public void removeFiles(int[] selection)
{
    for (int i=selection.length - 1; i>=0; i--)
        ((DefaultTableModel)table.getModel()).removeRow(selection[i]);
    // das int - Array entspricht den markierten Zeilen der Tabelle
}

Dabei funktioniert leider nur die Methode zum Hinzufügen. Bei der zum Löschen bekomme ich beim Entfernen der letzten Zeile eine ArrayIndexOutOfBounds-Exception im zur Tabelle gehörigen CellEditor (wenn ich wieder auf ein Tabellenelement klicke) :bahnhof: Dieser erbt von der JTreeTable (http://java.sun.com/products/jfc/tsc/articles/treetable1/). Ansonsten stimmt das auch alles nicht so ganz, was da passiert.
In der Grundfunktionalität hab ich ihn einfach an den aus dem hiesigen Tutorial übernommen. Das Problem tritt in der fireEditingStopped()-Methode auf.

Jemand ne Ahnung wie das im Zusammenhang steht? Ich wäre sehr verbunden, mir sind nämlich die Ideen ausgegangen ....

Danke,
Matthias

 

[abollm]

Ich bin mir nicht ganz sicher (dazu bräuchte ich mehr Code zum Testen), aber versuch es einmal so, dass du die folgenden Zeilen am Beginn deiner Methode "removeFiles"einfügst:

...
		int index = table.getSelectedRow();
		if (index < 0)
			return;
...

 

[bambi]

Jepp. Wuerde mal vorschlagen, dass Du noch etwas mehr Code postest, wenn's net geht. Aus den 2
Zeilen kann man nicht so viel erkennen...

 

[sofa-surfer]

also dass die richtigen zeilen (die selektierten) übergeben werden, hab ich überprüft, das stimmt soweit
noch ein bisschen code kommt sofort ....

 

[sofa-surfer]

hier der CellEditor mit der Methode fireEditingStopped, wo ich eine Exception bekomme

public class TreeTableCellEditor extends JTreeTable implements TableCellEditor
{

    private Vector listeners = new Vector();

    public TreeTableCellEditor()
    {
        super(new MP3Model());
    }
    
    public boolean stopCellEditing()
    {
         fireEditingStopped();
         return true;  
    }
    
    protected void fireEditingStopped()
    {
        ChangeEvent e = new ChangeEvent(this);
        for (int i = 0, n= listeners.size(); i<n; i++)
        {
            ((CellEditorListener)listeners.get(i)).editingStopped(e); 
            // hier gibts ne Exception, d.h. in dem listeners-Vector soll ein Element gelesen werden, das es nicht mehr
             //gibt
        }    
    }
}

Alle anderen Methoden sind so wie im Tutorial.
Die Klasse in der die selektierten Zeilen gelesen werden:

public class DetectorGUI()
{
    // der Button zum Löschen der Zeilen:
    remove_button.addMouseListener
    (
        new MouseAdapter()
        {
            public void mouseClicked(MouseEvent e)
            {
                // bestimmen der aktuellen Tabelle (es ist möglich, mehrere geöffnet zu haben, Auswahl über eine JList 
                // classifiers
                ClassifierGUI classifier = (ClassifierGUI)classifiers.getSelectedValue();
                JTable table = classifier.getTable();
                int[] selection = table.getSelectedRows();
                // Aufruf der obigen Methode zum Löschen der Zeilen
                classifier.removeFiles(selection);
            }
        }

    );
}

Und dann noch die Klasse in der Tabelle sitzt und die auch die Methoden zum Löschen und Hinzufügen von Zeilen hat:

public class ClassifierGUI extends JPanel
{
    private ClassifierTable table;
    
    public ClassifierGUI(String label)
    {
        table = new ClassifierTable();
        this.add(table);
    }
}

So, fehlt noch die Tabelle an sich:

public class ClassifierTable extends JTable
{
    public ClassifierTable()
    {
        super(0,5);
        
        DefaultTableColumnModel cm = new DefaultTableColumnModel();
        TableColumn col;
        
        col = new TableColumn(0,200);
        col.setHeaderValue("File");
        // Setzen des CellEditors
        col.setCellEditor(new TreeTableCellEditor());

        cm.addColumn(col);

        // und halt noch alle anderen Spalten, die aber nix weiter besonderes sind

    }
}

Ich versteh nicht wie in dieser fireEditingStopped-Methode eine ArrayOutOfBounds-Exception entstehen kann, schließlich wird durch die for-Schleife doch abgesichtert, dass in dem Vector nur auf Elemente zugegriffen wird, die auch wirklich drin sind ( n= listeners.size() )

 

[Wildcard]

Was soll dann das für eine for-Schleife sein?

for (int i = 0, n= listeners.size(); i<n; i++)

 

[L-ectron-X]

Wieso Wildcard, die ist doch korrekt?
Damit die size()-Methode beim Prüfen der Bedingung nicht immer wieder aufgerufen wird, kann man sowas schon beim Initialisieren der Schleife erledigen und nur noch die int-Variablen prüfen.

 

[Wildcard]

Wieso Wildcard, die ist doch korrekt?
Damit die size()-Methode beim Prüfen der Bedingung nicht immer wieder aufgerufen wird, kann man sowas schon beim initialisieren der Schleife erledigen und nur noch die int-Variablen prüfen.

Schneller geschrieben als gedacht. Du hast natürlich recht!
sry
[edit]hielt das ',' für einen ';' :roll: [/edit]