fireTableDataChanged klappt nicht

[ricosta]

Hallo zusammen,

ich bin noch ein Anfänger was Java angeht. Hab bislang durch lesen und probieren/verzweifeln gelernt. Jetzt hab ich ein Problem, das ich selber nicht mehr lösen kann. Ich hoffe ihr könnt mir helfen und ich mich verständlich ausdrücken.

1. Ich habe eine JTable und die Daten werden über ein AbstractTableModel gelesen.
Die Daten für das Model kommen aus einer einer Methode getGamePlan(), die wieder auf eine MySQL DB zugreift.

2. Dann mache ich eine Änderung in der DB (umschreiben zweier int Werte) und stoße ein fireTableDataChanged an in der Methode updateGamePlan();
Das klappt noch wunderbar.

3. Dann mache ich noch eine Änderung in der DB (umschreiben genau der selben int Wert) und rufe wieder updateGamePlan() auf.

Jetzt erhalte ich einen Fehler: ArrayIndexOutOfBounds -1 bei dem Versuch getValueAt(.....)
Nachdem ich mir eine Ausgabe reingebastelt habe, hab ich gesehen das er da versucht mit row = -1 zu arbeiten. Wie kann das sein?

Vermutlich hats jetzt fast keiner Verstanden aber bitte fragt nach oder sagt was ihr an Code braucht um den Fehler finden zu können.

Vielen Dank und Grüsse

TMS_Functions func = new TMS_Functions();
static String[] gamePlanHeader = {"Heim", "Gast", "Ergebnis1", "Ergebnis2", "Gruppe"};
static String[][] gamePlanData;

Methode getGamePlanData():

	private void getGamePlanData(){
		gamePlanData = func.getGamePlan();
	}

Methode getGamePlan():

	public String[][] getGamePlan(){
		numberOfRows = getTableSize("gameplan");

		data = new String[numberOfRows][5];
		
		try {
			db = new TMS_Database();
			ResultSet rs = db.query("select team1, team2, goal1, goal2, group_name from gameplan");

			int i = 0;
			for (; rs.next();){

				for (int j = 1; j <= 5; j++){
					data[i][j-1] = rs.getString(j);
				}

				i++;
			}
			return data;
		} catch (Exception e) {
			logger.error("Konnte Spielplan nicht auslesen");
			return data;
		}
	}

TableModel:

	class modelGamePlan extends AbstractTableModel{
		private static final long serialVersionUID = 1L;

		public int getColumnCount() {
			return gamePlanHeader.length;
		}

		public int getRowCount() {
			return gamePlanData.length;
		}

		public String getColumnName(int col) {
			return gamePlanHeader[col];
		}

		public Object getValueAt(int row, int col) {
			logger.info("row: " + row);
			logger.info("col: " +col);
			return gamePlanData[row][col];
		}

		public boolean isCellEditable(int row, int col) {
			return false;
		}
	}

Und zu guter letzt noch der Fehler:

Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: -1
	at com.tms.TMS_GuiMain$modelGamePlan.getValueAt(TMS_GuiMain.java:287)
	at javax.swing.JTable.getValueAt(JTable.java:2686)
	at com.tms.TMS_GuiMain$SelectionListener.valueChanged(TMS_GuiMain.java:328)
	at javax.swing.DefaultListSelectionModel.fireValueChanged(DefaultListSelectionModel.java:167)
	at javax.swing.DefaultListSelectionModel.fireValueChanged(DefaultListSelectionModel.java:147)
	at javax.swing.DefaultListSelectionModel.fireValueChanged(DefaultListSelectionModel.java:194)
	at javax.swing.DefaultListSelectionModel.changeSelection(DefaultListSelectionModel.java:388)
	at javax.swing.DefaultListSelectionModel.changeSelection(DefaultListSelectionModel.java:398)
	at javax.swing.DefaultListSelectionModel.removeSelectionIntervalImpl(DefaultListSelectionModel.java:559)
	at javax.swing.DefaultListSelectionModel.clearSelection(DefaultListSelectionModel.java:403)
	at javax.swing.JTable.clearSelection(JTable.java:2080)
	at javax.swing.JTable.clearSelectionAndLeadAnchor(JTable.java:2088)
	at javax.swing.JTable.tableChanged(JTable.java:4433)
	at javax.swing.table.AbstractTableModel.fireTableChanged(AbstractTableModel.java:280)
	at javax.swing.table.AbstractTableModel.fireTableDataChanged(AbstractTableModel.java:182)
	at com.tms.TMS_GuiMain.updateGamePlan(TMS_GuiMain.java:260)
	at com.tms.TMS_GuiMain$ButtonListener.actionPerformed(TMS_GuiMain.java:395)
	at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2028)
	at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2351)
	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:6352)
	at javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
	at java.awt.Component.processEvent(Component.java:6117)
	at java.awt.Container.processEvent(Container.java:2085)
	at java.awt.Component.dispatchEventImpl(Component.java:4714)
	at java.awt.Container.dispatchEventImpl(Container.java:2143)
	at java.awt.Component.dispatchEvent(Component.java:4544)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4621)
	at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4282)
	at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4212)
	at java.awt.Container.dispatchEventImpl(Container.java:2129)
	at java.awt.Window.dispatchEventImpl(Window.java:2478)
	at java.awt.Component.dispatchEvent(Component.java:4544)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:635)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:296)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:211)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:201)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:196)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:188)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)

 

[SlaterB]

der getValue()-Aufruf stammt anscheindend von deinem Code:
> at com.tms.TMS_GuiMain$SelectionListener.valueChanged(TMS_GuiMain.java:328)

wäre es nicht hochinteressant, diese Zeile 328 bzw. den ganzen Listener zu posten?
allgemein muss man damit rechnen dass SelectionEvents mit row -1 für Deselektion oder wer weiß was komisches kommt,
prüfe die row-Indexe die da kommen und frage nicht sofort den Value dazu ab,
also im Model sowieso, falls die JTable sowas komisches anfragt

vor allem aber in deinem SelectionListener als aktuelle Problemursache, wenn man mal den Weg dahin als unbeeinflussbar ansieht

 

[ricosta]

Hallo,

danke erstmal für die Rückmeldung.

Was mich an der Sache so stutzig macht ist, das es für bestimmte aktionen immer funktioniert und für genau den einen Sachverhalt eben nicht.

Also, hier ist der SelectionListener. Ich hoffe der hilft bei der Fehlersuche.

	class SelectionListener implements ListSelectionListener {
		JTable table;

		SelectionListener(JTable table) {
			this.table = table;
		}
		public void valueChanged(ListSelectionEvent e) {
			if (e.getSource() == table.getSelectionModel() && table.getRowSelectionAllowed()) {

				updateGroupTable(table.getValueAt(table.getSelectedRow(),4).toString());

			} else if (e.getSource() == table.getColumnModel().getSelectionModel()
					&& table.getColumnSelectionAllowed()) {

				logger.warn("Unexpected Failure");
			}
		}
	}

Bei updateGroupTable ist die Zeile 328.

Vielen Dank und Grüsse

 

[MySelV]

Hi,

doch noch den Fehler gefunden:
In deiner Methode getValueAt() wird folgendes zurückgegeben:

gamePlanData[row][col];

Das Problem ist also: Die selectedRow bei deinem Aufruf von updateGroupTable ist "-1" - es ist quasi nix selektiert.

Grüße

 

[SlaterB]

was gibst da noch zu suchen, du hast
> table.getSelectedRow()
und das kann eben genau -1 sein, wenn gerade nix selektiert ist, auch in einer kurzen Zeitspanne zwischen zwei Selektionen,
dann kommt vielleicht wenig später noch ein Event und du kannst beim ersten ruhigen Gewissens nichts machen

 

[ricosta]

was gibst da noch zu suchen...

Das sagst du so einfach. Ich habs noch nicht kapiert warum der Fehler auftritt.

Vielleicht kurz zum ablauf. Ich selektieren ein Zeile in der Tabelle, und klicke dann außerhalb der Tabelle auf einen Button der zuerst die Änderung in der DB macht und dann das Update aufruft.

Gilt beim klick auf den Button die Zeile noch als markiert? (Sie ist noch blau hinterlegt)

Grüsse

 

[SlaterB]

deine Beschreibung sagen nicht viel, vielleicht werden beim Update die Daten der Tabelle ausgetauscht,
dann ist ja logisch dass die Selektion nicht konstant bleibt,
aber was spielt das für eine Rolle?

der Listener kann doch ganz unabhängig von dem was drumherum passiert sinnvoll agieren,
das bedeutet dass du prüfst ob table.getSelectedRow() -1 ist und dann nicht das machst was du da machst

 

[ricosta]

Tut mir leid das die Beschreibungen etwas dürftig sind, aber ich hab alles gegeben.

Hab mein Problem jetzt durch das abfangen von row = -1 gelöst.

Danke euch beiden und schönen Abend.