JTable 保持颜色与 model 不排

Question

I have a dilemma. I am writing a program that will view logs from an application and depending on the log level (eg Info, Debug, Error, etc.) the row gets highlighted a certain color. I also give the users the ability to hide log levels. So if the clicks off "show info", then all the log entries that are "Info" are hidden. The problem comes that when entries are hidden, the color they were highlighted gets pushed down to the row that was below them.

I am looking for a way that the color stays with the entry and not render the color in the next cell when the object is hidden

Here is what I have done thus far:

I initialize the renderer and set it as the default:

tableRenderer = new ModelTableRenderer();
table.setDefaultRenderer(Object.class, tableRenderer);

and then I create my sorter:

sorter = new TableRowSorter<DefaultTableModel>(model);
table.setRowSorter(sorter);

and I have a callback method that each of checkboxes use called showSelected . This takes a list of LogLevels to show and applies the filter to the sorter:

  public void showSelected(List<LogLevel> showList) {
    Set<LogLevel> hideSet = showList.stream().collect(Collectors.toSet());
    sorter.setRowFilter(new RowFilter<DefaultTableModel, Object>(){

      @Override
      public boolean include(Entry<? extends DefaultTableModel, ? extends Object> entry) {
        LogLevel level = (LogLevel) entry.getValue(LogModel.ERROR_COLUMN);
        return hideSet.contains(level);
      }
    });
  }

ModelTableRenderer class

This class is what I use to render the row colors. It looks at the current row and grabs the column value where the LogLevel exists and depending on what the log level is. This works fine when rows are not hidden.

public class ModelTableRenderer extends DefaultTableCellRenderer {

  private static final long serialVersionUID = 425091150909034479L;

  public ModelTableRenderer() { }


  @Override
  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);

    if (!c.isVisible()) return c;

    Color color = Color.WHITE;

    LogLevel level = (LogLevel) table.getModel().getValueAt(row, LogModel.ERROR_COLUMN);
    switch (level) {
      case DEBUG:
        color = Color.GREEN;
        break;
      case ERROR:
        color = Color.ORANGE;
        break;
      case FATAL:
        color = Color.RED;
        break;
      case INFO:
        color = Color.WHITE;
        break;
      case TRACE:
        color = Color.GRAY;
        break;
      case WARN:
        color = Color.YELLOW;
        break;
      default:
        break;
    }

    if ((Boolean)table.getModel().getValueAt(row, LogModel.SUPPRESS_COLUMN)) {
      color = Color.BLACK;
    }

    c.setBackground(color);
    c.setForeground(Color.BLACK);
    table.repaint();
    return c;
  }
}

This is the results with no filters. All the rows are highlighted correctly

but when I click off "show trace" we can see now the first row is highlighted grey when it still should be green

Both components (highlighting and hiding) work on their own as intended but coupling them together has been a challenge. Any help would be great.

Answer 1

The age old problem with JTables... a view index is not a model index. Do not use a view index to lookup in the model and vice versa.

Look up JTable.convertRowIndexToView , JTable.convertRowIndexToModel , JTable.convertColumnIndexToView and JTable.convertColumnIndexToModel .

The indexes received in the renderer are view indexes. You are using the view index received to lookup in the model. Use the proper "convert" method first to convert to a model index, before looking up in the model.

Answer 2

Thanks to TT's suggestion i was able to fix the problem. I changed my renderer to:

public class ModelTableRenderer extends DefaultTableCellRenderer {

  private static final long serialVersionUID = 425091150909034479L;

  public ModelTableRenderer() { }


  @Override
  public Component getTableCellRendererComponent(JTable table, Object value, boolean isSelected, boolean hasFocus, int row, int column) {

    int modelRow = table.convertRowIndexToModel(row);
    int modelCol = table.convertRowIndexToModel(column);

    Component c = super.getTableCellRendererComponent(table, value, isSelected, hasFocus, modelRow, modelCol);

    Color color = Color.WHITE;

    LogLevel level = (LogLevel) table.getModel().getValueAt(modelRow, LogModel.ERROR_COLUMN);
    switch (level) {
      case DEBUG:
        color = Color.GREEN;
        break;
      case ERROR:
        color = Color.ORANGE;
        break;
      case FATAL:
        color = Color.RED;
        break;
      case INFO:
        color = Color.WHITE;
        break;
      case TRACE:
        color = Color.GRAY;
        break;
      case WARN:
        color = Color.YELLOW;
        break;
      default:
        break;
    }

    if ((Boolean)table.getModel().getValueAt(modelRow, LogModel.SUPPRESS_COLUMN)) {
      color = Color.BLACK;
    }

    c.setBackground(color);
    c.setForeground(Color.BLACK);
    return c;
  }
}

I also added a tableModelRenderer to render the suppression when the user clicks on the checkbox. Since this is the only value that changes, we can call repaint on that

model.addTableModelListener(new TableModelListener() {

  @Override
  public void tableChanged(TableModelEvent e) {
    table.repaint();
  }
});