Swing JTable updaten

[Khal Drogo]

N'Abend,

ich habe mich heute mal ein wenig mit Swing beschäftigt und frage mich jetzt, ob es nicht eine schönere Lösung für das folgende Problem gibt: Ich habe eine Datenstruktur (es folgt Pseudocode):

class Data {
  private String s1;
  private String s2;
  // ...
}

Diese wird in einer anderen Klasse in Form einer ArrayList verwaltet:

class Manager {
  public static final String[] header = new String[] {
    "S1", "S2" // ...
  }
  private static ArrayList<Data> dataList;
  public static String[][] dataListToStringArray() {
    // @return Die komplette Liste in Form eines 2D-Arrays
  }
}

Und jetzt kommt der Knackpunkt:

class Panel extends JPanel {
  private JTable table;
  private JScrollPane pane;

  public Panel() {
    table = new JTable(Manager.dataListToStringArray(), Manager.header);
    pane = new JScrollPane(table);
    this.add(pane);
  }

  // Es geht um diese Stelle!
  public void update(Manager m) {
    this.remove(pane);
    table = new JTable(Manager.dataListToStringArray(), Manager.header);
    pane = new JScrollPane(table);
    this.add(pane);
    this.revalidate();
  }
}

Ich habe jetzt nach Möglichkeit wirklich nur das Wichtigste heraussondiert. Ich hoffe, dass dadurch möglichst abstrakt klar wird, worum es geht. Das Panel wird dann entsprechend als Komponente in ein JFrame eingesetzt.

1. Gibt es eine elegantere Möglichkeit, die Tabelle zu aktualisieren, wenn die ArrayList erweitert wird? Da die Erweiterung per Buttondruck erfolgt, habe ich in der actionPerformed() des ActionListeners des Buttons momentan panel.update() stehen.
2. Ist die allgemeine Struktur des Programmes in Ordnung so, oder mache ich etwas grundlegendes falsch? Falls euch da was auffällt, macht mich bitte darauf aufmerksam!

Mit freundlichen Grüßen
Xelsarion

EDIT: Zur Struktur, die Klasse Panel liegt mit anderen Klassen zusammen in einem Package gui, die Klasse Manager im Package manager und die Klasse Data schließlich im Package data. In meinem tatsächlichen Programm beziehen sich die Klassennamen natürlich etwas mehr auf ihre Funktionalität.

 

[JBelarbi]

Also versuch mal mit Table refreshing

((AbstractTableModel)table.getModel()).fireTableCellUpdated();

 

[Khal Drogo]

Das habe ich bei meiner Recherche via Google auch gefunden, allerdings scheine ich es nicht richtig zu benutzen, d. h. die Tabelle ändert sich nicht, obwohl die Liste definitiv erweitert wurde (das weiß ich, weil ich die Daten bei Programmende in einer Textdatei speicher). Muss ich noch etwas anderes machen, als die Daten in die ArrayList hinzuzufügen und die Methode aufzurufen?

Mit freundlichen Grüßen
Xelsarion

 

[JBelarbi]

hast du auch mit dem TableModelListener probiert?

table.getModel().addTableModelListener(new TableModelListener() {
        @Override
        public void tableChanged(TableModelEvent e) {
           TableModel myTableModel=new DefaultTableModel(Manager.dataListToStringArray(), 
Manager.header);
           table= new JTable(myTableModel);
        }
     
    });

http://www.java2s.com/Tutorial/Java/0240__Swing/ListeningtoJTableEventswithaTableModelListener.htm

 

[JBelarbi]

Also ich habe es mit TableModel probiert und die Daten sind sichtbar.
ich habe es auch mit ein Thread getestet, der nach 5 sec in der Tabelle neue Daten einfügt.

                String[] colNames = {"Name", "Surname"};
                String[][] data = {{"Nation", "Chirara"}, {"Bill", "Thompson"}};
                frame.setSize(300, 200);
                frame.setVisible(true);

                DefaultTableModel model = new DefaultTableModel(data, colNames);
                frame.add( new JTable(model));
                Thread t = new Thread(new Runnable() {

                    @Override
                    public void run() {
                        try {
                           
                            Thread.sleep(5000);
                            System.out.println("setValue2");
                            model.addRow(new Object[]{"v1", "v2"});
                        } catch (Exception ex) {
                           
                        }
                       
                    }
                });
                t.start();

 

[Khal Drogo]

Also erstmal vielen Dank für die Hilfe und Mühe!
Aufgrund deines ersten Vorschlags habe ich überhaupt mal manuell ein DefaultTableModel erstellt und meine Tabelle darüber konstruiert:

private DefaultTableModel model;
private JTable table;
private JScrollPane pane;

public Panel() {
  model = new DefaultTableModel(Manager.dataListToStringArray(), Manager.header);
  table = new JTable(model);
  pane = new JScrollPane(table);
  this.add(pane);
}

Mit Threads kenn ich mich leider nur sehr sehr oberflächlich aus, allerdings schien mir das an dieser Stelle aber auch nicht die richtige Lösung zu sein, da ich ja nicht ständig nach neuen Daten suchen will oder sie hinzufügen will, sondern wirklich nur bei Betätigung eines dafür vorgesehenen JButtons. Da dieser JButton allerdings durch die Struktur des Programmes von der Tabelle weiß und Zugriff darauf hat, habe ich kurzerhand folgendes gemacht:

// Das ist in einem anderen Panel!
add = new JButton("Hinzufügen");
add.addActionListener(new ActionListener() {
  @Override
  public void actionPerformed(ActionEvent e) {
    Data d = new Data(param1, param2, ...); // Parameter werden aus verschiedenen JTextFields ausgelesen
    Manager.add(d); // Fügt Element d ans Ende der ArrayList
    panel.getModel().addRow(d.toStringArray());
  }
});

Das funktioniert jetzt wunderbar, scheint mir an sich auch schon mal eine sauberere Lösung zu sein. Jetzt frage ich mich, ist es schlechter Stil, dass mein "Kontroll-Panel" vom "Anzeige-Panel" weiß (ich habe eine Referenz private Panel panel im Kontroll-Panel, die per Konstruktor übergeben wird), wenn ja, wie kann ich es besser lösen? Eventuell über die gemeinsame Superklasse?
Desweiteren: Ist das jetzt schon zu viel Funktionalität in den GUI-Klassen? Kann ich das auslagern, ohne mit Referenzen um mich schmeißen zu müssen?

Mit freundlichen Grüßen
Xelsarion

EDIT: Ich habe/hatte zwar ein private Panel panel in meinem Kontroll-Panel, das wurde auch initialisiert, ich habe aber im restlichen Konstruktor nicht mit this.panel, sondern einfach mit panel aus dem Konstruktor gearbeitet, also mein Kontroll-Panel weiß vom Anzeige-Panel, speichert die Referenz allerdings nicht dauerhaft.

 

[Harry Kane]

Du kannst auch einfach nur die TableModel Referenz an das "andere Panel" übergeben.

 

[Khal Drogo]

Du kannst auch einfach nur die TableModel Referenz an das "andere Panel" übergeben.

Und dann eine extra Methode schreiben, statt das im Konstruktor zu regeln?

Mit freundlichen Grüßen
Xelsarion

 

[Harry Kane]

Also ich würds architektonisch doch anders angehen. Zum einen gefällt es mir nicht, wenn Ableitungen von JPanel erzeugt werden, nur um darauf JPanel Methoden aufzurufen oder um per getter an Komponenten auf dem JPanel ranzukommen. Zum zweiten finde ich es schlecht, eine Gui auf mehrere JPanel Unterklassen zu verteilen, weil diese Unterklassen im Allgemeinen doch wieder Referenzen aufeinander benötigen.

Für eleganter würde ich folgendes Vorgehen halten: schreibe dir einen ActionListener, dem du Referenzen zu JTextFields und zu einem DefaultTableModel verpassen kannst. In der actionPerformed liest du die JTextFields aus, und schreibst die Daten als neue row in das TableModel.
So in etwa:

public class TableUpdater implements ActionListener{
   private ArrayList<JTextFields> felder;
   private DefaultTableModel model;
   public TableUpdater(DefaultTableModel model){
     this.model = model;
     this.felder = new ArrayList<JTextFields>();
   }
   public void actionPerformed(ActionEvent ae){
     Vector<String> row = new Vector();
     for(JTextField tf: felder){
       row.add(tf.getText());
     }
     model.addRow(row);
   }
   public void registerInput(JTextField jtf){
     felder.add(jtf);
   }
}

Dann kannst du beispielsweise in einem Fenster eine Tabelle anzeigen, in einem anderen Fenster die JTextFields und den Button. Die beiden Fenster müssen dabei nichts voneinander wissen. Es müssen nur das TableModel und die JTextFields beim ActionListener "angemeldet" sein.

 

[Khal Drogo]

@Harry Kane: Vielen Dank für den Hinweis, ich werde mich vermutlich am Wochenende, wenn ich genügend Zeit dafür habe, nochmal dransetzen. Werde dann deinen Vorschlag auf jeden Fall mal ausprobieren.

Mit freundlichen Grüßen
Xelsarion