JTable mit wechselnden Spalten - sehr Langsam

[Milo]

Hallo,

ich stelle in einer JTable (quadratische) Matrizen dar. Diese können größere Dimensionen annehmen. Wenn ich eine Matrix mit ~500.000 Zeilen und Spalten das erste Mal lade, geht es verhältnismäßig fix. Wenn ich anschließend in die selbe Tabelle eine 5x5 Matrix lade, dauert das ewig. Das Problem scheint am Entfernen bzw. Hinzufügen der Spalten zu liegen (fireTableStructureChanged()). Ich habe diesen Artikel bereits gefunden aber das Abräumen wird damit nicht zwingend besser. Gibt es hier einen Trick, die Tabellengröße einigermaßen schnell zu ändern? Sofern es wichtig ist: Ich leite von AbstractTableModel mein Model ab.

Schöne Grüße
Micha

 

[André Uhres]

fireTableStructureChanged()

Hallo Micha,

hast Du schon versucht, mit JTable#setModel() einfach ein neues TableModel zu setzen?

Gruß,
André

 

[Milo]

Hi André,

hast Du schon versucht, mit JTable#setModel() einfach ein neues TableModel zu setzen?

sorry, ja das habe ich auch probiert - ohne Erfolg. War auch mein erster Gedanke, bevor ich mich auf die Suche nach einer Alternativen gemacht habe. ;-)

Schöne Grüße
Micha

 

[Milo]

Hi,

hier mal ein Beispiel. Ein Dummy für eine Matrixklasse

package spielwiese.table;


public class UpperSymmSparseMatrix {
	int size;
	public UpperSymmSparseMatrix(int size) {
		this.size = size;
	}
		
	public boolean isSquare() {
		return true;
	}
	
	public double get(int row, int column) {
		return (row == column) ? 1:0;
	}
	
	public int numColumns() {
		return size;
	}
	
	public int numRows() {
		return size;
	}
}

Das Modell

package spielwiese.table;

import javax.swing.table.AbstractTableModel;


public class UpperSymmMatrixModel extends AbstractTableModel {
	private static final long serialVersionUID = -7577195827050146891L;
	private UpperSymmSparseMatrix matrix = null;
	private int dim;
	public UpperSymmMatrixModel(int dim) {
		this.setDimension(dim);
	}
	
	// Dimension der Punkte: 1,2 oder 3
	public void setDimension(int dim) {
		this.matrix = new UpperSymmSparseMatrix(dim);
		this.dim = dim;
		this.fireTableStructureChanged();
	}
	
	public int getDimension() {
		return this.dim;
	}
	
	@Override
	public int getColumnCount() {
		return matrix.numColumns();
	}
	
	public void setMatrix(UpperSymmSparseMatrix matrix) {
		int oldColumns = this.matrix.numColumns();
		int newColumns = matrix.numColumns();
		boolean hasEqualSize = matrix.isSquare() && oldColumns == newColumns;
		
		this.matrix = matrix;

		if (hasEqualSize)
			this.fireTableDataChanged();
		else {
//			[url=http://www.chka.de/swing/table/columns.html]JTable and TableModel: Dynamic column changes[/url]
//			// Spalten hinzufuegen
//			this.table.setAutoCreateColumnsFromModel(false);
//			TableColumnModel columnModel = this.table.getColumnModel();
//			if (newColumns > oldColumns) {
//				for (int columnIndex = oldColumns; columnIndex < newColumns; columnIndex++) {
//					TableColumn c = new TableColumn(columnIndex);
//					c.setHeaderValue(this.getColumnName(columnIndex));
//					columnModel.addColumn(c);
//				}
//			}
//			else if (newColumns < oldColumns) {
//				for (int column = oldColumns-1; column >= newColumns; column--) {
//					int index = this.table.convertColumnIndexToView(column);
//					if (index != -1)
//						columnModel.removeColumn(columnModel.getColumn(index));
//				}
//				
//			}
//			this.table.setAutoCreateColumnsFromModel(true);
//			this.fireTableDataChanged();
			
			this.fireTableStructureChanged();
		}	
	}

	@Override
	public int getRowCount() {
		return matrix.numRows();
	}

	@Override
	public Object getValueAt(int row, int column) {
		return this.matrix.get(row, column);
	}

	@Override
	public Class<?> getColumnClass(int column) {
		return Double.class;
	}
}

und ein Frame

package spielwiese.table;

import java.awt.BorderLayout;
import java.awt.Container;
import java.awt.Dimension;
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.WindowConstants;

public class TableDemo extends JFrame {
	
	private static final long serialVersionUID = 1L;


	private class ResizeMatrix implements ActionListener {

		@Override
		public void actionPerformed(ActionEvent e) {
			
			UpperSymmSparseMatrix matrix = new UpperSymmSparseMatrix(size);
			
			if (e.getSource() == button1) {
				System.out.println("Behalte Modell und setze Matrix " + size);
				model.setMatrix(matrix);
			}
			
			else if (e.getSource() == button2) {
				System.out.println("Erzeuge neues Modell und setze Matrix " + size);
				model = new UpperSymmMatrixModel(size);
				table.setModel(model);
				model.setMatrix(matrix);
			}
			
			size = size > 100 ? 5 : 700000;
		}
		
	}
	int size = 700;
	int frameWidth = 500;
	int frameHeight = 500;
	UpperSymmMatrixModel model = new UpperSymmMatrixModel(3);
	JTable table = new JTable(new UpperSymmMatrixModel(3));
	JButton button1 = new JButton("Gleiches Modell");
	JButton button2 = new JButton("Neues Modell");
	public TableDemo() {
		ResizeMatrix l = new ResizeMatrix();
		super.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
	    this.setPreferredSize( new Dimension(frameWidth,frameHeight) );
	    this.setLocationRelativeTo(null);
	    
	    table = new JTable(model);
	    table.setAutoResizeMode(JTable.AUTO_RESIZE_OFF);
	    table.getTableHeader().setReorderingAllowed(false);
	    
	    JPanel panel = new JPanel();
	    button1.addActionListener(l);
	    button2.addActionListener(l);
	    panel.add(button1);
	    panel.add(button2);
	    Container contentPane = getContentPane();
	    	    
	    contentPane.add(new JScrollPane(table), BorderLayout.CENTER);
	    contentPane.add(panel, BorderLayout.SOUTH);
	    
	    this.pack();
	    this.setResizable(true);
	    this.setVisible(true);
	}
	 
	
	public static void main(String[] args) {
		new TableDemo();
	}
}

Vielleicht sieht man es so besser...

Gruß Micha

 

[André Uhres]

Wenn ich eine Matrix mit ~500.000 Zeilen und Spalten das erste Mal lade, geht es verhältnismäßig fix.

Dann würde ich einfach alles neu erzeugen, Tabelle und Modell, dann ist es ja immer das erste Mal, etwa so:

...
public class TableDemo extends JFrame {
   private final JScrollPane scrollpane;
...
            else if (e.getSource() == button2) {
                System.out.println("Erzeuge neue Tabelle und neues Modell" + size);
                model = new UpperSymmMatrixModel(size);
//                table.setModel(model);
//                model.setMatrix(matrix);
                reloadTable();
            }
...
   private void reloadTable()
   {
      table = new JTable(model);
      table.setAutoResizeMode(JTable.AUTO_RESIZE_OFF);
      table.getTableHeader().setReorderingAllowed(false);
      scrollpane.getViewport().setView(table);
   }
...
    JButton button2 = new JButton("Neue Tabelle und Neues Modell");
...
        contentPane.add(scrollpane = new JScrollPane(table), BorderLayout.CENTER);
...
}

Damit umgehen wir das ineffektive Neuzeichnen der Tabelle durch fireTableStructureChanged/setModel, und das Neuzeichnen müsste jetzt wesentlich schneller gehen 😉.

Gruß,
André

 

[Milo]

Hi André,

Dann würde ich einfach alles neu erzeugen, Tabelle und Modell, dann ist es ja immer das erste Mal

Stimmt! Das klingt logisch und es geht genau so, wie ich es mir erhofft hatte. Besten Dank für Deine Hilfe!

Schönen Sonntag
Micha