Du verwendest einen veralteten Browser. Es ist möglich, dass diese oder andere Websites nicht korrekt angezeigt werden. Du solltest ein Upgrade durchführen oder ein alternativer Browser verwenden.
Ich erstelle eine JTable, welcher ich ein DefaultTableModel(Object[][] rowData, Object[] columnNames) zuweise.
Die JTable sieht dann folgendermaßen aus:
Code:
[User Name][ Delete ]
Name 1 Delete-Button
Name 2 Delete-Button
Name 3 Delete-Button
Die User Names lese ich aus einer Datenbank. Soweit klappt alles wunderbar.
Jetzt mein Versuch die JTable bzw. das DefaultTableModel zu aktualisieren:
1. Der Delete-Button in der Tabelle wird gedrückt. (ActionListener ausgelöst)
2. Der User Name wird aus der Datenbank gelöscht.
3. Die User Names werden neu aus der Datenbank gelesen und in ein Object[][] gespeichert.
Dann hab ich folgendes versucht (was wahrscheinlich Quatsch ist, aber ich finde einfach nicht die richtige Lösung):
auch hier musst du die Daten korrekt ändern und die Table korrekt informieren,
die beiden Befehle sehen an sich korrekt aus, der erste setModel-Aufruf sollte für sich schon ein Neuzeichnen auslösen,
fireTableDataChanged(); ist danach eigentlich nicht nötig
warum das nicht geht ist nicht zu erkennen, alles ist denkbar, allein schon dass this.user_table nicht die in der GUI angezeigte Tabelle ist
du erstellst ein Array rowData und machst ab Zeile 19 nichts mehr damit,
es gibt keine Verknüpfung zwischen rowData und dt_model,
ist das wirklich schwer zu erkennen?
die ArrayIndexOutOfBoundsException kann an vielen Stellen auftreten, Stacktrace wäre nützlich,
allein schon ob es bei dir, im Model beim Setzen der Daten oder später in der JTable kracht,
passt this.column_names eigentlich zu den Daten mit genau 2 Einträgen?
this.column_names = new Object[] { "User Name", "Delete" };
Java:
Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: -1
at java.util.Vector.elementAt(Vector.java:430)
at javax.swing.table.DefaultTableModel.setValueAt(DefaultTableModel.java:648)
at javax.swing.JTable.setValueAt(JTable.java:2719)
at javax.swing.JTable.editingStopped(JTable.java:4721)
at javax.swing.AbstractCellEditor.fireEditingStopped(AbstractCellEditor.java:125)
at tournamentsystem.ButtonEditor.fireEditingStopped(ButtonEditor.java:83)
at tournamentsystem.ButtonEditor$1.actionPerformed(ButtonEditor.java:30)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1995)
at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2318)
at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:236)
at java.awt.Component.processMouseEvent(Component.java:6216)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3265)
at java.awt.Component.processEvent(Component.java:5981)
at java.awt.Container.processEvent(Container.java:2041)
at java.awt.Component.dispatchEventImpl(Component.java:4583)
at java.awt.Container.dispatchEventImpl(Container.java:2099)
at java.awt.Component.dispatchEvent(Component.java:4413)
at javax.swing.plaf.basic.BasicTableUI$Handler.repostEvent(BasicTableUI.java:924)
at javax.swing.plaf.basic.BasicTableUI$Handler.mouseReleased(BasicTableUI.java:1138)
at java.awt.AWTEventMulticaster.mouseReleased(AWTEventMulticaster.java:273)
at java.awt.Component.processMouseEvent(Component.java:6216)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3265)
at java.awt.Component.processEvent(Component.java:5981)
at java.awt.Container.processEvent(Container.java:2041)
at java.awt.Component.dispatchEventImpl(Component.java:4583)
at java.awt.Container.dispatchEventImpl(Container.java:2099)
at java.awt.Component.dispatchEvent(Component.java:4413)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4556)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4220)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4150)
at java.awt.Container.dispatchEventImpl(Container.java:2085)
at java.awt.Window.dispatchEventImpl(Window.java:2475)
at java.awt.Component.dispatchEvent(Component.java:4413)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
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)
PS: Sollte passieren, in der Zeile in der das Model neu gesetzt wird.
ok, du editierst die Tabelle während dieses Löschens bzw. der Button ist in der Tabelle,
das steht ja schon in deinem ersten Post,
dann kann es durchaus Probleme machen, mittendrin die Tabelle zu ändern
versuche bitte
Java:
SwingUtilities.invokeLater(new Runnable()
{
public void run()
{
user_table.setModel(dt_model);
}
});
als letzten Befehl,
also die Aktion des Änderns etwas nach hinten zu verschieben, nachdem der aktuelle Listener und weitere laufende Aktionen abgeschlossen sind
Danke, dass funktioniert wunderbar und ohne Fehlermeldungen:
Java:
SwingUtilities.invokeLater(new Runnable()
{
public void run()
{
user_table.setModel(dt_model);
}
});
Jetzt wäre ich dir noch dankbar, wenn du mir etwas genauer schildern würdest, was diese wenigen Zeilen genau machen bzw. was das für mein Programm bedeutet, dass ich das auch verstanden habe. Ein Danke sicher ich dir aber natürlich gleich zu ;-)
wie gesagt, diese Aktion nach hinten verschieben bis alles andere aktuelle getan ist,
siehe auch API-Beschreibung der Methode SwingUtilities (Java Platform SE 6)
warum?:
dein Listener ist nur eine von mehreren Aktionen, die beim Buttonklick ausgeführt werden,
der CellEditor oder irgendwelche Zeichenmethoden werden auch vorher und nachher informiert, machen irgendwas, fragen Werte von der Table ab, und wer weiß was alles,
genaues kann ich dazu gar nicht sagen
vielleicht musst du den Editor auch neu setzen nach Änderung des Models oder wie kam es überhaupt, dass anfangs Buttons angezeigt wurden?
vielleicht reicht es hier als Änderung, nicht das Model neu zu erzeugen, sondern nur diesem die geänderten Daten zu übergeben,
falls es ein DefaultTableModel ist:
setDataVector(Vector dataVector, Vector columnIdentifiers)
this.dt_model = new DefaultTableModel(rowData, this.column_names);
SwingUtilities.invokeLater(new Runnable() {
public void run() {
user_table.setModel(dt_model);
user_table.getColumn("Delete").setCellRenderer(new ButtonRenderer());
user_table.getColumn("Delete").setCellEditor(new ButtonEditor(
new JCheckBox(), conn, user_table));
}
});
Mit folgendem Ergebnis:
Java:
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at tournamentsystem.ButtonEditor.getCellEditorValue(ButtonEditor.java:66)
at javax.swing.JTable.editingStopped(JTable.java:4720)
at javax.swing.AbstractCellEditor.fireEditingStopped(AbstractCellEditor.java:125)
at tournamentsystem.ButtonEditor.fireEditingStopped(ButtonEditor.java:81)
at tournamentsystem.ButtonEditor$1.actionPerformed(ButtonEditor.java:28)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1995)
at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2318)
at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:236)
at java.awt.Component.processMouseEvent(Component.java:6216)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3265)
at java.awt.Component.processEvent(Component.java:5981)
at java.awt.Container.processEvent(Container.java:2041)
at java.awt.Component.dispatchEventImpl(Component.java:4583)
at java.awt.Container.dispatchEventImpl(Container.java:2099)
at java.awt.Component.dispatchEvent(Component.java:4413)
at javax.swing.plaf.basic.BasicTableUI$Handler.repostEvent(BasicTableUI.java:924)
at javax.swing.plaf.basic.BasicTableUI$Handler.mouseReleased(BasicTableUI.java:1138)
at java.awt.AWTEventMulticaster.mouseReleased(AWTEventMulticaster.java:273)
at java.awt.Component.processMouseEvent(Component.java:6216)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3265)
at java.awt.Component.processEvent(Component.java:5981)
at java.awt.Container.processEvent(Container.java:2041)
at java.awt.Component.dispatchEventImpl(Component.java:4583)
at java.awt.Container.dispatchEventImpl(Container.java:2099)
at java.awt.Component.dispatchEvent(Component.java:4413)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4556)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4220)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4150)
at java.awt.Container.dispatchEventImpl(Container.java:2085)
at java.awt.Window.dispatchEventImpl(Window.java:2475)
at java.awt.Component.dispatchEvent(Component.java:4413)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
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)
Im Moment kommt folgende Meldung wenn ich refreshen will:
Java:
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at tournamentsystem.ButtonEditor.getCellEditorValue(ButtonEditor.java:66)
at javax.swing.JTable.editingStopped(JTable.java:4720)
at javax.swing.AbstractCellEditor.fireEditingStopped(AbstractCellEditor.java:125)
at tournamentsystem.ButtonEditor.fireEditingStopped(ButtonEditor.java:81)
at tournamentsystem.ButtonEditor$1.actionPerformed(ButtonEditor.java:28)
at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1995)
at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2318)
at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:387)
at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:242)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:236)
at java.awt.Component.processMouseEvent(Component.java:6216)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3265)
at java.awt.Component.processEvent(Component.java:5981)
at java.awt.Container.processEvent(Container.java:2041)
at java.awt.Component.dispatchEventImpl(Component.java:4583)
at java.awt.Container.dispatchEventImpl(Container.java:2099)
at java.awt.Component.dispatchEvent(Component.java:4413)
at javax.swing.plaf.basic.BasicTableUI$Handler.repostEvent(BasicTableUI.java:924)
at javax.swing.plaf.basic.BasicTableUI$Handler.mouseReleased(BasicTableUI.java:1138)
at java.awt.AWTEventMulticaster.mouseReleased(AWTEventMulticaster.java:273)
at java.awt.Component.processMouseEvent(Component.java:6216)
at javax.swing.JComponent.processMouseEvent(JComponent.java:3265)
at java.awt.Component.processEvent(Component.java:5981)
at java.awt.Container.processEvent(Container.java:2041)
at java.awt.Component.dispatchEventImpl(Component.java:4583)
at java.awt.Container.dispatchEventImpl(Container.java:2099)
at java.awt.Component.dispatchEvent(Component.java:4413)
at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4556)
at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4220)
at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4150)
at java.awt.Container.dispatchEventImpl(Container.java:2085)
at java.awt.Window.dispatchEventImpl(Window.java:2475)
at java.awt.Component.dispatchEvent(Component.java:4413)
at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
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)
kann es sein dass im Moment die Methode refresh_del_user() überhaupt nicht mehr ausgeführt wird?
dann wäre doch völlig egal, was in SwingUtilies bzw. in der Methode an sich drin steht, dann hast du derzeit einen völlig anderen Fehler,
der StackTrace sieht jedenfalls danach aus, Zeille 66 ist überhaupt erst der Aufruf
> this.dt_model.refresh_del_user();
und der muss logischerweise immer eine NullPointerException werfen, denn dt_model ist immer null,
hast du die Methode erst kürzlich nach del_user verschoben, war die vorher im Editor drin?
das würde die Vorgänge ein wenig erklären..
dt_model ist null, dessen Methode kann nicht aufgerufen werden, im Konstruktor kannst du das del_user-Objekt übergeben,
kleingeschriebene Klassen und _ sind übrigens extrem hinderlich beim Verständnis,
die Krönung ist aber, dass dt_model gar nicht das Model ist sondern das Panel del_user,
im Panel del_user ist dt_model aber das Model, Mehrfachverwendung von Variablennamen mit unterschiedlichen Typen.., heftig böse
-------
generell bietet sich an, nicht auf mit der komischen isPushed+getTableCellEditorComponent-Kombination zu arbeiten,
eine Vorlage dazu hatte ich auch schon gerade im Internet gesehen Button Table Example : Grid TableSwing ComponentsJava
und dir folgendes Beispiel-Programm zusammengebastet:
Java:
public class Test
{
public static void main(final String[] args)
{
JButtonTableExample frame = new JButtonTableExample();
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
}
}
class JButtonTableExample
extends JFrame
{
DefaultTableModel dm;
JTable table;
public JButtonTableExample()
{
super("JButtonTable Example");
dm = new DefaultTableModel();
dm.setDataVector(new Object[][]
{
{"button 1", "foo"},
{"button 2", "bar"},
{"button 3", "bar"},
{"button 4", "bar"}}, new Object[]
{"Button", "String"});
table = new JTable(dm);
table.getColumn("Button").setCellRenderer(new ButtonRenderer());
table.getColumn("Button").setCellEditor(new ButtonEditor(new JCheckBox()));
JScrollPane scroll = new JScrollPane(table);
getContentPane().add(scroll);
setSize(400, 150);
setVisible(true);
}
class ButtonRenderer
extends JButton
implements TableCellRenderer
{
public ButtonRenderer()
{
setOpaque(true);
}
public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row,
int column)
{
if (isSelected)
{
setForeground(table.getSelectionForeground());
setBackground(table.getSelectionBackground());
}
else
{
setForeground(table.getForeground());
setBackground(UIManager.getColor("Button.background"));
}
setText((value == null) ? "" : value.toString());
return this;
}
}
class ButtonEditor
extends DefaultCellEditor
{
protected JButton button;
private String label;
public ButtonEditor(JCheckBox checkBox)
{
super(checkBox);
button = new JButton();
button.setOpaque(true);
button.addActionListener(new ActionListener()
{
public void actionPerformed(ActionEvent e)
{
fireEditingStopped();
dm.setDataVector(new Object[][]
{
{"button 1", "foo"},
{"button 2", "bar"},
{"button 3", "bar"}}, new Object[]
{"Button", "String"});
table.getColumn("Button").setCellRenderer(new ButtonRenderer());
table.getColumn("Button").setCellEditor(new ButtonEditor(new JCheckBox()));
}
});
}
public Component getTableCellEditorComponent(JTable table, Object value, boolean isSelected, int row, int column)
{
if (isSelected)
{
button.setForeground(table.getSelectionForeground());
button.setBackground(table.getSelectionBackground());
}
else
{
button.setForeground(table.getForeground());
button.setBackground(table.getBackground());
}
label = (value == null) ? "" : value.toString();
button.setText(label);
return button;
}
public Object getCellEditorValue()
{
return label;
}
}
}
also die Aktion besser normal in der actionPerformed, dann gehts vielleicht auch ohne invokeLater
Nach ein-zwei stündigem Aufwand funktioniert es jetzt endlich.
Ich hab mein Programm jetzt auf dein letztgenannten Vorschlag angepasst und das fehlende ergänzt was ich noch braucht um die JTable zu handeln.
Vielen Dank für deine Mühe.