Добавление элемента в JComboBox из другого класса

Question

Я пытался заставить JComboBox обновляться при добавлении нового элемента в базу данных бэкэнда.

В реальном коде есть отдельный класс, который обрабатывает диалог добавления, когда новый элементдобавил, он обновляет базу данных, а затем должен добавить тот же элемент в раскрывающееся меню, вызывая метод, который принимает строку в главном классе GUI.(пытается следовать Model-View-Controller).

Ниже приведен минимальный пример, который вызывает ошибку, хотя в реальном приложении она молча терпит неудачу.

У меня есть подозрение, что это связано сэкземпляры объектов.Также, чтобы заполнить список, я перебираю список, используя addItem (), чтобы убедиться, что он работает, и ComboBox изменчив.

Спасибо за любую помощь, Том

import java.awt.*;
import javax.swing.*;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;
public class TestComboBox extends JPanel implements ActionListener{

JComboBox moduleList = new JComboBox(new DefaultComboBoxModel());
TestComboBox testComboBox;
JFrame frame;

public void actionPerformed(ActionEvent e){
    if("additem".equals(e.getActionCommand())){
        addItem("Item");
    }
    if("additemfail".equals(e.getActionCommand())){
        testComboBox.addItemFail("Item Fail");
    }
}

public void addItem(String item){
    moduleList.addItem(item);
}

public void addItemFail(String item){
    testComboBox = new TestComboBox();
    moduleList.addItem(item);
}


protected JPanel createPanel(){
    JPanel panel = new JPanel(false);

    String[] getModuleList = {"MODULE 1", "MODULE 2"};
    moduleList = new JComboBox(new DefaultComboBoxModel(getModuleList));
    panel.add(moduleList);

    JButton additem = new JButton("Add Item");
    additem.setActionCommand("additem");
    additem.addActionListener(this);
    panel.add(additem);

    JButton additemfail = new JButton("Add Item Fail");
    additemfail.setActionCommand("additemfail");
    additemfail.addActionListener(this);
    panel.add(additemfail);

    return panel;
}

public void createAndShowGui(){
    testComboBox = new TestComboBox();
    frame = new JFrame("JComboTest");
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

    frame.add(testComboBox.createPanel());
    frame.setSize(450, 150);
    frame.setVisible(true);
}

public static void main(String[] args){
    TestComboBox t = new TestComboBox();
    t.createAndShowGui();
}
}

Исключение, котороеброшенный

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at TestComboBox.actionPerformed(TestComboBox.java:16)
    at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2012)
    at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2335)
    at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:404)
    at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:259)
    at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:253)
    at java.awt.Component.processMouseEvent(Component.java:6268)
    at javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
    at java.awt.Component.processEvent(Component.java:6033)
    at java.awt.Container.processEvent(Container.java:2045)
    at java.awt.Component.dispatchEventImpl(Component.java:4629)
    at java.awt.Container.dispatchEventImpl(Container.java:2103)
    at java.awt.Component.dispatchEvent(Component.java:4455)
    at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4633)
    at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4297)
    at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4227)
    at java.awt.Container.dispatchEventImpl(Container.java:2089)
    at java.awt.Window.dispatchEventImpl(Window.java:2517)
    at java.awt.Component.dispatchEvent(Component.java:4455)
    at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:649)
    at java.awt.EventQueue.access$000(EventQueue.java:96)
    at java.awt.EventQueue$1.run(EventQueue.java:608)
    at java.awt.EventQueue$1.run(EventQueue.java:606)
    at java.security.AccessController.doPrivileged(Native Method)
    at java.security.AccessControlContext$1.doIntersectionPrivilege(AccessControlContext.java:105)
    at java.security.AccessControlContext$1.doIntersectionPrivilege(AccessControlContext.java:116)
    at java.awt.EventQueue$2.run(EventQueue.java:622)
    at java.awt.EventQueue$2.run(EventQueue.java:620)
    at java.security.AccessController.doPrivileged(Native Method)
    at java.security.AccessControlContext$1.doIntersectionPrivilege(AccessControlContext.java:105)
    at java.awt.EventQueue.dispatchEvent(EventQueue.java:619)
    at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:275)
    at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:200)
    at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:190)
    at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:185)
    at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:177)
    at java.awt.EventDispatchThread.run(EventDispatchThread.java:138)

Answer 1

Ваша проблема в том, что ваш класс создает внутри себя еще один экземпляр класса. Сначала вы создаете TestComboBox (TCB) со ссылкой на другой TestCombobox, который имеет значение Null , поскольку вы нигде не устанавливали его для чего-либо еще.

Первый TCB (A) - это ваш t с нулевой ссылкой TCB, созданной в main ().

Эта TCB ссылка устанавливается путем создания нового TCB (B) (, который случайно содержит другую нулевую ссылку tcb ) в createAndShowGui (). Это также тот, который получает все панели, а также всех слушателей.

Поэтому, когда вы нажимаете эту кнопку на панели, событие события запускается и принимается B, так как оно является единственным со слушателями. Но затем он пытается получить доступ к каналу TCB в B, который равен нулю , вызывая исключение nullpointerexception.

Еще больше путаницы вызывает тот факт, что ваш addItemFail создает еще один TCB, который добавляется в B.

Я предлагаю вам переосмыслить и перепроектировать это. Это слишком запутанно в данный момент. Кроме того, в коде нет ни одного комментария, который делает исходное намерение за кодом еще более трудным для понимания. Вы захотите избавиться от всех новых творений создания TCB внутри класса.

Answer 2

Вместо testComboBox.addItemFail("Item Fail");, вы должны просто вызвать addItemFail("Item Fail"); в вашем методе actionPerformed.

Редактировать: И вам не следует заново создавать объект того же класса в методе addItemFail (какой смысл делать это снова?)