Jtable falscher Wert wird übergeben

[T1M]

Hi,
hab ne tabelle in der ich per button die hintergrundfarbe einer zelle ändern kann. wenn ich das tue, dann wird der inhalt der zelle überschrieben.
Bsp.: wenn ich der zelle "welt" nen roten hintergrund geben will wird aus:

|-----------|-------------|
| Hallo     |  Welt       |
|-----------|-------------|

wird

|-----------|-------------|
| Hallo     |  Hallo      |
|-----------|-------------|

aber wenigsten stimmt die hinterfrundfarbe. :wink:
mal n bisl code ...
mein renderer

class ColorTableCellRenderer extends DefaultTableCellRenderer
{
	  public Component getTableCellRendererComponent(JTable table, Object value,
	                                                  boolean isSelected,
	                                                  boolean hasFocus, int row,
	                                                  int column)
	  {
	  /* super.getTableCellRendererComponent(table, value, isSelected,
	                                        hasFocus, row, column);*/

	  	System.out.println("row: "+row+" column: "+column);
	  	System.out.println("#### Value: "+ value.toString());
	    System.out.println("Klasse: " + value.getClass());
	    
	    if (value instanceof Color)
	    
	    {
	       	System.out.println("###Value1: "+ value);
	    	Color color = (Color)value;
	    	System.out.println("row: "+row+" column: "+column);
	    	this.setBackground(color);
	    	
	    	//setText(color == null ? "" : color.getRed() + ", " + color.getBlue() + ", "
	    	//  		+ color.getGreen());
	    } 
	    else
	    {
	    		super.getTableCellRendererComponent(table, value, isSelected,
                    hasFocus, row, column);
                    setBackground(table.getBackground());
	    }
	    
	    return this;
	  }
}

die tabelle (2d-array) :

table = new JTable(data, columnNames)
		{
	     public Class getColumnClass(int i)
	     {
	     	switch (i)
			{
				case 0: return Color.class;
				case 1: return Color.class;
				case 2: return Color.class;
				case 3: return Color.class;
				case 4: return Color.class;
				default: return String.class;
			}
	     }
	   };

Dankö schööön,
T1M

 

[m@nu]

liefert dein TableModel denn eine Color-instanz per getValueAt()?
das TableModel wäre nützlich zu sehen, denn "value" im renderer entspricht nicht der ausgabeklasse der zelle, sondern dem value, welches aus dem TableModel geliefert wird....

 

[Bleiglanz]

braucht man da nicht ein paar breaks beim switch??

und dreh die reihenfolge um, die mit den meisten sollte im default sein

 

[m@nu]

hab da mal was zusammen geschustert auf die schnelle... vielleicht kannst du einiges daraus wiederverwenden:

/*
 * Created on 01.06.2005
 */
package ColorTableModel;

import java.awt.Color;

/**
 * @author Manuel Alabor
 */
public class ColorBean {

    /* Eigenschaften: */
    private String text;
    private Color color;
       
    /**
     * @param text
     * @param color
     */
    public ColorBean(String text, Color color) {
        super();
        this.text = text;
        this.color = color;
    }
    
    
    public Color getColor() {
        return color;
    }
    public void setColor(Color color) {
        this.color = color;
    }
    public String getText() {
        return text;
    }
    public void setText(String text) {
        this.text = text;
    }
}

/*
 * Created on 01.06.2005
 */
package ColorTableModel;

import javax.swing.table.AbstractTableModel;

/**
 * @author Manuel Alabor
 */
public class ColorTableModel extends AbstractTableModel {

    /* Objekte: */
    private ColorBean[] beans;
    
    public ColorTableModel(ColorBean[] beans) {
        this.beans = beans;
    }
    
    public int getColumnCount() {
        return 2;
    }

    public String getColumnName(int col) {
        if(col == 0) {
            return "Text";
        } else if(col == 1) {
            return "Farbe";
        } else {
            return "";
        }
    }
    
    public int getRowCount() {
        return beans.length;  // Text; Farbe
    }
    
    public Object getValueAt(int row, int col) {
        ColorBean bean = beans[row];
        
        if(col == 0) {
            return bean.getText();
        } else if(col == 1) {
            return bean.getColor();
        } else {
            return "";
        }
    }
}

/*
 * Created on 01.06.2005
 */
package ColorTableModel;

import java.awt.Color;
import java.awt.Component;

import javax.swing.BorderFactory;
import javax.swing.JLabel;
import javax.swing.JTable;
import javax.swing.table.TableCellRenderer;

/**
 * @author Manuel Alabor
 */
public class ColorTableCellRenderer extends JLabel implements TableCellRenderer {

    public ColorTableCellRenderer() {
        setOpaque(true);
        setBorder(BorderFactory.createEmptyBorder(0,3,0,0));
    }
    
    public Component getTableCellRendererComponent(JTable table, Object value,
            boolean isSelected, boolean hasFocus, int rowIndex, int vColIndex) {

        /* Farbe setzen: */
        if (isSelected) {
            setBackground(table.getSelectionBackground());
            setForeground(table.getSelectionForeground());
        } else {
            setBackground(table.getBackground());
            setForeground(table.getForeground());
        }
        
        /* Value auswerten: */
        if(value instanceof Color) {
            setText("");
            setBackground((Color)value);
        } else {
            setText("Color n/a");
        }
        
        /* Rückgabe: */
        return this;
    }

    public void validate() {}
    public void revalidate() {}
    protected void firePropertyChange(String propertyName, Object oldValue, Object newValue) {}
    public void firePropertyChange(String propertyName, boolean oldValue, boolean newValue) {}
	
}

/*
 * Created on 01.06.2005
 */
package ColorTableModel;

import java.awt.BorderLayout;
import java.awt.Color;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JTable;

/**
 * @author Manuel Alabor
 */
public class ColorTableModelTest extends JFrame {

    public ColorTableModelTest(ColorBean[] beans) {
        super("ColorTableModel Demo");
        
        setSize(200,300);
        setLocation(100,100);
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        
        JPanel contentPane = (JPanel)getContentPane();
        contentPane.setLayout(new BorderLayout());
        
        JTable tblTable = new JTable(new ColorTableModel(beans));
        tblTable.getColumnModel().getColumn(1).setCellRenderer(new ColorTableCellRenderer());
        contentPane.add(tblTable, BorderLayout.CENTER);
    }
    
    public static void main(String[] args) {
        // Beans erstellen:
        ColorBean[] beans = new ColorBean[3];
        beans[0] = new ColorBean("Rot", Color.red);
        beans[2] = new ColorBean("Gelb", Color.yellow);
        beans[1] = new ColorBean("Blau", Color.blue);
        
        // Frame instanzieren:
        ColorTableModelTest frame = new ColorTableModelTest(beans);
        
        // Anzeigen:
        frame.show();
    }
}

ausschauen tuts dann so:

is natürlich ein weg unter vielen... aber der is mir gleich am leichtesten gefallen
du brauchst jetzt nur noch das entsprechende bean zu aktualisieren und anschliessend das entsprechende event im model auszulösen... voilà

@Bleiglanz: nö, brauchts nicht... per return wird ja sowieso die codeausführung beendet break wäre hier sinnlos

 

[T1M]

wow, thx.
werd mich da mal durcharbeiten.
@ bleiglanz:
ne, denke das sollte so stimmen. 0-4 sind die anzahl der spalten. denn es soll ja n farbe zurück gegeben werden und nicht die farbe als string.

hmm, seh gerad, du arbeitest mit nem tablemodel.
das hab ich net gemacht.
bei mir schaut des so aus:

String[] columnNames = {"First Name",
            "Last Name",
            "Sport",
            "# of Years",
            "Vegetarian"};
    Object[][] data =
    {
    		{
    			"Mary", "Campione", "Snowboarding", new Integer(5), new Boolean(false)
    		},
			{
    			"Alison", "Huml", "Rowing", new Integer(3), new Boolean(true)
			},
			{
				"Kathy", "Walrath", "Knitting", new Integer(2), new Boolean(false)
			},
			{
				"Sharon", "Zakhour", "Speed reading", new Integer(20), new Boolean(true)
			},
      }

aber ich werd mal versuchen mich da durchzubeißen.

THX,
T1M

 

[m@nu]

kein problem weis wie lästig die JTable is... hab auch genug lange damit gekämpft

ich kann dir die arbeit mit models nur empfehlen... sobald du da mal durchblickst ist praktisch nix mehr unmöglich
solange du die daten später bei der laufzeit nicht ändern willst, ist die methode mit dem DefaultTableModel ne ganz gute variante (dein array wird im hintergrund in ein solches eingesetzt) ... doch sobald du änderungen an der JTable vornehmen möchtest, bist du mit einem eigenen model meistens besser beraten.

 

[T1M]

jep.
aber wenn das mal fertig ist, dann müssen glaub ich auch zur laufzeit daten verändert werden können. bzw. die daten sollen dann aus ner datenbank kommen. per aktualisieren muß dann immer alles aufm neusten stand sein.
aber, bis dahin dauerts noch ..., ich meld mich dann! :wink:

 

[T1M]

jep.
aber wenn das mal fertig ist, dann müssen glaub ich auch zur laufzeit daten verändert werden können. bzw. die daten sollen dann aus ner datenbank kommen. per aktualisieren muß dann immer alles aufm neusten stand sein.
aber, bis dahin dauerts noch ..., ich meld mich dann! :wink:

 

[T1M]

Hey hey, du hast einfach um mein problem herumgeschifft. so gayts edde! :wink:
und zwar hier:

/* Value auswerten: */
        if(value instanceof Color) {
            setText("");
            setBackground((Color)value);
        }

durch setText(""); verschwindet der text der zelle. ok, es steht dann auch nix falsches drin, allerdings sollte des vom inhalt schon so bleiben wie es vor dem einfärben war.
haste da ne idee?

 

[m@nu]

momentan ist das ganze so geschrieben, dass das bean für die zweite zelle nur eine farbe speichert...
wenn du jetzt aber auch noch einen text für die zweite spalte haben möchtest, so müsstest du das bean erweitern.

du musst immer so denken: etwas was ich hinten nicht im model hab, kann ich vorne im GUI nicht anzeigen...

/*
 * Created on 01.06.2005
 */
package net.msites.drivesync.data.core.models;

/**
 * @author Manuel Alabor
 */
public class ColorTableRowBean {

    /* Eigenschaften: */
    private String text;
    private ColorBean color;
    
    public ColorTableRowBean(String text, ColorBean color) {
        this.text = text;
        this.color = color;
    }
    
    // ...
    
}

wenn du jetzt diese bean-klasse anstatt der alten in das model einbindest, die möglichkeit, eine farbe und einen text für die zweite spalte zu speichern...

/* Objekte: */
private ColorTableRowBean[] beans;

// ....

public Object getValueAt(int row, int col) {
    if(col == 0) {
        return beans[row].getText();
    } else if(col == 1) {
        return beans[row].getColorBean();
    } else {
        return("");
    }
}

// ....

natürlich müsstest du den renderer auch anpassen... eine möglichkeit wäre die folgende:

if(value instanceof ColorBean) {
    ColorBean colorBean = (ColorBean)value;
    setBackground(colorBean.getColor());
    setText(colorBean.getText());
}

ist ein wenig wackelig erklärt, aber falls du mit dem beispiel nicht klar kommst meld' dich nochmal

<edit>
falls du nach dem ganzen hier dann noch nerven hast, hier gibts ne weitere JTable spielerei von mir:
:arrow: http://jroller.com/page/swissManu
</edit>

 

[Sky]

braucht man da nicht ein paar breaks beim switch??

Nein. Mit 'return' wird die Methode verlassen.

 

[m@nu]

braucht man da nicht ein paar breaks beim switch??

Nein. Mit 'return' wird die Methode verlassen.

kannst mich auch wiederholen :bae: gggg nichts für ungut... konnt' ich mir net verkneifen gg

 

[Sky]

braucht man da nicht ein paar breaks beim switch??

Nein. Mit 'return' wird die Methode verlassen.

kannst mich auch wiederholen :bae: gggg nichts für ungut... konnt' ich mir net verkneifen gg

Sorry, hatte ich vorhin nicht gesehen... habe bei dem vielem Quellcode weiter gescrollt und bin zum nächsten Post gegangen

 

[m@nu]

wie gesagt nix für ungut (hab ich mir schon gedacht )