Daten einer JTable lassen sich nicht ändern

[crazyd]

Moin,

die JTables machen mich noch wahnsinnig! :autsch:
Ich versuche die Daten eines Arrays ("liste") in einer JTable anzuzeigen. Das klappt auch!
Ich will jetzt aber Daten im JTable anändern, das klappt leider nicht! Ich habe die setValueAt-Methode überschrieben, aber sie wird als "nicht benutzt" von Eclipse markiert.

Vielleicht habt ihr ja eine Idee was ich falsch mache. Würde mich über eure Kommentare freuen:

frame = new JFrame( "Tabelle" );
    AbstractTableModel tableModel = new AbstractTableModel() {
	Integer[][] werte = liste;
	 
	    @Override
	    public int getColumnCount() {
	        return werte[0].length;
	    }
	 
            @Override
            public int getRowCount() {
                return werte.length;
            }
 
            @Override
            public Object getValueAt(int rowIndex, int columnIndex) {
                return werte[rowIndex][columnIndex];
            }   
            
//Eclipse meint dazu: "The method setValueAt(Integer, int, int) from the type new AbstractTableModel(){} is never used locally"
            public void setValueAt(Integer value, int row, int col) {
                werte[row][col] = value;
                fireTableCellUpdated(row, col);
            }

            
        };   
        frame.getContentPane().setLayout (new BorderLayout());

		Box box = Box.createVerticalBox();
		box.setBorder(BorderFactory.createEmptyBorder(15,15,0,15));
		table = new JTable(tableModel);
		JScrollPane scroll = new JScrollPane( table );
                box.add( scroll );
        
	frame.getContentPane().add(box, BorderLayout.NORTH);	
	    


        //Das hier sollte die Werte in der Jtable ändern? Geht nicht!
        System.out.println(table.getValueAt(3, 2));
        table.setValueAt("54", 3, 2);
        System.out.println(table.getValueAt(3, 2));
		
		    
        frame.setDefaultCloseOperation( JFrame.HIDE_ON_CLOSE ); 
        frame.pack();
        frame.setVisible( true );

        );

Ein kleiner Ausschnitt von meinem Code... Alles was nicht von Interesse ist, hab ich rausgeschnitten (JButtons, JLabels etc)
Was mach ich falsch?

Viele Grüße!

 

[Suinos]

Du überschreibst die Methode ja auch nicht, da die Methodensignatur nicht übereinstimmt:

public void setValueAt(Object aValue, int row, int column)

Mit dem

@Override

vor der Methode hättest du das auch gemerkt.

 

[ymene]

Du hast die Methode versucht zu überschrieben, aber die falsche Signatur verwendet. Die Signatur der setValueAt-Methode ist setValueAt( Object, int, int )

            @Override
            public void setValueAt(Object value, int row, int col) {
               //Inhalt prüfen und setzen.
            }

Da du die Methode aber mit einem String aufrufst, wird deine Integer Methode nie genutzt.
Und auch wenn du die Methode mit einem Integer aufrufen würdest, würde auch die Methode der Superklasse aufgerufen werden, da die Signatur auf diese zutrifft.

Du musst sie mit Object überschreiben, und dann dein Value in den entsprechenden Datentyp parsen.

Grüße,
ymene

PS: Besorg dir mal einen CodeFormatter

 

[SlaterB]

isCellEditable() muss auch überschrieben werden

 

[crazyd]

Ok, danke für die Hilfe:

Ich fasse noch mal zusammen:
- Wenn ich eine Methode überschreiben will, muß nicht nur der Methodenname übereinstimmen, sondern auch die Parameter.
- Ich muß die Methode zum Überschreiben mit "@Override" markieren.

Richtig? Wieder was gelernt!

Hab trotzdem ein Problem:
-Wie bekomme ich meinen value-Parameter ins Integer-Format
Funktioniert das so:

@Override
            public void setValueAt(Object value, int row, int col) {
                
            	String valueString = value.toString(); 
            	Integer valueInteger = Integer.parseInt(valueString);
            	werte[row][col] = valueInteger;
            	
                fireTableCellUpdated(row, col);
            }

Sieht imho ziemlich "verrückt" aus

 

[ARadauer]

oder kurz.. werte[row][col] = Integer.parseInt((String) value);

 

[Suinos]

Je nachdem, was für ein Objekt dahinter steckt, kannst du ihn gleich casten.

Du kannst ja mal die Klasse des Objekts ausgeben, dass siehst du, was dort wirklich für ein Objekt ist:

System.out.println(value.getClass());

--

Ich muß die Methode zum Überschreiben mit "@Override" markieren.

Nein. Müssen nicht, aber gewöhne es dir an.

@Override

ist eine Annotation, welche dem Compiler sagt, dass die nachfolgende Methode eine Methode der Superklasse überschreiben oder die eines Interfaces implementieren muss.
Wenn dies nicht der Fall ist, wird der Compiler eine Fehlermeldung ausgeben, und du merkst spätestens dann, dass dort etwas nicht stimmen kann.