Изменение цвета строки JTable в зависимости от значения столбца - при всплывающем щелчке - PullRequest
Новая
       17

Изменение цвета строки JTable в зависимости от значения столбца - при всплывающем щелчке

0 голосов
/ 04 мая 2011

Мой jtable загружен данными, и здесь я вызываю функцию всплывающих окон на jTable.

jTable.addMouseListener(new TablePopupListener(jTable));
displayTable();

Таким образом, если я щелкну правой кнопкой мыши по строке, появится всплывающее окно (проверка кредитоспособности), и если щелкнуть по нему, будет установлено значение для последней ячейки в этой строке.Теперь, основываясь на значении ячейки этого столбца, я должен определить цвет строки.Скажем, если значение ячейки не удалось, тогда поверните строку в красный, а в зеленый.Я попытался customCellRenderer и определил мое состояние, но нет никаких изменений в цвете строки.Пользовательский рендерер ячеек отлично работал для функциональности кнопок, которые мне пришлось написать.В приведенном ниже коде используется метод подготовки cellRenderer, который, по моему мнению, прост, но я не вижу каких-либо изменений в цвете строки.

Мне не хватает некоторого соединения, пожалуйста, помогите мне.

Заранее спасибо.

    class TablePopupListener extends MouseAdapter implements ActionListener { 
        JPopupMenu popup; 
        JTable table; 
        int[] selRows; 
        TableModel model; 
        ArrayList rowValueList = new ArrayList(); 
        JMenuItem creditCheck = new JMenuItem("Credit Check");

        public TablePopupListener(JTable jTable) {
            this.table = jTable;
            model = table.getModel();
            popup = new JPopupMenu();
            JMenuItem creditCheck = new JMenuItem("Credit Check");
            creditCheck.addActionListener(this);
            popup.add(creditCheck);

        }

        public void mousePressed(MouseEvent me) {
           firePopup(me);
        }

        public void mouseReleased(MouseEvent me) {
           firePopup(me);
        }

        public void firePopup(MouseEvent me) {

            /*
             * The popup menu will be shown only if there is a row selection in the
             * table
             */

            // popup.show(table, me.getX(), me.getY());
            if (me.isPopupTrigger() && table.getModel().getRowCount() != 0
               && table.getSelectedRow() != -1) {
              // popup.show(table,me.getX(),me.getY());
              if (me.isPopupTrigger()) {
                   JTable source = (JTable) me.getSource();
                   int row = source.rowAtPoint(me.getPoint());
                   int column = source.columnAtPoint(me.getPoint());

                   if (!source.isRowSelected(row))
                      source.changeSelection(row, column, false, false);

                   popup.show(table, me.getX(), me.getY());

              }
            }
         }

    public void actionPerformed(ActionEvent ae) {
        if (ae.getActionCommand().equals("Credit Check")) {
            System.out.println("you have clicked creditCheckpopup");
            selRows = table.getSelectedRows();
            if (selRows.length > 0) {
                for (int i = 0; i < selRows.length; i++) {
                  // get Table data
                    for (int j = 1; j < (table.getColumnCount()) - 1; j++) {
                        rowValueList.add(model.getValueAt(selRows[i], j));
                    }
                    System.out.println("Selection : " + rowValueList);
                }
            } else {
                System.out.println("you have clicked something idiot");
            }

            int result = new COpxDeal(rowValueList).CheckCredit();
            if (result == 1)
                rowValueList.add("pass");
            else
                rowValueList.add("fail");
            String aValue = (String) rowValueList.get(14);
            for (int i = 0; i < selRows.length; i++) {
                model.setValueAt(aValue, selRows[i], 15);
            }

     // inserted comment (Kleopatra): where are we? that's outside of the TablePopup?
     // okay, nothing like copying the code into an IDE and let that do the formatting, silly me ;-)
     // this is indeed _inside_ the popup, that is the table is recreated
            table = new JTable(model) {
                public Component prepareRenderer(TableCellRenderer renderer,
                        int row, int column) {
                    Component c = super.prepareRenderer(renderer, row, column);
                    JComponent jc = (JComponent) c;

                    // if (!isRowSelected(row)){
                    // c.setBackground(getBackground());
                    // System.out.println(isRowSelected(row));
                    // }
                    int modelRow = convertRowIndexToModel(row);
                    String strTestValue = "fail";
                    String strTblValue = (String) getModel().getValueAt(
                            modelRow, 15);
                    System.out.println("result :" + strTblValue);
                    if (strTblValue == null || strTblValue.equals(""))
                        System.out.println("there is nothing in strTblValue");
                    else if (strTestValue.equals(strTblValue)) {
                        jc.setBackground(Color.RED);
                    } else {
                        jc.setBackground(Color.green);
                    }

                    return c;
                }
            };

        }

    }
}

1 Ответ

2 голосов
/ 04 мая 2011

после некоторого форматирования (поверьте мне, важно, чтобы код был читабельным ;-) похоже, что вы создаете новую таблицу внутри вашего popupMenu, и только эта таблица имеет пользовательский рендерер.Что вы можете сделать, но не влияет на ваш реальный стол.

Переместите prepareRenderer в вашу реальную таблицу (ту, которую вы передаете во всплывающее окно в качестве параметра), и вы должны увидеть окраску.Осторожно: из-за ошибки в DefaultTableCellRenderer вы должны всегда устанавливать цвет, то есть 

 if (nothingToDo) {
     setBackground(normal)
 } else if ... {
     setBackground(one)
 } else {
     setBackground(other)
 }

Редактировать: пытаться объяснить изменения в структуре кода, фрагменты псевдокода

Текущийукажите, что вы делаете: 

JTable table = new JTable();
table.addMouseListener(new TablePopupListener(table));
     // keep listener-local reference to table
    JTable table = table;
    ....
    // in the listener guts, the reference is replaced
    table = new JTable() {
         @Override
         Component prepareRenderer(... 
    }

Измените на то, что вы должны сделать: 

 JTable table = new JTable() {
      @Override
      Component prepareRenderer(...
 };
 table.addMouseListener(new TablePopupListener(table));
     // keep listener-local reference to table
     JTable table = table;
     // don't replace ever, it's for reading only

edit 2: - изменил псевдокод, чтобы фактически зарегистрировать слушателя)- код с отступом ниже addMouseListener означает схему кода внутри TablePopupListener

...