ActionListener 失败 - Java For-Each 分配给 JMenuItem

Question

我之前以更大的形式发布了这个问题，并希望以一种允许社区更好地帮助我的方式再次发布这个问题，减少关于你们所有人的信息。

基本上，我有这段代码，解释如下：

for(JMenuItem x : chem3_x_y){
  x.addActionListener(new ActionListener(){
    public void actionPerformed(ActionEvent k) {
      new SwingImplementation(1, tt+1);
    }
  }); 
  gEleven[6].add(x);
  tt++;
}
tt=0;

首先，我循环遍历数组 chem3_x_y 中的所有 JMenuItem-s

然后，我将一个 ActionListener 添加到“x”或数组的每个项目，这将创建一个新的 SwingImplementation，其参数为 1，变量“tt”

在此之后，JMenu gEleven 接收到有问题的 JMenuItem，然后我将 tt 增加 1。

上述代码的目的是自动化将 JMenuItems 添加到 gEleven 和 ActionListeners 到每个菜单项的过程。

问题，为变量 'tt' 传递的参数始终相同，为 1，就好像将 'tt' 参数为 1 的动作侦听器应用于每个 JMenuItem - 为什么会这样，以及如何我使用不断增加的 tt 创建一个 ActionListener per JMenuItem，参数为 (1, tt) 而 tt 始终为 1

Answer 1

您的问题是 tt 在调用侦听器之前不会递增。虽然您的动作侦听器代码是内联的，但事实上，它并非全部同步运行 - 直到调用动作侦听器才会调用 actionPerformed()。所以tt 在你循环时永远不会改变。

你想要的是这样的：

class MyListener implements ActionListener {
  private final int tt;

  MyListener (int tt) { 
    this.tt = tt;
  }

  public void actionPerformed(ActionEvent k) {
     new SwingImplementation(1, tt+1);
  }
}

for(JMenuItem x : chem3_x_y){
  x.addActionListener(new MyListener(tt++)); 
  gEleven[6].add(x);
}

这样您就可以递增 tt 并将结果存储在与需要该值的侦听器关联的 final 字段中。

Answer 2

编辑：你可能指的是tt++，而不是tt + 1。我无法从发布的代码中看出。也许，寻找一个隐藏预期变量的tt。基于这个example，下面对应的代码使用++i。也可以考虑使用Action 来封装你的监听器。

import java.awt.BorderLayout;
import java.awt.EventQueue;
import java.awt.event.ActionEvent;
import java.io.File;
import javax.swing.AbstractAction;
import javax.swing.Action;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JToolBar;

/**
 * @see https://stackoverflow.com/a/19626219/230513
 * @see https://stackoverflow.com/questions/4038605
 */
public class FileMenu {

    public static void main(String[] args) {

        EventQueue.invokeLater(new Runnable() {

            public void run() {
                new FileMenu().create();
            }
        });
    }

    void create() {
        File userDir = new File(System.getProperty("user.dir"));
        File[] files = userDir.listFiles();

        JMenu menu = new JMenu("Recent Files");
        JToolBar toolBar = new JToolBar(JToolBar.VERTICAL);
        JLabel label = new JLabel(" ", JLabel.CENTER);
        int i = 0;
        for (File f : files) {
            if (f.isFile() && !f.isHidden()) {
                RecentFile rf = new RecentFile(f, label, ++i);
                menu.add(new JMenuItem(rf.getAction()));
                toolBar.add(rf);
            }
        }
        JMenuBar menuBar = new JMenuBar();
        menuBar.add(menu);

        JFrame f = new JFrame("FileMenu");
        f.setJMenuBar(menuBar);
        f.add(toolBar, BorderLayout.CENTER);
        f.add(label, BorderLayout.SOUTH);
        f.pack();
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        f.setLocationRelativeTo(null);
        f.setVisible(true);
    }
}

class RecentFile extends AbstractAction {

    private final File file;
    private final JLabel label;

    public RecentFile(final File file, final JLabel label, int i) {
        this.file = file;
        this.label = label;
        this.putValue(Action.NAME, String.valueOf(i) + " " + file.getName());
        this.putValue(Action.SHORT_DESCRIPTION, file.getAbsolutePath());
    }

    public void actionPerformed(ActionEvent e) {
        label.setText(file.getName());

    }

    public Action getAction() {
        return this;
    }
}