JAVA：Swing - JMenuItem 上的 actionPerformed() 触发但更新的值不持久

Question

好的，所以我一直在为我一直在开发的 opengl 图形引擎构建一个简单的基于 Java Swing 的 GUI。

我遇到的问题是菜单栏没有按照我的意愿运行。 具体来说，我有一个 MenuBarBuilder 类来构建各种菜单、子菜单和菜单项。 当一个菜单项被添加时，它被分配一个新的 actionEventListener 并被赋予一个简单的任务，将一个布尔值设置为 true； 此布尔值存储在与该菜单关联的状态列表中。 每个菜单项状态都是在将菜单项添加到菜单时创建的。 状态作为“State”类型的对象存储在“StatesList”类中​​。

在 MenuBarBuilder 类中，需要作为一个单元运行的每个菜单项都被添加为一个“组”。 完成此操作后，将向每个菜单项添加第二个操作事件侦听器，将菜单组的所有关联值设置为 false。 最终，这会确保组中只有一个菜单项在状态列表中具有设置为 true 的状态。

单击菜单项时，将触发动作侦听器事件。 当被触发时，它们会打印出一个列表来确定它们被触发的顺序并确保状态值按预期设置，见下文：

setfalse
list
MenuItem1Name: false
MenuItem2Name: false

MenuItem1Name
setTrue
list
MenuItem1Name: true
MenuItem2Name: false

setfalse
list
MenuItem1Name: false
MenuItem2Name: false

MenuItem2Name
setTrue
list
MenuItem1Name: false
MenuItem2Name: true

这可确保事件以正确的顺序触发。 我们可以看到，当MenuItem1Name按钮被按下时，值被正确设置为 false，相关字段随后更新为 true。 同样可以看到MenuItem2Name 。 现在开始解决实际问题。

当我获取状态列表并通过 GUI 类检查循环中的值时，我只能从状态列表中获得“假”值。 我已经使用 Swing 工具栏以类似的方式实现了这个模式，它工作正常，但我根本无法理解为什么我无法检索由动作侦听器事件正确设置的值。 我看不到值被覆盖的地方或对象实例化的奇怪顺序/混合的地方。

所以，现在的问题是：为什么测试状态列表时，我只拿到假的？

更新：尝试 SSCCE

---

这是带有相关类的 UML 类图

下面是相关类的实现。

状态文件

public class State {

    private String name;
    private Object value;

    State(String name, Object value){
        this.name=name;
        this.value=value;
    }

    public String getName() {
        return name;
    }

    public void setName(String name) {
        this.name = name;
    }

    public Object getValue() {
        return value;
    }

    public void setValue(Object value) {
        this.value = value;
    }

    public String toString(){   
        return getName() + ": " + getValue();   
    }

}

状态列表.java

import java.util.ArrayList;

public class StatesList {

    private ArrayList<State> states;

    StatesList(){
        states= new ArrayList<State>();
    }

    void addState(String name, Object value){
        states.add(new State(name, value));
    }

    State getState(String name){
        for(State s : states){
            if(s.getName().equals(name)){
                return s;
            }
        }
        return null;
    }

    ArrayList<State> getStates() {
        return states;
    }

    public String toString(){
        String rString="";
        rString+="list\n";
        for (State s : states) {
            rString+=s.toString()+"\n";
        }
        return rString;
    }

}

菜单栏.java

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;

public class MenuBar {

    private JMenuBar menuBar;
    private StatesList menustates;

    public MenuBar() {
        menuBar = new JMenuBar();
        menustates = new StatesList();
    }

    void createMenu(String title) {
        JMenu menu = new JMenu(title);
        menuBar.add(menu);
    }

    void createMenuItem(JMenu menu, String itemTitle) {

        JMenuItem menuItem = new JMenuItem(itemTitle);

        menustates.addState(itemTitle, false);

        menuItem.addActionListener(new ActionListener() {
            int menuItemindex = menustates.getStates().size()-1 ;

            @Override
            public void actionPerformed(ActionEvent actionEvent) {
                System.out.println(actionEvent.getActionCommand());
                menustates.getStates().get(menuItemindex).setValue(true);
                //print list of current states
                System.out.println("setTrue");
                System.out.println(menustates.toString());
            }
        });
        menu.add(menuItem);
    }

    //not a real group, just ensures MenuItem behaves the same as other menu items
    void addMenuGroup(JMenuItem[] jMenuItems, int startComponentIndex) {    

        for (int i = 0; i < jMenuItems.length; i++) {
            jMenuItems[i].addActionListener(new ActionListener() {
                @Override
                public void actionPerformed(ActionEvent e) {
                    for (int i2 = 0; i2 < jMenuItems.length; i2++) {
                        //make sure each state related to the menuItemGroup is to false
                        menustates.getStates().get(i2 + startComponentIndex).setValue(false);
                    }
                    //print list of current states
                    System.out.println("setfalse");
                    System.out.println(menustates.toString());
                }
            });
        }
    }

    JMenuBar getMenuBar() {
        return menuBar;
    }

    public StatesList getStates() {
        return menustates;
    }


}

菜单栏生成器.java

import javax.swing.JMenuBar;
import javax.swing.JMenuItem;

public class MenuBarBuilder {

    private MenuBar DataViewMenuBar;

    MenuBarBuilder() {
        DataViewMenuBar = buildDataViewMenu();
    }

    MenuBar buildDataViewMenu() {
        MenuBar tb = new MenuBar();

        //create menu
        tb.createMenu("TestScenes");

        //add two menuItems
        tb.createMenuItem(tb.getMenuBar().getMenu(0), "MenuItem1Name");
        tb.createMenuItem(tb.getMenuBar().getMenu(0), "MenuItem2Name");

        //add buttons two a 'group' sort of
        tb.addMenuGroup(new JMenuItem[] { 
                (JMenuItem) tb.getMenuBar().getMenu(0).getItem(0),
                (JMenuItem) tb.getMenuBar().getMenu(0).getItem(1), },
                tb.getStates().getStates().size() - 2);

        return tb;
    }

    StatesList getDataViewMenuBarStates() {
        return DataViewMenuBar.getStates();
    }

    JMenuBar getDataViewMenuBarMenubar() {
        return DataViewMenuBar.getMenuBar();
    }

}

图形用户界面

import javax.swing.JFrame;

public class GUI {

    MenuBarBuilder mbb;
    boolean Selected = false;

    GUI() {
        mbb = new MenuBarBuilder();
    }

    void run() {
        final JFrame frame = new JFrame("Main");
        frame.setJMenuBar(mbb.buildDataViewMenu().getMenuBar());

        frame.pack();
        frame.setVisible(true);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

        while (!Selected) {
            selectLoadScene();
        }
        System.out.print("bap- selection must have occured");
    }

    void selectLoadScene() {
        String selectedTest = "";


    // The Test Occurs here
    // loop through states list
        System.out.println(mbb.getDataViewMenuBarStates().getStates().toString());
        for (State s : mbb.getDataViewMenuBarStates().getStates()) {
            // check the value for each state -this value is always false,..
            // thats the problem
            if ((boolean) s.getValue() == true) {
                // if true save value
                selectedTest = s.getName();
                break;
            }
        }

        // test selected value and see if it matches the following cases
        switch (selectedTest) {
        case "MenuItem1Name":
            // dosomething
            Selected = true; // breaks while loop
            break;
        case "MenuItem2Name":
            // dosomething
            Selected = true;// breaks while loop
            break;
        default:
            // do nothing
        }
    }
}

启动文件

public class Start {

    GUI gui;

    public static void main(String[] args) {
        GUI gui= new GUI(); 
        gui.run();
    }

}

Answer 1

while 循环中的菜单栏与操作执行方法中的菜单栏不同。 为什么？ 如果您仔细查看类 GUI 中的以下 2 行，您就会知道。

mbb = new MenuBarBuilder();
frame.setJMenuBar(mbb.buildDataViewMenu().getMenuBar());

这里创建了 2 个不同的菜单栏。 作为 MenuBarBuilder 的构造函数，内部调用 buildDataViewMenu()，因此 2 次调用“buildDataViewMenu()”。 这将最终创建 2 个菜单栏。

为避免这种情况

不要在中调用“buildDataViewMenu()”

frame.setJMenuBar(mbb.buildDataViewMenu().getMenuBar());

由于您只需要菜单栏，因此您应该以这样一种方式管理代码，即您将获得 this.tb = new MenuBar() 的句柄，您可以为此添加 getter 和 setter。

当我添加与下面相同的 getter 和 setter 时，它开始工作。

public class MenuBarBuilder {

private MenuBar DataViewMenuBar;
MenuBar tb;

MenuBarBuilder() {
    DataViewMenuBar = buildDataViewMenu();
}

MenuBar buildDataViewMenu() {
   this.tb = new MenuBar();

    //create menu
    tb.createMenu("TestScenes");

    //add two menuItems
    tb.createMenuItem(tb.getMenuBar().getMenu(0), "MenuItem1Name");
    tb.createMenuItem(tb.getMenuBar().getMenu(0), "MenuItem2Name");

    //add buttons two a 'group' sort of
    tb.addMenuGroup(new JMenuItem[] { 
            (JMenuItem) tb.getMenuBar().getMenu(0).getItem(0),
            (JMenuItem) tb.getMenuBar().getMenu(0).getItem(1), },
            tb.getStates().getStates().size() - 2);

    return tb;
}

public MenuBar getMenuBar(){
    return this.tb;
}

StatesList getDataViewMenuBarStates() {
    return DataViewMenuBar.getStates();
}

JMenuBar getDataViewMenuBarMenubar() {
    return DataViewMenuBar.getMenuBar();
}

}

在课堂上 GUI 调用应该是这样的，

frame.setJMenuBar(mbb.getMenuBar().getMenuBar());

这将解决问题。 如果您遇到任何问题，请告诉我。