JTable | getValueAt(..) wird zu oft aufgerufen.

Dit_

Bekanntes Mitglied
Hallo,

mir ist aufgefallen dass die Methode
Code:
getValueAt()
in meinem TableModel (erweitert AbstractTableModel) permanent aufgerufen wird. Ist das normal? Die Daten sind konstant (es wird nicht gescrollt), trotzdem wird die Tabelle auf diese weise permanent "refresht".

Hoffe jemand hat einen Tipp für mich 😳

Danke schon mal
 
Was recht häufig vorkommt sind fehlerhafte TableCellRenderer, die eine Endlosschleife auslösen.
Werden in der JTable eigene Renderer genutzt?
 
Ja und zwar so:

Java:
		TableColumn column;
		CenterCellRenderer cR = new CenterCellRenderer();
		int[] sizes = { 26,   180, 	 180, 	110,  70,     70,  70, 	70, 		70 };
		for (int i = 0; i < sizes.length; i++) {
			column = _table.getColumnModel().getColumn(i);
			if (i == 0 || i >= 5) {
				column.setCellRenderer(cR);
			}
			column.setPreferredWidth(sizes[i]);
			column.setResizable(true);
		}


Java:
public class CenterCellRenderer extends DefaultTableCellRenderer {

	private static final long serialVersionUID = 1L;


	public CenterCellRenderer() {
	}

	
	@Override
	public int getHorizontalAlignment() {
		return SwingConstants.CENTER;
	}

}
 
Wenn der Renderer nur aus der einen überschriebenen Methode besteht wird es daran nicht liegen.

Kannst Du hier kompilierbaren Demo Code einstellen, mit dem das Verhalten reproduziert werden kann.
Oft stösst man dabei selbst auf die Ursache des Problems.
 
ich zeige erstmal TableModel

Java:
public abstract class ComplexTableModel<T extends Comparable<T> & SearchableObject> extends AbstractTableModel {
	
	private static final long serialVersionUID = 1L;
	protected final String[]  header;
	protected List<T>         data;
	
	// Suche
	protected String          searchingWord    = "";
	protected int             foundedRow       = -1;
	protected boolean         sortData         = true;
	
	/**
	 * Konstruktor.
	 * 
	 * @param header
	 *            Header fuer diese Tabelle, Spaltennamen.
	 */
	public ComplexTableModel(String[] header) {
		this.header = header;
		this.data = new ArrayList<T>();
	}
	
	/**
	 * Aktualisiert eine Zeile in der Tabelle.
	 * 
	 * @param row
	 *            Zeile die aktualisiert werden soll.
	 * @param newData
	 *            neue Daten.
	 */
	public void updateRow(int row, T newData) {
		if (row >= 0 && row < data.size()) {
			// FIXME if-bedingung
			data.set(row, newData);
			fireTableRowsUpdated(row, row);
		}
	}
	
	/**
	 * Loescht eine Zeile aus der Tabelle.
	 * 
	 * @param row
	 *            Zeile.
	 */
	public void deleteRow(int row) {
		if (row >= 0 && row < data.size()) {
			// FIXME if-bedingung
			data.remove(row);
			fireTableRowsDeleted(row, row);
		}
	}
	
	/**
	 * Fuegt eine neue Zeile ein.
	 * 
	 * @param newObj
	 */
	public void insertNewRow(T newObj) {
		data.add(newObj);
		if (sortData) {
			Collections.sort(data);
			fireTableDataChanged();
		} else {
			fireTableRowsInserted(data.size() - 1, data.size() - 1);
		}
	}
	
	/**
	 * Setzt die Daten.
	 * 
	 * @param newdata
	 *            Neue Daten die angezeigt werden sollen.
	 */
	public void setData(List<T> newdata) {
		data.clear();
		data.addAll(newdata);
		if (sortData) {
			Collections.sort(data);
		}
		fireTableDataChanged();
	}
	
	/**
	 * Gibt den aktuellen Datensatz als eine Liste.
	 * 
	 * @return Daten der Tabelle.
	 */
	public List<T> getData() {
		return data;
	}
	
	/**
	 * Gibt ein TabellenObjekt zurueck.
	 * 
	 * @param index
	 *            Zeile
	 * @return Ein Objekt einer Zeile.
	 */
	public final T getObjectAt(int index) {
		if (index < 0 || index >= data.size()) {
			return null;
		}
		return data.get(index);
	}
	
	/**
	 * Loescht alles Daten aus der Tabelle.
	 */
	public void clear() {
		this.data.clear();
		fireTableDataChanged();
	}
	
	@Override
	public int getColumnCount() {
		return header.length;
	}
	
	@Override
	public String getColumnName(int column) {
		return header[column];
	}
	
	@Override
	public int getRowCount() {
		return data.size();
	}
	
	@Override
	public boolean isCellEditable(int rowIndex, int columnIndex) {
		return false;
	}
	
	/**
	 * Dursucht die Tabelle und gibt die Zeile in der der gesuchte Begriff
	 * gefunden wurde zurueck.
	 * 
	 * @param what
	 *            Suchbegriff
	 * @return Zeilennummer
	 */
	public int findRow(String... what) {
		String sTerm = what[0];
		int row = -1;
		int from = -1;
		
		if (!searchingWord.equals(sTerm)) {
			searchingWord = sTerm;
			foundedRow = 0;
			from = 0;
		} else {
			from = foundedRow + 1;
		}
		
		for (int i = from; i < data.size(); i++) {
			if (data.get(i).contains(what)) {
				row = i;
				break;
			}
		}
		
		if (row >= 0) {
			foundedRow = row;
		} else {
			searchingWord = "";
			foundedRow = -1;
		}
		return foundedRow;
	}
	
	public void filterByKey(String key) {
		//
	}
}

Es wäre leider viel zu viel Code um alles hier zu posten. Vielleicht ist der Fehler hier drin.
Habe 2 Arten von Tablemodel. SimpleTableModel und ComplexTableModel s.o. Der Fehler betrifft nur die Tabellen die das ComplexTableModel verwenden. :bahnhof:
 
hab noch mal geprueft die Methode
Code:
 getRowCount()
wird ebenfalls permanent aufgerufen...
 
Mach mal ein sysout in deine eigenen Methoden. Vllt rufst du irgendeine Methode ständig auf - das führt dann zu einem fire() Aufruf, und dadurch werden dann diverse Methode wie getRowCount() usw aufgerufen.
 
Mit Hilfe dieser einer Fabrik Methode erstelle ich alle Tabellen im Programm:
Java:
public class JTableFactory {
	
	
	
	
	public static JTable createTable(AbstractTableModel model, Color bgrColor,
			EventListener eL) {
		JTable table = new JTable(model);
		table.setAutoResizeMode(JTable.AUTO_RESIZE_SUBSEQUENT_COLUMNS); 
		table.setBackground(bgrColor);
		table.setShowHorizontalLines(false);
		table.setShowVerticalLines(false);
		
		table.setRowSelectionAllowed(true);
		table.setAutoCreateRowSorter(false);
		table.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
		//new TableRowSorter<TableModel>(model)
		table.setRowSorter(null);
		
		if (eL != null) {
			if (eL instanceof KeyListener) {
				table.addKeyListener((KeyListener) eL);
			}
			if (eL instanceof MouseListener) {
				table.addMouseListener((MouseListener) eL);
			}
			if (eL instanceof FocusListener) {
				table.addFocusListener((FocusListener) eL);
			}
			
			if((eL instanceof ListSelectionListener)){
			    ListSelectionListener l = (ListSelectionListener) eL;
			    table.getSelectionModel().addListSelectionListener(l);
			    table.getColumnModel().getSelectionModel().addListSelectionListener(l);
			}
		}
		return table;
	}
}
 
hmm kann sein dass ich zusätzlich
Code:
TableModelListener
brauche?

habe mir die Methode von AbstractTablemodel angeschaut:

Java:
/**
     * Forwards the given notification event to all
     * <code>TableModelListeners</code> that registered
     * themselves as listeners for this table model.
     *
     * @param e  the event to be forwarded
     *
     * @see #addTableModelListener
     * @see TableModelEvent
     * @see EventListenerList
     */
    public void fireTableChanged(TableModelEvent e) {
	// Guaranteed to return a non-null array
	Object[] listeners = listenerList.getListenerList();
	// Process the listeners last to first, notifying
	// those that are interested in this event
	for (int i = listeners.length-2; i>=0; i-=2) {
	    if (listeners[i]==TableModelListener.class) {
		((TableModelListener)listeners[i+1]).tableChanged(e);
	    }
	}
    }

Der einziger TableListener den ich habe ist (per Default) JTable selbst. JTable wiederum ruft
Code:
fireTableChanged
Methode von AbstractListener. Ich glaube das stimmt was nicht :/ .
 
Unglaublich...
Habe versucht Programm zu debuggen, gabs JVM Absturz "A fatal error has been detected by the Java Runtime Environment ...".

getValueAt wird mehrere hundertemale pro sekunde aufgerufen und zwar nur sichtbarer Teil der Tabelle...
 
Dann nutz einfach die "Quick and Dirty" Variante und hau ein

Java:
Thread.dumpStack();

in die entsprechenden Methoden rein, bei denen du sehen möchtest, von wo sie aufgerufen werden. Da deine so häufig aufgerufen wird, würde ich mir jede Methode für sich anschauen. Dazu kannst du deine Anwendung kurz nach Programmstart wieder schließen und deinen Konsoleninhalt auswerten. Dadurch solltest du den Übeltäter finden können, der die Methode aufruft.
 

Neue Themen


Zurück
Oben