JTable Aktualisierung nach hinzufügen eines Datensatzes

[drei1padsvb]

Hi und frohes neues Jahr!

Ich habe ein DefaultTableModel und ein JTable, dass ich mit Daten aus einer DB per Vectoren fülle.
Möchte ich nun ein Datensatz hinzufügen, funktioniert das auch, allerdings aktualisiert sich die Tabelle nicht. D.h. dass der neue Datensatz nicht zu sehen ist.

Bei mir ist das so, dass Ich eine "Hauptklasse" (TreeDemo) habe, auf der ich ein SplitPane habe (leftp = JTree, rightp = CardLayout).
Bei auswahl eines Knotens im Tree, erhalte ich rechts im CardLayout die entsprechende Ansicht einer Klasse (AirportManage).
In AiportManage hab ich ein JTable, welches sich Daten aus einer DB holt.
Hier habe ich die Möglichkeit einen neuen Airport anzulegen (Klasse "AirportCreate" öffnet sich).

Lege ich nun einen neuen Datensatz an, wird dieser auch in der Datenbank gespeichert, aber wie gesagt, nicht im JTable angezeigt.

Hier mein JTable in AirportManage

// Vector für Spaltenüberschrift mit Werten belegen
    columnNames = new Vector<String>();
        columnNames.add("Int. Kürzel");
        columnNames.add("Name");
        columnNames.add("Stadt");
        columnNames.add("Land");

    // Vector mit allen Airports aus DB
    Vector<Airport> allAirports = admin.getAllAirports();

    // Vector mit nachfolgender for-Schleife mit Werten für JTable belegen
    rowData = new Vector<Vector>();

        for(int i = 0; i<allAirports.size(); i++) {

            Vector<String> TempString = new Vector<String>();

                String internationalId = allAirports.get(i).getInternationalId();

                    TempString.addElement(internationalId);

                String portName = allAirports.get(i).getPortName();

                    TempString.addElement(portName);

                String portCity = allAirports.get(i).getPortCity();

                    TempString.addElement(portCity);

                String portCountry = allAirports.get(i).getPortCountry();

                    TempString.addElement(portCountry);

            rowData.addElement(TempString);
        }

    model = new DefaultTableModel( rowData, columnNames );
    
    table = new JTable(model);
    table.setModel(model);
    table.setAutoCreateRowSorter(true);
//    table.setAutoResizeMode( JTable.AUTO_RESIZE_OFF );
    table.getColumnModel().getColumn(0).setPreferredWidth(10);
    table.getColumnModel().getColumn(1).setPreferredWidth(150);
    table.getColumnModel().getColumn(2).setPreferredWidth(100);
    table.getColumnModel().getColumn(3).setPreferredWidth(100);
    constraints.gridx = 0;
    constraints.gridy = 2;
    constraints.gridwidth = 3;
//    constraints.fill = GridBagConstraints.HORIZONTAL;
//    constraints.weighty = 1.0;
    table.setPreferredScrollableViewportSize(new Dimension(550,200));
    table.setVisible(true);

in Airport habe ich es im ActionListener folgendermaßen versucht

   button_Anlegen.addActionListener(new ActionListener() {
            @Override
    public void actionPerformed(ActionEvent e){
                    try {
                        admin.createAirport(textfield2.getText(), textfield1.getText(), textfield3.getText(), textfield4.getText());

                        DefaultTableModel model = new DefaultTableModel(am.rowData,am.columnNames);
                        am.table.setModel(model);
                        model.fireTableDataChanged();

                        ((CardLayout) td.rightp.getLayout()).show(td.rightp, "AirportM");
                       
                    } catch (RemoteException ex) {
                        Logger.getLogger(AirportCreate.class.getName()).log(Level.SEVERE, null, ex);
                    }
        }
        
    });

So klappt es aber leider nicht.
Habt ihr eine Idee, wie ich es hinkriegen kann, dass sich die Tabelle korrekt aktualisiert/refreshed?

 

[bERt0r]

Mir fällt nichts augenscheinliches auf, ein Schuss ins Blaue: liegt dein Table in einer ScrollPane?

 

[drei1padsvb]

Hi und danke für deine Antwort.

Und ja. Mein Table liegt in einer ScrollPane.

jp.add( new JScrollPane(table), constraints);

 

[bERt0r]

Ah, probiers mal mit fireTableRowsInserted()

 

[drei1padsvb]

Bei fireTableRowsInserted() weiß ich leider nicht, was in die Klammern kommt.
Ist es so, dass dort die erste und letzte Reihe, des Models reinkommt?
also:

model.fireTableRowsInserted(0, model.getRowCount()-1);

 

[bERt0r]

Für sowas gibts die Javadoc: AbstractTableModel (Java Platform SE 6)

Aber eigentlich müsste ein DefaultTableModel die Aktualisierung durchführen wenn du addRow ausführst, auch ohne fire irgendwas.
Bei nochmaligem lesen deins codes stellt sich mir die Frage, wo fügst du eigentlich den neuen Datensatz zum Model hinzu? Ich hab dir ein KSKB gemacht, wie sowas aussehen sollte:

import java.awt.BorderLayout;
import java.awt.EventQueue;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.border.EmptyBorder;
import javax.swing.table.DefaultTableModel;


public class TableRowsInserter extends JFrame {

	private JPanel contentPane;
	private JTable table;
	private DefaultTableModel model;
	
	/**
	 * Launch the application.
	 */
	public static void main(String[] args) {
		EventQueue.invokeLater(new Runnable() {
			public void run() {
				try {
					TableRowsInserter frame = new TableRowsInserter();
					frame.setVisible(true);
				} catch (Exception e) {
					e.printStackTrace();
				}
			}
		});
	}

	/**
	 * Create the frame.
	 */
	public TableRowsInserter() {
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(100, 100, 450, 300);
		contentPane = new JPanel();
		contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
		contentPane.setLayout(new BorderLayout(0, 0));
		setContentPane(contentPane);
		
		JScrollPane scrollPane = new JScrollPane();
		contentPane.add(scrollPane, BorderLayout.CENTER);
		
		
		model=new DefaultTableModel(
			new Object[][] {
				{"1", "A"},
				{"2", "B"},
				{"3", "C"},
				{"4", "D"},
				{"5", "E"},
			},
			new String[] {
				"New column", "New column"
			}
		);
		table = new JTable(model);
		scrollPane.setViewportView(table);
		
		JButton btnInsertRows = new JButton("Insert 5 Rows");
		btnInsertRows.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent arg0) 
			{
				Object[] row=new Object[2];
				row[0]="0";
				row[1]="X";
				for(int i=0;i<5;i++)
				{
					TableRowsInserter.this.model.addRow(row);
				}	
			}
		});
		contentPane.add(btnInsertRows, BorderLayout.SOUTH);
	}
}

 

[drei1padsvb]

Tut mir leid, aber ich krieg das grad einfach nicht hin.

Also die Tabelle wird per Vectoren befüllt.

Und in die Datenbank werden die neuen Daten mit folgender Methode befüllt.

admin.createAirport(textfield2.getText(), textfield1.getText(), textfield3.getText(), textfield4.getText());

Wenn ich das Programm schließe und wieder öffne, werden die neuen Daten angezeigt.

 

[SlaterB]

dein Code ist nicht ausreichend um den Fehler zu erkennen, die geposteten Anteile sollten für sich ausreichen
(auch wenn ich diesmal darauf verzichte eine funktionierende Version zusammenzukratzen)

du setzt ein neues Model, damit erübrigen sich alle fire-Aufrufe,

meine Vermutung nach einer ziemlich einfachen Ursache wäre, dass du das neue Model in einer Table setzt, welches gar nicht in der GUI zu sehen ist,
weil 'am' nicht das ist, was es sein soll, es gibts zwei JFrames oder irgendwas anderes ist doppelt

etwas deutlicher wäre an der Stelle
> am.table.setModel(model);
der Table ein komplett anderes Model mit nur einer Spalte zu geben, ändert das auch nichts?
versuche am.table aus dem jeweiligen Layout herauszunehmen, versuche irgendeine andere Komponente neu einzufügen,
versuche in ein Textfeld den Text zu ändern oder irgendwas um grundsätzlich die funktionierende GUI festzustellen,

was macht
> ((CardLayout) td.rightp.getLayout()).show(td.rightp, "AirportM");
? funktioniert das?

für außenstehende ist komplett nicht zu beurteilen, ob am.table oder td.rightp irgendwas sinnvolles sind,
idealerweise immer komplett lauffähige möglichst kurze Testprogramme posten