JTable & Problem, das nicht die komplette Zeile gefärbt

[Muecke]

Habe bereits ein wenig rumexperimentiert allerdings werden von meinen 11 Spalten nur 5 mit meiner Wunsch-Hintergrundfarbe eingefärbt und ich weiß nicht weshalb nicht die komplette Zeile eingefärbt wird.

Hier einmal meine Codes:

KlasseMain => Erstellen der Tabelle

/* Panel für die Übersichts-Tabelle */
JTable tableOverview = new JTable(new EBayOrgaMainDTableModel());
JTableHeader header = tableOverview.getTableHeader();
header.setBackground(Color.GRAY);
header.setForeground(Color.white);
tableOverview.getModel().addTableModelListener(this);
tableOverview.setDefaultRenderer(Object.class, new RowColorRenderer());

Klasse AbstractTable Model:

public class EBayOrgaMainDTableModel extends AbstractTableModel
{
	private boolean DEBUG = false;
	
    private String[] columnNames = { "Nr.", "eBayName", "Name", "Vorname", "Ort", "Email", 
    		"Betrag", "Porto", "Bezahlt", "Verschickt", "Bewertet" };

    private Object[][] data =
    {		
    	{ new Integer(0), "","","","","", new Integer(0), new Integer(0), new Boolean(false), new Boolean(false), new Boolean(false) },
    	{ new Integer(1), "","","","","", new Integer(0), new Integer(0), new Boolean(false), new Boolean(false), new Boolean(false) },
    	{ new Integer(2), "","","","","", new Integer(0), new Integer(0), new Boolean(false), new Boolean(false), new Boolean(false) },
    	{ new Integer(3), "","","","","", new Integer(0), new Integer(0), new Boolean(false), new Boolean(false), new Boolean(false) },
    	{ new Integer(4), "","","","","", new Integer(0), new Integer(0), new Boolean(false), new Boolean(false), new Boolean(false) },
    	{ new Integer(5), "","","","","", new Integer(0), new Integer(0), new Boolean(false), new Boolean(false), new Boolean(false) },
    	{ new Integer(6), "","","","","", new Integer(0), new Integer(0), new Boolean(false), new Boolean(false), new Boolean(false) },
    	{ new Integer(7), "","","","","", new Integer(0), new Integer(0), new Boolean(false), new Boolean(false), new Boolean(false) },
    };
 
    public int getColumnCount()
    {
    	return columnNames.length;
    }

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

    public String getColumnName(int col)
    {
    	System.out.println(columnNames[col]);
    	return columnNames[col];
    }
    public Object getValueAt(int row, int col)
    {
    	return data[row][col];
    }

    /* Checkboxen in Spalten setzen! Funktionsweise leider noch ???? -> Es geht einfach... */
    public Class getColumnClass(int c)
    {
    	return getValueAt(0, c).getClass();
    }
    
    public boolean isCellEditable(int row, int col)
    {
    	return true;
    }
   public void setValueAt(Object value, int row, int col)
    {
      if (DEBUG)
      {
        System.out.println("Setting value at " + row + "," + col
            + " to " + value + " (an instance of "
            + value.getClass() + ")");
      }

      data[row][col] = value;
      fireTableCellUpdated(row, col);

      if (DEBUG)
      {
        System.out.println("New value of data:");
        printDebugData();
      }
    }

    private void printDebugData()
    {
      int numRows = getRowCount();
      int numCols = getColumnCount();

      for (int i = 0; i < numRows; i++)
      {
        System.out.print("    row " + i + ":");
        for (int j = 0; j < numCols; j++)
        {
          System.out.print("  " + data[i][j]);
        }
        System.out.println();
      }
      System.out.println("--------------------------");
    }
}

Deine Klasse Table-Renderer:

class RowColorRenderer implements TableCellRenderer
{
    public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column)
    {
        Border border = BorderFactory.createEmptyBorder(1, 1, 1, 1);
        JLabel label = new JLabel((String) value);
		label .setOpaque(true);
		label .setBorder(border);
		label .setFont(table.getFont());
		label .setForeground(table.getForeground());
		label .setBackground(table.getBackground());
		label .setHorizontalAlignment(SwingConstants.RIGHT);
		column = table.convertColumnIndexToModel(column);
		if (table.getColumnCount() == 0)
		{
		     System.out.println("ColumnCount ist 0 ");
		}
		else
		{
		    if (row == 0)
		    {
		    	label .setBackground(Color.RED); 
		    }
		    if (row == 1)
		    {
		    	label .setBackground(Color.RED);
		    }
		}
		return label ;
    }  
}

und auch dieser Renderer hat nicht den Erfolg gebracht:

class RowColorRenderer extends javax.swing.table.DefaultTableCellRenderer
{
	public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column)
	{
		// component will actually be this.
		super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
		
		setBackground(Color.yellow);
		return this;
	}
	

}

Kann mir evt. jemand helfen und sagen weshalb nicht alle Spalten der ersten und zweiten Teile rot makiert werden???
Danke im voraus!
Gruss
Muecke

 

[semi]

Es gibt bereits Default-Renderer für Integer und Boolean. Diese werden bevorzugt verwendet.

Siehe

System.out.println(tableOverview.getDefaultRenderer(Integer.class) instanceof RowColorRenderer);
System.out.println(tableOverview.getDefaultRenderer(Boolean.class) instanceof RowColorRenderer);

Vermeide es in Renderern Objekte jeglicher Art zu erzeugen. In deinem erster Renderer erzeugst du einen Haufen
Labels und Border. Allein schon wenn du den Mauszeiger über die Tabelle bewegst werden die Renderer zig mal
aufgerufen.

PS: Hier eine Lösung für das Problem

tableOverview.setDefaultRenderer(String.class, new RowColorRenderer(tableOverview.getDefaultRenderer(String.class)));
tableOverview.setDefaultRenderer(Integer.class, new RowColorRenderer(tableOverview.getDefaultRenderer(Integer.class)));
tableOverview.setDefaultRenderer(Boolean.class, new RowColorRenderer(tableOverview.getDefaultRenderer(Boolean.class)));
   
   
class RowColorRenderer implements TableCellRenderer
{
   private TableCellRenderer renderer; 
      
   public RowColorRenderer(TableCellRenderer renderer)
   {
      this.renderer = renderer;
   }
      
   @Override
   public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column)
   {
      Component result = renderer.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
      result.setBackground(Color.YELLOW);
      return result;
   }
}

 

[Muecke]

Klasse Sache!
Ich habe heute schon haufenweise Seiten durchforstet und nirgends konnte ich eine gescheite Antwort finden!
Vielen Dank!

Aber jetzt noch mal eine Frage dazu:
Ich möchte gerne abhängig von den Checkboxen (true oder false) die Zeilen unterschiedlich einfärbern.
Wo mache ich diese Überprüfung, ob bspw. eine Checkbox true oder false gesetzt ist???
Ich kann mir nicht vorstellen, daß das in der Renderer Klasse geschehen soll, oder doch??

Danke im voraus.
Gruss
Muecke

 

[semi]

Klar, im Renderer. Einfach den Parameter "value" auswerten.

 

[Muecke]

Mhhh :?: :?:

Ich habe ja 3 Spalten mit Checkboxen, wie unterscheide ich die nun???
Möchte gerne wenn in Spalte 11 true steht eine Farbe setzen und wenn ich Spalte 10 true stehe eine andere Farbe!!
und das gleiche für Spalte 9 !!!

 

[semi]

Du kannst im Renderer auch die Werte der anderen Spalten abfragen.
z.B.

@Override
public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column)
{
   Component result = renderer.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
   TableModel model = table.getModel();
   // Wenn Model-Column 5 und Wert in Model-Column 9 TRUE
   if( 5 == table.convertColumnIndexToModel(column) 
      && Boolean.TRUE.equals(model.getValueAt(table.convertRowIndexToModel(row), 9)) )
   {
      result.setBackground(Color.PINK);
   }
   else
   {
      result.setBackground(Color.YELLOW);
   }
   return result;
}

 

[Guest]

Perfekt, danke dir!
Werde es gleich morgen ausprobieren!

Ps.: Ich glaube, da wäre ich im Leben nicht auf diesens Konstrukt gekommen!