Tabellen Spalte löschen

[acdc]

Hallo,

Ich habe folgendes Problem:

mit den Zeilen:

 public void mousePressed(MouseEvent e)
    {
      if(e.getButton()==MouseEvent.BUTTON3)  //nur wenn die rechte Maustaste gedrückt wird
      {
        JTableHeader th = (JTableHeader)e.getSource();
        Point p = e.getPoint();
        int col = th.columnAtPoint(p);
                      
        InputDialog input=new InputDialog();
        input.viewDialog();
        System.out.println(col);

        if(input.getValue())
        {
          th.getColumnModel().getColumn(col).setHeaderValue(input.getText());         
          th.resizeAndRepaint();

          newData=tableModel.getColumnIdentifiers(); //Eigene Methode in OwnTableModel "return columnIdentifiers"
          newData.setElementAt(input.getText(),col); // neuer Text aus eigenem Dialog
       }
      }
    }

ändere ich die Spaltennamen.

Nach der Änderung lösche ich genau diese, eben umbenannte Spalte mit:

      int selCol=Tabelle1.getSelectedColumn();
    
      Vector headers;
      headers=modelTabelle1.getColumnIdentifiers();
      headers.removeElementAt(selCol);

      int index = Tabelle1.convertColumnIndexToView(selCol);
      if (index != -1)
        Tabelle1.getColumnModel().removeColumn(Tabelle1.getColumnModel().getColumn(index));

Dabei erscheind folgende Meldung:

Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: -1
at java.util.Vector.elementAt(Vector.java:430)
at javax.swing.table.DefaultTableColumnModel.getColumn(DefaultTableColumnModel.java:277)
at javax.swing.plaf.basic.BasicTableHeaderUI.getHeaderRenderer(BasicTableHeaderUI.java:654)
at javax.swing.plaf.basic.BasicTableHeaderUI.paintCell(BasicTableHeaderUI.java:670)
at javax.swing.plaf.basic.BasicTableHeaderUI.paint(BasicTableHeaderUI.java:646)
at javax.swing.plaf.ComponentUI.update(ComponentUI.java:143)
at javax.swing.JComponent.paintComponent(JComponent.java:763)
at javax.swing.JComponent.paint(JComponent.java:1029)
at javax.swing.JComponent.paintChildren(JComponent.java:864)
at javax.swing.JComponent.paint(JComponent.java:1038)
at javax.swing.JViewport.paint(JViewport.java:747)
at javax.swing.JComponent.paintChildren(JComponent.java:864)
at javax.swing.JComponent.paint(JComponent.java:1038)
at javax.swing.JComponent.paintToOffscreen(JComponent.java:5124)
at javax.swing.BufferStrategyPaintManager.paint(BufferStrategyPaintManager.java:278)
at javax.swing.RepaintManager.paint(RepaintManager.java:1220)
at javax.swing.JComponent._paintImmediately(JComponent.java:5072)
at javax.swing.JComponent.paintImmediately(JComponent.java:4882)
at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:803)
at javax.swing.RepaintManager.paintDirtyRegions(RepaintManager.java:714)
at javax.swing.RepaintManager.seqPaintDirtyRegions(RepaintManager.java:694)
at javax.swing.SystemEventQueueUtilities$ComponentWorkRequest.run(SystemEventQueueUtilities.java:128)
at java.awt.event.InvocationEvent.dispatch(InvocationEvent.java:209)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:597)
at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:269)
at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:184)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:174)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:169)
at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:161)
at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)


Lösche ich jedoch eine andere Spalte, so funktioniert das einwandfrei.
Da werden vielleicht in den Tiefen von JTabel oder so Werte micht richtig übernommen!?
Ich habe sonst absolut keine Idee, warum das sein könnte.

Hat jemand einen Vorschlag?

DANKE

acdc

 

[Ebenius]

Frage: Warum benennt man eine Spalte um, um sie anschließend zu löschen?

Das (oder: Ein) Problem ist, dass Du mit dem Modell im Code-Block 2 Zeile 5 falsch umgehst. Wenn aus Sicht des Tabellenmodells die Spaltenanzahl verändert wird, muss das Modell alle Zuhörer mit einem entsprechenden Event benachrichtigen (structure changed). Ansonsten verhält es sich falsch. Damit hat Dein Fehler in jedem Fall zu tun.

Ebenius

 

[acdc]

@Ebenius
Der Benutzer hat ja viele Einfälle und so muss ich das Programm so schreiben, dass es ihm Stand hält!

habs nun mal mit

      Vector headers;
      headers=modelT1.getColumnIdentifiers();
      System.out.println(headers.toString());
      headers.removeElementAt(selCol);
      modelT1.fireTableStructureChanged();

versucht.

habs auch mit

      Vector headers=new Vector();
      headers=modelT1.getColumnIdentifiers();
      headers.removeElementAt(selCol);
      modelT1.setColumnIdentifiers(headers);

versucht, da wird ja auch ein StructureChange ausgeführt.

Leider weis ich nicht mit welchen Methoden ich sehen kann wo etwas nicht stimmt. Ich habe auch alle möglichen getColumnCounts
(Tabelle1.getColumnCount();Tabelle1.getColumnModel().getColumnCount() ausgegeben und die waren immer richtig und gleich.


Vielleicht wird nicht sauber umbenannt? Leider finde ich keine andere Möglichkeit Spalten umzubenennen.

acdc

 

[Ebenius]

Oh da fällt mir auf: im Code-Block 2 in Deinem ersten Beitrag in Zeile 7 konvertierst Du die selektierte Spalte. [c]selCol[/c] ist doch bereits eine Angabe in der View, die kann man nicht nochmal konvertieren. Probier's mal so aus:

      int selCol=Tabelle1.getSelectedColumn();

      final TableColumnModel cm = Tabelle1.getColumnModel();
      cm.removeColumn(cm.getColumn(selCol));

      Vector headers;
      headers=modelTabelle1.getColumnIdentifiers();
      headers.removeElementAt(selCol);
      modelTabelle1.fireTableStructureChanged();

Ist autoCreateColumnsFromModel [c]false[/c]? Wenn nicht, dann kannst Du Dir die Zeilen 3 und 4 auch sparen.

Ebenius

 

[acdc]

autoCreateColumnsFromModel ist true;

Habe daher die 2 Zeilen auch weggelassen und nur

      Vector headers;
      headers=modelTabelle1.getColumnIdentifiers();
      headers.removeElementAt(selCol);
      modelTabelle1.fireTableStructureChanged();

probiert.

-> keine Änderung.

liegt es vielleicht an der Methode wie ich den Spaltennamen ändere? Bibt es eine bessere bzw. Richtigere? Gei Google hab ich bis jetzt noch Nichts gefunden. Hat es vielleicht mit irgendeinem Eventlistener zutun? - Wenn ich debugge (ich benutze Netbeans) wird das fireActionPerformed() (JAbstrcatButton) ausgeführt - fragt glaub ich alle Listener ab - dann gibt irgendwann diese Exception bei

at javax.swing.table.DefaultTableColumnModel.getColumn(DefaultTableColumnModel.java:277)

...weis nicht... der Debugger hilft mir hier, denk ich, nicht weiter.

danke für die Antworten

acdc

 

[Ebenius]

Hier mal ein Beispiel (KSKB) wie das Löschen funktioniert. Ich würde in jedem Fall der Tabelle verbieten, das ColumnModel selbst zu erzeugen. Ansonsten werden die Spaltennamen zurück gesetzt, veränderte Spaltenbreiten gehen verloren und die Spaltenreihenfolge wird auch zurück gesetzt, wenn man eine Spalte löscht.

/* (@)TableColumnRemoveFun.java */

/* Copyright 2010 Sebastian Haufe

 * Licensed under the Apache License, Version 2.0 (the "License");
   you may not use this file except in compliance with the License.
   You may obtain a copy of the License at

       [url]http://www.apache.org/licenses/LICENSE-2.0[/url]

 * Unless required by applicable law or agreed to in writing, software
   distributed under the License is distributed on an "AS IS" BASIS,
   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
   See the License for the specific language governing permissions and
   limitations under the License. */

package com.ebenius;

import java.awt.BorderLayout;
import java.awt.Window;
import java.awt.event.InputEvent;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.util.Enumeration;
import java.util.Vector;

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

public class TableColumnRemoveFun {

  private static class MyTableModel extends DefaultTableModel {

    private static final long serialVersionUID = 1L;

    MyTableModel(int rowCount, int columnCount) {
      super(rowCount, columnCount);

      // fix the column names
      final String[] columnNames = new String[columnCount];
      for (int i = 0; i < columnCount; i++) {
        columnNames[i] = getColumnName(i);
      }
      setColumnIdentifiers(columnNames);
    }

    @SuppressWarnings("unchecked")
    public void removeColumn(int columnIndex) {
      columnIdentifiers.remove(columnIndex);

      for (int i = columnIndex; i < getColumnCount(); i++) {
        ((Vector<Object>) dataVector.get(i)).remove(i);
      }
      fireTableStructureChanged();
    }
  }

  /**
   * Test main method.
   * 
   * @param args ignored
   */
  public static void main(String[] args) {
    final JTable table = new JTable(new MyTableModel(5, 5));
    table.createDefaultColumnsFromModel();
    table.setAutoCreateColumnsFromModel(false);

    table.getTableHeader().addMouseListener(new MouseAdapter() {

      @Override
      public void mouseClicked(MouseEvent e) {
        if (SwingUtilities.isLeftMouseButton(e)) {
          final JTableHeader source = (JTableHeader) e.getComponent();
          final Window root = SwingUtilities.getWindowAncestor(source);
          final int mods = e.getModifiersEx();
          final String name;
          final int columnIndex = source.columnAtPoint(e.getPoint());
          if (columnIndex == -1) {
            return;
          }
          final JTable table = source.getTable();
          final TableColumnModel cm = table.getColumnModel();
          final TableColumn column = cm.getColumn(columnIndex);
          final int modelCol = table.convertColumnIndexToModel(columnIndex);

          // set column header value
          if ((mods & InputEvent.SHIFT_DOWN_MASK) != 0
                && (name =
                      JOptionPane.showInputDialog(root,
                            "Enter new Column Name")) != null) {
            column.setHeaderValue(name);
            source.resizeAndRepaint();
          }

          // delete column
          if ((mods & InputEvent.CTRL_DOWN_MASK) != 0) {
            cm.removeColumn(column);

            // need to fix all column model indices beyond the deleted column
            for (Enumeration<TableColumn> all = cm.getColumns(); all
                  .hasMoreElements();) {
              final TableColumn tmp = all.nextElement();
              if (tmp.getModelIndex() > modelCol) {
                tmp.setModelIndex(tmp.getModelIndex() - 1);
              }
            }

            // remove the column from the data model
            ((MyTableModel) table.getModel()).removeColumn(modelCol);
          }
        }
        super.mouseClicked(e);
      }
    });

    final JPanel contentPane = new JPanel(new BorderLayout(6, 6));
    contentPane.add(new JLabel("<html>SHIFT+Click: Rename Column<br>"
          + "CTRL+Click: Remove Column"), BorderLayout.NORTH);
    contentPane.add(new JScrollPane(table), BorderLayout.CENTER);

    final JFrame f = new JFrame("Test Frame: TableColumnRemoveFun"); //$NON-NLS-1$
    f.setContentPane(contentPane);
    f.pack();
    f.setLocationRelativeTo(null);
    f.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
    f.setVisible(true);
  }
}

Der Grund warum Dein Code oben nicht richtig funktioniert hat wahrscheinlich mit den Zeilen 99-106 in meinem Beispiel zu tun.

Ich hoffe das hilft. Happy Hacking!

Ebenius

 

[acdc]

Danke für dein Beispiel - jetzt funktionierts!

was bei mir nun genau der Fehler war hab ich nicht ganz heraus gefunden - vielleicht weil ich über einen Butten und einer markierten Zeile eine Spalte löschen wollte.

Nachdem ich nicht Stunden für ein Problem verbrauchen will habe ich nun mit Hilfe deines Codes die Klasse JTable und was so dazu gehört besser verstehen gelern und einfach meine Methoden neu geschrieben!

-es funktioiniert jetzt und ich finde es hat auch noch vorteile, weil man bei der Gestaltung der Tabelle etwas flexibler ist!

danke (echt super, diese positive Erfahrung hatte ich noch nie in einem Forum)