Display of text of different rows with different color - JTable

Question

Hi all !
I have struck in a problem. I have to display the different rows with different color in the JTable. I have created an arraylist in the model class, which stores color of all the rows. Now in renderer class, I am just picking up color from the arraylist, taking rowIndex as index for arraylist. I used to set foreground color for each row in the renderer class. The problem is that it is not showing all the rows properly, sometimes it show all the rows correctly, but as i resize the window/panel, it starts behaving abnormally, some text is painted properly, but other aren't.
Does anybody have any solution regarding this problem.
One thing i want to mention is that in getTableCellRendererComponent() method, i used to retrive the color everytime, and setting the foreground color of the text. Is this a right approach or some other technique have to be followed.
here is code of renderer class ---
import java.awt.*;
import javax.swing.*;
* Renderer to plot stage record table.
public class ExStageCellRenderer extends javax.swing.table.DefaultTableCellRenderer
    Color curColor;
     * returns component to be painted, overridding this method from
     * DefaultCellRenderer
     * @param table
     *            table whose component has to be plotted
     * @param value
     *            value of that particular cell
     * @param isSelected
     *            is Cell selected
     * @param hasFocus
     *            has the cell got focus
     * @param row
     *            row of the cell
     * @param column
     *            column of the cell
     * @return painted component
    public Component getTableCellRendererComponent (JTable table, Object value, boolean isSelected, boolean hasFocus, int row,
            int column)
        Component component = super.getTableCellRendererComponent (table, value, isSelected, hasFocus, row, column);
        ExStageDataModel model = (ExStageDataModel)table.getModel ();       
        if (curColor instanceof Color) {
                curColor = model.getColor(row);
            } else {
                // If color unknown, use table's foreground color
                curColor = table.getForeground();
        String tooltip = "";
        StringBuffer stringBuffer = new StringBuffer ();
        for (int i = 0; i < table.getColumnCount (); i++)
                Object Value = table.getValueAt (row, i);
                if (Value == null) continue;
                stringBuffer.append (Value.toString () + "    ");
        tooltip += stringBuffer.toString ();
        tooltip = tooltip.trim ();
        ((JComponent) component).setToolTipText (tooltip);
        if (column == 1)
            this.setHorizontalAlignment (SwingConstants.CENTER);
        else if (column == 0 || column == 4 || column == 5)
            this.setHorizontalAlignment (SwingConstants.LEFT);
        else
            this.setHorizontalAlignment (SwingConstants.RIGHT);
        // LOOK!! should turn tip off if there is none !!
        component.setForeground (this.curColor);
        //System.out.println("render : " + component.getBounds ());
        //component.validate ();
        component.update (component.getGraphics ());
        return component;
Waiting for reply....
Code snooker

Answer 1

Hmm, why are you doing the curColor instanceof? What does this accomplish? I don't see any reason to ask what it was before; all you should be interested in here is what you want it to be now.
All you should have to do is say:
this.setForeground(model.getColor(row));I also don't see why you're doing the update() -- JTable is going to do that for you at the appropriate time, you have no idea if now is the right time to do it or not. In fact it probably isn't, you're probably updating the previously-drawn cell with your new color and alignment, which may be the strange behavior you're seeing.
Also, I don't see why you're doing the super. All that's going to do is return "this", so why not just use "this"?
Well, without knowing your larger code or exactly what you're trying to accomplish, maybe I'm just missing something.
I've just recently been working on a program where I had some similar requirements -- different foreground and background colors and alignment -- and I found it much cleaner to create a class to hold all this data, and then create a default renderer for that class. Then I made some constructors for this "attribute" class that let me set whatever I need, like
public CellAttrib(String s,Color c,int alignment,Border border)
  this.s=s;
  this.c=c;
  this.border=border;
// with suitable defaults ...
pubic CellAtrtrib(String s)
  this.s=s;
  this.c=Color.BLACK;
  this.border=null;
}Then the renderer just queried the cell-attributes class, as in:
setForeground(value.getColor());
setHorizontalAlignment(value.getAlignment());
setBorder(value.getBorder);