当我更改第5列的单元格时,更改单元格中的颜色,他不会停在这里,他会更改下一个单元格

时间:2016-10-30 15:42:11

标签: java swing colors tablecellrenderer

我在更改单元格中的颜色方面遇到问题,当我更改第5列的单元格时,他并没有停在这里,他改变了下一个单元格...... 这是我的代码:

public class MyRenderer extends DefaultTableCellRenderer {
    public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus,
            int row, int column) {
        Component c = super.getTableCellRendererComponent(table, value, isSelected, hasFocus, row, column);
        Etudiant E = new Etudiant();

        if (column == 0) {
            int id = E.getAIdEtud(table.getModel().getValueAt(row, 1).toString(), table.getModel().getValueAt(row, 2).toString());
            if (E.IsRoudoublan(id) && E.IsExcclu(id)) {
                c.setBackground(new java.awt.Color(0, 0, 255));
            } else if (E.IsExcclu(id)) {
                c.setBackground(new java.awt.Color(255, 0, 0));
            } else if (E.IsRoudoublan(id)) {
                c.setBackground(new java.awt.Color(20, 200, 0));
            }
        } else if (column == 5) {
            if (Integer.parseInt(table.getModel().getValueAt(row, 5).toString()) >= 3) {
                c.setBackground(new java.awt.Color(20, 200, 20));
            }
        } else {
            c.setBackground(new java.awt.Color(100, 100, 100));
        }

        return c;

    }
}

1 个答案:

答案 0 :(得分:1)

我拿走了你的片段并纠正了我发现的所有问题。检查所有带注释的行。

注意:

  • 为所有可能的代码路径提供颜色,您忘记在两个位置提供颜色(请参阅// << provide ELSE part here, eg:行)
  • TableCellRenderer.getTableCellRendererComponent方法中报告的索引是视图索引。
  • 索引模型时,需要使用模型索引。您正在使用视图索引来索引模型。这可能导致问题,例如当您的表被排序,重新排列列或应用行筛选器时。在末尾查找带有注释的行以进行更正。
public class MyRenderer extends DefaultTableCellRenderer {
    public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int rowViewId, int columnViewId) {
        Component c = super.getTableCellRendererComponent(table, value, isSelected, hasFocus, rowViewId, columnViewId);
        Etudiant E = new Etudiant();

        int rowModelId = table.convertRowIndexToModel(rowViewId); // << for lookup of values in the model
        int colModelId = table.convertColumnIndexToModel(columnViewId); // << for lookup of values in the model

        if (colModelId == 0) { // << you mean to check if the model index is 0 here
            int id = E.getAIdEtud(table.getModel().getValueAt(rowModelId, 1).toString(), table.getModel().getValueAt(rowModelId, 2).toString()); // << when indexing the model, use model indexes
            if (E.IsRoudoublan(id) && E.IsExcclu(id)) {
                c.setBackground(new java.awt.Color(0, 0, 255));
            } else if (E.IsExcclu(id)) {
                c.setBackground(new java.awt.Color(255, 0, 0));
            } else if (E.IsRoudoublan(id)) {
                c.setBackground(new java.awt.Color(20, 200, 0));
            }
            // << provide ELSE part here, eg:
            else {
                c.setBackground(new java.awt.Color(100, 100, 100));
            }
        } else if (colModelId == 5) { // << you mean to check if the model index is 5 here
            if (Integer.parseInt(table.getModel().getValueAt(rowModelId, 5).toString()) >= 3) { // << when indexing the model, use model indexes
                c.setBackground(new java.awt.Color(20, 200, 20));
            }
            // << provide ELSE part here, eg:
            else {
                c.setBackground(new java.awt.Color(100, 100, 100));
            }
        } else {
            c.setBackground(new java.awt.Color(100, 100, 100));
        }

        return c;
    }
}