JAVA:JMenuItem上的Swing-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类在循环中检查值时，我只会从状态列表中获得"false"值。我使用Swing工具栏以类似的方式实现了这个模式，它运行良好，但我无法理解为什么我无法检索由动作侦听器事件正确设置的值。我看不到覆盖值的地方，也看不到对象实例化的奇怪顺序/混合的地方。

因此，问题是：为什么在测试状态列表时只得到false

更新：尝试SSCCE

---

这是带有相关类的UML类图

下面是相关类的实现。

State.java

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();   
}
}

StatesList.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+="listn";
for (State s : states) {
rString+=s.toString()+"n";
}
return rString;
}
}

MenuBar.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;
}

}

MenuBarBuilder.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();
}
}

GUI.java

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
}
}
}

启动.java

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());

这里创建了两个不同的菜单栏。作为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());

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