Действие Java ActionListener не выполнено

У меня есть JButton, который должен удалить выбранную строку из JTable в моем графическом интерфейсе. Однако по какой-то причине код в ActionListener(), который я использую для этого, кажется, никогда не вызывается...

private void addListeners(){
    ...
    //removeBtn = new JButton("Remove");
    removeBtn.addActionListener(new ActionListener(){
        public void actionPerformed(ActionEvent e){
            System.out.println("Remove button clicked. Printed from actionListener. ");
            removeRow(e);
        }
    });

Метод removeRow() определен позже в том же классе:

public void removeRow(ActionEvent arg0){
    System.out.println("'Remove' button pressed, Printed from 'removeRow()' method. ");
    int selectedRow = jEntityFilterTable.getSelectedRow();
    DefaultTableModel model = (DefaultTableModel)jEntityFilterTable.getModel();
    model.removeRow(selectedRow);
    System.out.println("Selected row should have been removed. Printed from 'removeRow()' method. ");

removeBtn объявляется как глобальная переменная в верхней части класса с помощью строки:

private JButton removeBtn = new JButton("Remove");

Когда я запускаю код и нажимаю кнопку «Удалить» в графическом интерфейсе, ничего не происходит, и я даже не вижу отладки, отображаемой в консоли... но я не могу понять, почему - может ли кто-нибудь заметить, что я делаю неправильно? Почему метод actionListener/removeRow() не вызывается?

Изменить 07.07.2014 в 13:20

Вот полный код соответствующих частей класса по запросу:

public class JConfigurationPane extends JPanel implements UndoableEditListener, ChangeListener, ActionListener
{
    ...
    private JButton addBtn = null;
    private JButton saveBtn = null;
    private JButton removeBtn = new JButton("Remove"); //null;
    private JButton editBtn = null;
    public boolean addBtnClicked = false;
    public boolean saveBtnClicked = false;
    public boolean removeBtnClicked = false;
    public boolean editBtnClicked = false;
    /**
     * This method initializes 
     * 
     */
    public JConfigurationPane(ConfigurationDataModel dataModel)
    {
    super();

    this.dataModel = dataModel;

    initialize();
    initialiseData();
    addListeners();
}

public JConfigurationPane()
{
    super();

    initialize();

    addListeners();
}

private void addListeners()
{
    System.out.println("--- 'addListeners()' method has been called. ---");
    jcbRxFilterExcludes.addChangeListener(this);

    docRxAddress = jfRxAddress.getDocument();
    docRxPort = jfRxPort.getDocument();
    docRxExerciseID = jfRxExerciseID.getDocument();
    docRxMaxPduSize = jfRxMaxPduSize.getDocument();

    docRxAddress.addUndoableEditListener(this);
    docRxPort.addUndoableEditListener(this);
    docRxExerciseID.addUndoableEditListener(this);
    docRxMaxPduSize.addUndoableEditListener(this);

    addBtn.addActionListener(this);
    /*Add action listeners for other buttons (07/07/2014 @ 08:35) 
    saveBtn.addActionListener(this);
    removeBtn.addActionListener(this);
    editBtn.addActionListener(this);

    Causes an "Exception in thread 'main', java.lang.nullPointerException on: 
        'saveBtn.addActionListener(this);
        'addListeners();' call in 'public JConfigurationPane()'
        'JConfigurationPane panel = new JConfigurationPane()' call in 'main(String[] args)'
    */
    //removeBtn = new JButton("Remove");
    removeBtn.addActionListener(new ActionListener(){
        public void actionPerformed(ActionEvent e){
            System.out.println("Remove button clicked. Printed from actionListener. ");
            removeRow(e);
        /*  System.out.println("'Remove' button pressed. ");
            int selectedRow = jEntityFilterTable.getSelectedRow();
            DefaultTableModel model = (DefaultTableModel)jEntityFilterTable.getModel();
            model.removeRow(selectedRow);
            System.out.println("Selected row should have been removed. "); */
        }
    });
}

private void initialiseData()
{
    ...
}

/**
 * This method initializes this
 * 
 */
private void initialize() {
    ...

    /*Create filter buttons */
    System.out.println("------------------JButtons BEING CREATED---------------------");
    addBtn = new JButton("Add");
    addBtn.setBounds(950, 135, 125, 25);
   //     addBtn.addActionListener(new ActionListener(){
//      public void actionPerformed(ActionEvent e){

//      }
//  });
    JButton saveBtn = new JButton("Save");
    saveBtn.setBounds(1100, 135, 125, 25);
    JButton removeBtn = new JButton("Remove");
    removeBtn.setBounds(950, 200, 125, 25);
    JButton editBtn = new JButton("Edit");
    editBtn.setBounds(1100, 200, 125, 25);
    System.out.println("------------------JButtons CREATED-------------------------");

    ...

    /*Add filter buttons */
    System.out.println("-------------------JButtons BEING ADDED TO GUI--------------------");
    this.add(addBtn);
    this.add(saveBtn);
    this.add(removeBtn);
    this.add(editBtn);
    System.out.println("------------------JButtons ADDED TO GUI---------------------");
}

/**
 * This method initializes jfRxAddress  
 *  
 * @return javax.swing.JTextField   
 */
private JTextField getJfRxAddress()
{
    ...
}

/**
 * This method initializes jfRxPort 
 *  
 * @return javax.swing.JTextField   
 */
private JTextField getJfRxPort()
{
    ...
}

/**
 * This method initializes jfRxExerciseID   
 *  
 * @return javax.swing.JTextField   
 */
private JTextField getJfRxExerciseID()
{
    ...
}

/**
 * This method initializes jEntityFilterPane    
 *  
 * @return javax.swing.JScrollPane  
 */
private JScrollPane getJEntityFilterPane()
{
    ...
}

/**
 * This method initializes jEntityFilterTable   
 *  
 * @return javax.swing.JTable   
 */
private JTable getJEntityFilterTable()
{
    ...
}

/**
 * This method initializes jEntitySymbolsPane   
 *  
 * @return javax.swing.JScrollPane  
 */
private JScrollPane getJEntitySymbolsPane()
{
    ...
}

/**
 * This method initializes jEntitySymbolsTable  
 *  
 * @return javax.swing.JTable   
 */
private JTable getJEntitySymbolsTable()
{
    ...
}

/*Method to add buttons to 'Plugin Configuration' window */
private void addButtons(){
    /*Buttons moved up to initialize() method on 25/06/2014 @ 17:00 */
    JButton addBtn = new JButton("Add");
    JButton saveBtn = new JButton("Save");
    JButton removeBtn = new JButton("Remove");
    JButton editBtn = new JButton("Edit");

    addBtn.addActionListener(new ActionListener(){
        public void actionPerformed(ActionEvent e){
            /*(25/06/2014 @ 15:20) Need to add code here to add a new editable row to 'Entity Filter' table */
            addBtnClicked = true;
        }
    });
    //addBtn.setBounds(1150, 135, 30, 15);
    saveBtn.addActionListener(new ActionListener(){
        public void actionPerformed(ActionEvent e){
            /*(25/06/2014 @ 15:20) Need to add code here to save the data in the 'Entity Filter' table to a set of variables */
            saveBtnClicked = true;
        }
    });
    //saveBtn.setBounds(1190, 135, 30, 15);
    removeBtn.addActionListener(new ActionListener(){
        public void actionPerformed(ActionEvent e){
        /*(25/05/2014 @ 15:25) Need to add code here to remove the data in selected row from variables, and remove row from table */
            removeBtnClicked = true;
        }
    });
    //removeBtn.setBounds(1150, 160, 30, 15);
    editBtn.addActionListener(new ActionListener(){
        public void actionPerformed(ActionEvent e){
            /*(25/06/2014 @ 15:25) Need to add code here to enable editing of data in selected row */
            editBtnClicked = true;
        }
    });
    //editBtn.setBounds(1190, 160, 30, 15);
    System.out.println("'addButtons()' method is being called by 'initialize()' in JConfigurationPanel.java");
}

/**
 * This method initializes jcbRxFilterExcludes  
 *  
 * @return javax.swing.JCheckBox    
 */
private JCheckBox getJcbRxFilterExcludes()
{
    ...
}

/**
 * This method initializes jfRxMaxPduSize   
 *  
 * @return javax.swing.JTextField   
 */
private JTextField getJfRxMaxPduSize()
{
    ...
}

@Override
public void undoableEditHappened(UndoableEditEvent editEvent)
{
    ...
}

@Override
public void stateChanged(ChangeEvent changeEvent)
{
    ...
}

public static void main(String[] args) {
    JFrame frame = new JFrame();
    frame.setMinimumSize(new Dimension(500,500));
    frame.setSize(new Dimension(500,500));

    JConfigurationPane panel = new JConfigurationPane();

    frame.add(panel);
    frame.pack();
    frame.setVisible(true);
}

@Override
public void actionPerformed(ActionEvent arg0) {
    /*Check which button has been pressed, perform a different action depending on which button it was. Reset the check variables
     * to false after performing action */

    // TODO Auto-generated method stub
    /*Code to add row to table when button is pressed */
    System.out.println("'Add' button pressed.");

    DefaultTableModel model = (DefaultTableModel)jEntityFilterTable.getModel();

    Object[] obj = new Object[]{}; 

    System.out.println("Row Count: " + model.getRowCount());
    System.out.println("1st Column: " + model.getColumnName(0));
    System.out.println("Column Count: " + model.getColumnCount());

    model.addRow(obj);

    System.out.println("--- ActionListener added to 'addBtn' ---");

    /*Code to remove selected row from the table when button is clicked 
    int selectedRow = jEntityFilterTable.getSelectedRow();
    model.removeRow(selectedRow); */

}

public void removeRow(ActionEvent arg0){
    System.out.println("'Remove' button pressed. Printed from 'removeRow()' method. ");
    int selectedRow = jEntityFilterTable.getSelectedRow();
    DefaultTableModel model = (DefaultTableModel)jEntityFilterTable.getModel();
    model.removeRow(selectedRow);
    System.out.println("Selected row should have been removed. Printed from 'removeRow()' method. ");
}
}

person Noble-Surfer    schedule 07.07.2014    source источник
comment
Недостаточно кода. Вы должны предоставить stackoverflow.com/help/mcve . Но могу поспорить, что с помощью removeBtn = new JButton("Remove"); вы создаете новую кнопку, которая отличается от той, что вы видите на экране (которая могла быть создана где-то еще...)   -  person Marco13    schedule 07.07.2014
comment
Убедитесь, что вы по ошибке не заменили где-нибудь JButton, чтобы в итоге вы получили кнопку, к которой не подключен ActionListener.   -  person Kayaman    schedule 07.07.2014
comment
Ранее я объявил removeBtn глобальной переменной с private JButton removeBtn = null;. Однако, учитывая ваш комментарий, я попытался изменить глобальное объявление на private JButton removeBtn = new JButton("Remove");, а затем удалить строку removeBtn = new JButton("Remove"); из кода в методе addListeners(), однако у меня все еще есть та же проблема: ничего не происходит, когда я нажимаю кнопку «Удалить» в графическом интерфейсе. ..   -  person Noble-Surfer    schedule 07.07.2014
comment
Глупый вопрос: вы действительно вызываете addListeners() где-то в своем коде?   -  person tobias_k    schedule 07.07.2014
comment
Да, я вызываю addListeners() в конструкторе.   -  person Noble-Surfer    schedule 07.07.2014
comment
Будет намного лучше, если вы разместите весь свой код здесь. И дайте нам знать.   -  person gprathour    schedule 07.07.2014
comment
Класс довольно большой, поэтому я вырежу все остальное и просто опубликую соответствующие части.   -  person Noble-Surfer    schedule 07.07.2014


Ответы (3)


В вашем методе initialize() removeBtn, которое вы добавляете на панель, является локальной переменной этого метода. То же самое происходит и в методе addButtons(). Затем, позже, когда вы вызываете addListeners(), вы добавляете ActionListener к частному члену JButton, который не был добавлен в панель. Удалите локальный JButton removeBtn из метода initialize() и вместо этого инициализируйте член.

person Omaha    schedule 07.07.2014
comment
Эй, спасибо за ваш ответ и за указание на эти проблемы. Я сделал то, что вы предложили, - избавился от всех «новых» частных JButtons, которые я создал, и вместо этого использовал элемент. Однако, когда я сейчас запускаю свой код, после того, как весь код инициализации запущен (и я получаю сообщения... будучи созданными... и метод addListeners() вызывается сообщением в консоли), я получаю исключение в потоке main с исключением нулевого указателя снова. На этот раз это происходит в строке dataModel.setEntityFilterExcludes(jcbRxFilterExcludes.isSelected()); в методе stateChanged() - person Noble-Surfer; 07.07.2014
comment
Что ж, хорошо, ваша первоначальная проблема решена, но если вы ищете совета по новой проблеме, вам нужно опубликовать содержание метода stateChanged(). - person Omaha; 07.07.2014

Попробуйте объявить кнопку как финальное поле. Если какая-то часть вашего кода изменит ваш объект кнопки, вы получите предупреждение.

person Raphoq    schedule 07.07.2014
comment
Я знаю, что вы еще не можете комментировать, но это действительно должен быть комментарий... хорошая идея, однако. - person tobias_k; 07.07.2014
comment
Я попробовал это, но это не имело никакого значения для того, что происходит, когда я запускаю код... - person Noble-Surfer; 07.07.2014

с исключением нулевого указателя снова.

В вашем коде вы используете:

JConfigurationPane panel = new JConfigurationPane();

вместо:

public JConfigurationPane(ConfigurationDataModel dataModel)

Почему у вас два конструктора? Зачем вам вообще передавать модель данных в класс?

Кроме того, не используйте setBounds() для установки размера/расположения ваших кнопок. Swing был разработан для использования с менеджерами компоновки, поэтому используйте менеджеры компоновки. Возможно, вы бы начали с панели, которая использует FlowLayout.

Наконец, следующий код (вероятно) не будет работать:

model.removeRow(selectedRow);

Выбранная строка относится к строке в представлении таблицы, а не к строке в модели таблицы. Если ваш код выполняет сортировку или фильтрацию, индекс может быть другим. Сначала вам нужно преобразовать выбранный индекс, используя:

JTable.convertRowIndexToModel(...);
person camickr    schedule 07.07.2014