JTable mit AbstractTableModel ändern funktioniert nicht

[Waldi]

Hallo Leute

Ich habe diese JTable:

private JTable getJTable() {
		if (jTable == null) {
			jTable = new JTable();
			
			jTable.setModel(atm);
			
			jTable.putClientProperty("terminateEditOnFocusLost", Boolean.TRUE); 
			jTable.setAutoResizeMode( JTable.AUTO_RESIZE_OFF ); 
			
			TableColumnModel columnModel = jTable.getColumnModel(); 
			columnModel.getColumn( 0 ).setPreferredWidth( 80 ); 
			columnModel.getColumn( 1 ).setPreferredWidth( 50 ); 
		    columnModel.getColumn( 2 ).setPreferredWidth( 140); 
		    columnModel.getColumn( 3 ).setPreferredWidth( 30 );
			
			TableColumn bezeichnungsColumn = jTable.getColumnModel().getColumn(2);
			bezeichnungsColumn.setCellEditor(new DefaultCellEditor(getJComboBoxBezeichnung()));
			
			//TableColumn idColumn = jTable.getColumnModel().getColumn(3);
						
			JTableHeader header = jTable.getTableHeader(); 
			header.setReorderingAllowed(false);
			
			atm.addTableModelListener(new InteractiveTableModelListener());
			
		}
		return jTable;
	}

Und dieses Model

public class MyTableModel extends AbstractTableModel
{ 
	/**
	 * 
	 */
	private static final long serialVersionUID = -9072279854289269980L;
	
	private Vector data;
	private Vector columNames;
	
	/**
	 */
	public MyTableModel(Vector columNames, Vector data)
	{
		this.columNames = columNames;
		this.data = data;
	} 
	
	
	
	public int getRowCount() {
		// TODO Automatisch erstellter Methoden-Stub
		return data.size();
	}
	public int getColumnCount() {
		// TODO Automatisch erstellter Methoden-Stub
		return columNames.size();
	}
	public Object getValueAt(int rowIndex, int columnIndex) {
		Vector rowVector = (Vector)data.elementAt(rowIndex);
		return rowVector.elementAt(columnIndex);
	}
	
	
	public boolean isCellEditable(int row, int column) 
	{
		if (column < 3) 
		{
			return true;
		}
		else
		{
			return false;
		}
	}
	
	public String getColumnName(int col) { 
		return columNames.elementAt(col).toString(); 
	}
}

Ich füge über normal Text Fields dateien in eine Datenbank ein. Danch aktualisiere ich die Vectoren die die Tabelle bilden(was auch funktioniert) und weise der Tabelle das Modell mit den neuen Vectoren zu. Das mach ich so:

getTableFromDatabase(1);
					getJTable().setModel(atm);
					atm.fireTableDataChanged();

Nur die Tabelle aktualisiert sich nicht was mach ich falsch?? Die oben genannte Aktion läuft ab wenn ich auf einen JButton klicke.

weiterhin habe ich noch ein:

public class InteractiveTableModelListener implements TableModelListener 
	{ 
		public void tableChanged(TableModelEvent evt) 
		{ 
			String abfrage = "UPDATE zeltdetail SET schlüssel = ?, anzahl = ?, bezeichnung = ?, zelt = ? WHERE zdid = ?";
			if (evt.getType() == TableModelEvent.UPDATE) 
			{ 
				//int column = evt.getColumn(); 
				int row = evt.getFirstRow(); 
				
				try
				{ 
					PreparedStatement psStmt = DBVerbindung.verbindung.prepareStatement(abfrage);
					
					for (int column = 0; column < getJTable().getColumnCount(); column++) 
					{ 
						Object value = getJTable().getValueAt(row, column); 
						
						if(value==null)
						{ 
							value = ""; 
						}
						if (column == 0) 
						{
							psStmt.setString(column+1,value.toString());
						}
						if (column == 1) 
						{
							psStmt.setInt(column+1,Integer.parseInt(value.toString()));
						}
						if (column == 2) 
						{
							int bezID = z.getZeltDetailBezeichnungID(value.toString());
							psStmt.setInt(column+1,bezID);
						}
					}
					
					psStmt.setInt(4,z.getIID());
					psStmt.setInt(5,Integer.parseInt(getJTable().getValueAt(row,3).toString()));
					psStmt.executeUpdate(); 
					psStmt.close(); 
				} 
				catch(SQLException my_sqlex) 
				{
					System.out.println("**************************************************************************");
					System.out.println("ERROR: Datenbankfehler: " + my_sqlex.getMessage());
					System.out.println("--------------------------------------------------------------------------");
					String sFehlermeldungSQL = my_sqlex.getMessage();
					JOptionPane.showMessageDialog (null,sFehlermeldungSQL);
					my_sqlex.printStackTrace();
					System.out.println("**************************************************************************");
				}
				catch(Exception my_ex) 
				{
					System.out.println("**************************************************************************");
					System.out.println("ERROR: Sonstiger Fehler: " + my_ex.getMessage());
					System.out.println("--------------------------------------------------------------------------");
					my_ex.printStackTrace();
					String sFehlermeldung = my_ex.getMessage();
					JOptionPane.showMessageDialog (null,sFehlermeldung);
					System.out.println("**************************************************************************");
				}
			} 
		}
		
	}

Dieser hat mal funktioniert, jetzt macht er gar nichts mehr.

Bitte verweist mich nicht auf die FAQ. Ich hab die schon so oft und runter gekaut. Vielleicht bin ich zu blöd dafür aber ich kann da kein Hinweis auf meine Probleme entecken.

Hoffe ich habe alles gut beschrieben.

 

[SlaterB]

zumindest musst du erstmal ein komplett lauffähiges Beispiel zusammenbasteln,
ohne DB natürlich,
nur mit nur einer Spalte
Inhalt eine Tabelle ist 'a', Inhalt andere Tabelle ist 'b'

und solchen Firlefanz wie Spaltenbreiten oder

public boolean isCellEditable(int row, int column) 
   { 
      if (column < 3) 
      { 
         return true; 
      } 
      else 
      { 
         return false; 
      } 
   }

oder

 System.out.println("**************************************************************************"); 
               System.out.println("ERROR: Sonstiger Fehler: " + my_ex.getMessage());

sofort rausschmeißen,

wenn du dann ein kleines feines Programm aus wenigen Klassen zusammengestellt hast mit einer Main und noch ner einfachen GUI drumherum,
dann kann jeder auch mal auf Run klicken und den Fehler beobachten + schnell lösen

 

[Waldi]

Wenn dir das weiterhilft kann ich das machen. Allerdings funktioniert ja alles. Also die tabelle wird richtig angezeigt nur eben die beschriebenen Funktionen wie ändern und neu laden funktionieren nicht. Ist das nach deiner sicht richtig so wie es jetzt ist??

 

[SlaterB]

sagen wir mal ich bin zu dumm, den Fehler mit bloßem Auge zu sehen,
allzu genau kenne ich mich damit aber nicht aus,
daher würde ich mir ein Beispiel schreiben was funktioniert und mit etwas Glück ist der Fehler dann in deinen Codefragmenten enthalten,
ist aber viel Arbeit..

kannst ja zunächt noch auf jemanden warten, dem es direkt einfällt

 

[Waldi]

So Leute

Wie slaterB schon vorgeschlagen hat habe ich eine Testaplikation erstellen. Jetzt kommt der Hammer: Ich hab vom Prinzip her alles genau so gemacht wie in meinem richtigen Programm und es hat funktioniert. Mit dem Debugger hab ich nun in meinem richtigen Programm nachgeschaut ob der Daten Vector überhaupt aktualisiert wird - wird er. Also denke ich mal das dass neu zuweisen des Models nid funktioniert. Mach ich vielleicht da was falsch?? Kurz zum Ablauf

Vector mit Daten1
Speichern in der Datenbank
Vector aktualisieren -> Vector mit Daten 2
Model nochmals zuweisen
model.fireTableDataChanged();

Hoffe nun ihr könnt mir nun einen Sprung weiterhelfen.

 

[Waldi]

So das Problem mit der Aktualisierung hätte ich gelöst

	private void getTableFromDatabase(int zeltID)
	{
		this.gtfd = new GetTableFromDatabase(zeltID);
		this.atm = new MyTableModel(gtfd.getColumn(),gtfd.getData());
	}

die erste Zeile im Rumpf hatte ich nicht drin, somit wurden die Daten nicht ans Modell weitergegeben. Nun habe ich allerdings das Problem das nach

atm.fireTableDataChanged();

mir die ColumnComboBox und die Breite der Spalten verloren geht. Warum?? Ich dachte Model und View sind voneinander getrennt. Somit müsste das eine doch nichts mit dem anderen zu tun haben oder??

 

[Guest]

Da ich ja im letzten Post ein bisschen Mus geschrieben habe, bezüglich Daten und Anzeige, kam mir nun die Idee man könnte ja alle sachen wie Spaltenbreite, ColumnComboBox usw. ins Model packen geht sowas überhaupt?? Dann hätte sich nämlich mein zweites Problem auch schon gelöst.

 

[SlaterB]

zu ComoboBox weiß ich nix,

bei den Spaltenbreiten hilft es vielleicht (und ist allgemein zu empfehlen),
nur ein Model zu verwenden und dessen Daten zu verändern statt ein neues Model zu setzen

 

[Waldi]

Auf diese idee bin ich auch schon gekommen, allerdings hab ich dann das Problem das sich die Tabelle nicht mehr aktualisiert. Es ist zum verückt werden. Was mach ich denn nur falsch. Bei allen schein das zu funktionieren nur bei mir nicht.

 

[HLX]

Wo ist die methode setValueAt(...) vom Tablemodel? Die hast du hoffentlich auch implementiert...

 

[Waldi]

Diese habe ich im Moment nicht drin. Dürfte aber doch auch nicht so schlimm sein oder? Änderungen die direkt in die Datenbank weitergegeben werden sollen sind nicht sooo wichtig für mich. Für mich ist jetzt eigentlich wichtiger das der Cell Editor und die Spalten so bleiben wie sie sind. Aber vielen Dank für den Hinweis so kann ich an meinem 3. Problem schon weiterfeilen. Ein kleiner Lichtblick am schwarzen Himmel ;-)

 

[HLX]

Diese habe ich im Moment nicht drin. Dürfte aber doch auch nicht so schlimm sein oder?

Wie sonst erfahren denn die Vektoren, dass sie sich verändert haben?

 

[Waldi]

guck ich hab das mir so gedacht (Ich machs vielleicht falsch - weis ich nicht).

Vector 1 -> mit Daten aus Datenbank geladen
Datensatz einfügen
Hole Daten
schreibe sie in einen Vector = Vector 1

In Vector 1 sind also immer die neusten Daten drin.
Vector model zuweisen
model tabelle zuweisen
fire...
Fertig

Wieso muss jetzt der Vector wissen das er sich verändert hat?

 

[HLX]

Für dein DB-Update verwendest du die Methode getValueAt() vom Tablemodel. Diese liefert dir das was du zuletzt geladen hast. Deine Veränderung im Editor wird nicht registriert. Das geschieht erst durch die Implementierung von setValueAt():

public Object setValueAt(int rowIndex, int columnIndex, Object value) { 
      Vector rowVector = (Vector)data.elementAt(rowIndex); 
      rowVector.setElementAt(columnIndex, value); 
   }

Erst damit kann getValueAt() die aktuellen Werte liefern.

 

[Waldi]

So geil. So geil. Es funktioniert. Zumindest das DB Update. Jetzt muss nur noch die Sache mit der Änderung richtig funzen. So das die Spalten sich nicht verändern und die Column in der richtigen Breite bleibt.

Wa ich nicht verstehe: Die jComboBox und die Spalte sind doch die Sache der Tabelle nicht des Models. Wenn ich das Model neu setze und sage das sich die Daten geändert haben warum verstellt er dann die Tabelle?? Ich habe das gemacht

getTableFromDatabase(1); //Aktualisiert mein Vectoren und das Model
               atm.fireTableDataChanged();

Und nichts passiert. Vielleicht kanns ja einer erklären. Das würde mir ja ein kleines Stückchen helfen. Wenn ich das geschafft habe kann ich beruhigt auf Fastnacht gehen und mir so einen in den Kanal schütten Uargh

 

[André Uhres]

Erstens würde ich das AbstractTableModel so coden (ist einfacher und kann fast alles "by default"):

   class MyTableModel extends DefaultTableModel{
        public MyTableModel(Vector columNames, Vector data) {
            super(columNames, data);
        }
        public boolean isCellEditable(int row, int column) {
            if (column < 3) {
                return true;
            } else {
                return false;
            }
        }
    }

Zweitens würde ich aus getJTable alles rausnehmen was nicht dazu gehört:

   private JTable getJTable() {
        jTable = new JTable();
        jTable.putClientProperty("terminateEditOnFocusLost", Boolean.TRUE);
        jTable.setAutoResizeMode( JTable.AUTO_RESIZE_OFF );
        JTableHeader header = jTable.getTableHeader();
        header.setReorderingAllowed(false);
        return jTable;
    }

und den Rest aus der Methode meinetwegen in eine neue Methode "adaptTableModel" packen.
getJTable braucht man nur einmal ganz am Anfang aufzurufen:

       jTable = getJTable();
        getContentPane().add( new JScrollPane(jTable) );

Zum Auffrischen der JTable genügt dann dies:

       getTableFromDatabase(1);//erzeugt ein neues "MyTableModel" --> atm
        jTable.setModel(atm);//zeichnet automatisch alles neu
        adaptTableModel();

 

[Beni]

(Die Erklärungen zu den folgenden Codestücken erfolgte im Chat. So wie ich das verstanden habe, scheint das Problem nun gelöst zu sein)

Im Model

public class MyTableModel ... {
  public void setData( Vector data ){
    this.data = data;
    fireTableDataChanged();
  }
}

Beim Update

this.gtfd = new ...
this.atm.setData( gtfd.data );

Zum Rumspielen:

import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Vector;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.table.AbstractTableModel;

public class Beispiel extends JFrame{
    private MyTableModel atm;
    
    public Beispiel(){
        JTable table = new JTable();
        Vector names = new Vector();
        names.add( "10" );
        names.add( "20" );
        names.add( "30" );
        atm = new MyTableModel( names, data());
        table.setModel( atm );
        add( new JScrollPane( table ));
        
        JButton button = new JButton( "Reload" );
        add( button, BorderLayout.SOUTH );
        button.addActionListener( new ActionListener(){
            public void actionPerformed( ActionEvent e ) {
                atm.setData( data() );
            }
        });
    }
    
    public static void main( String[] args ) {
        Beispiel b = new Beispiel();
        b.setBounds( 10, 10, 400, 400 );
        b.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );
        b.setVisible( true );
    }
    
    private Vector data(){
        Vector data = new Vector();
        while( Math.random() > 0.1 ){
            Vector row = new Vector();
            row.add( new Integer( (int)(Math.random() * 10 )));
            row.add( new Integer( (int)(Math.random() * 20 )));
            row.add( new Integer( (int)(Math.random() * 30 )));
            data.add( row );
        }
        return data;
    }
}


class MyTableModel extends AbstractTableModel
{
   /**
    *
    */
   private static final long serialVersionUID = -9072279854289269980L;
   
   private Vector data;
   private Vector columNames;
   
   /**
    */
   public MyTableModel(Vector columNames, Vector data)
   {
      this.columNames = columNames;
      this.data = data;
   }
   
   public void setData( Vector data ) {
       this.data = data;
       fireTableDataChanged();
   }
   
   public int getRowCount() {
      // TODO Automatisch erstellter Methoden-Stub
      return data.size();
   }
   public int getColumnCount() {
      // TODO Automatisch erstellter Methoden-Stub
      return columNames.size();
   }
   public Object getValueAt(int rowIndex, int columnIndex) {
      Vector rowVector = (Vector)data.elementAt(rowIndex);
      return rowVector.elementAt(columnIndex);
   }
   
   
   public boolean isCellEditable(int row, int column)
   {
      if (column < 3)
      {
         return true;
      }
      else
      {
         return false;
      }
   }
   
   public String getColumnName(int col) {
      return columNames.elementAt(col).toString();
   }
}