java - Action 监听器不适用于某些按钮

Question

“重新启动”我的 map 后，我无法让 actionlistener 在某些按钮上工作。问题是 btns[1] 到 btns[9] 根本不起作用!我不确定我的变量可能有问题。我尝试了一切。在我启动新 map 后，这些按钮根本不起作用(在我按下按钮 btns[0] .. 新 map 后)。这是我的代码，希望你能帮助我。 New Board(null) - 如果构造函数中有 null，它应该创建毯子 map ，我已经在 Board 类的构造函数中编码(我想这并不重要，因为它在我第一次启动新游戏时工作(空))。

如果您发现无法找出它不起作用的原因，我将我的整个游戏上传到 sendspace = http://www.sendspace.com/file/pvwtoo - jar 形，http://www.sendspace.com/file/l18khb - BlueJ 形式 - 必要时更好地协调。非常感谢您的帮助。卢克

import javax.swing.*;
import java.awt.event.*;
import java.awt.*;

public class Game extends JFrame implements ActionListener
{
Board b;
Menu m;
Container c = getContentPane();
JPanel pnl;
ImageIcon ii;
JLabel jl;
JTextArea jt;
private JButton [] btns = new JButton[10];
String selectMore = "Select more = false";
int posx,posy;

public Game(Map m) {
    createGui(m);
}

public void createGui(Map mm)
{
    b = new Board(mm);
    //m = new Menu();
    c.add(b);

    pnl = new JPanel();
    ii = new ImageIcon(this.getClass().getResource("menu.png"));
    pnl.setLayout(new GridLayout(16,10));
    pnl.setSize(100,608);
    //pnl.add(m);
    pnl.setBackground(Color.BLACK);
    c.add(pnl,BorderLayout.LINE_END);

    setTitle("Strgame");
    c.setBackground(Color.BLACK);  

    //this.pack();
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setSize(800, 608);
    setLocationRelativeTo(null);
    setResizable(false);

    //setUndecorated(true); 
    //setExtendedState(Frame.MAXIMIZED_BOTH); 

    setVisible(true);


    manageButtons();
    infoPanel();
    pnl.requestFocusInWindow();
    requestFocusInWindow();
}

public void manageButtons()
{
    if(pnl!=null){pnl.removeAll();}
    btns[0] = new JButton("New Map");
    btns[1] = new JButton("Change XY to House 1");
    btns[2] = new JButton("Change XY to House 2");
    btns[3] = new JButton("Change XY to Road 1");
    btns[4] = new JButton("Change XY to Road 2");
    btns[5] = new JButton("Change XY to Road 3");
    btns[6] = new JButton("Change XY to Grass");
    btns[7] = new JButton("Get info of selected");
    btns[9] = new JButton(selectMore);
    for(int i = 0; i < 10; i++)
    {
        if(btns[i] != null)
        {
            btns[i].addActionListener(this);
            btns[i].setPreferredSize(new Dimension(213,10));
            btns[i].setBorder(BorderFactory.createLineBorder(Color.BLACK, 1));
            btns[i].setBackground(Color.lightGray);
            pnl.add(btns[i]);
        }
    }
}

public void infoPanel()
{
    jl = new JLabel("Info");
    jl.setForeground(Color.lightGray);
    jl.setHorizontalAlignment(SwingConstants.CENTER);
    pnl.add(jl);        
    jt = new JTextArea();
    //jt.setPreferredSize(new Dimension(211,10));
    //jt.setBorder(BorderFactory.createLineBorder(Color.BLACK, 1));
    jt.setBackground(Color.lightGray);
    //jt.setHorizontalAlignment(SwingConstants.CENTER);
    jt.setMargin(new Insets(2,2,2,2));
    jt.setEditable(false);
    jt.setText(" Not selected");
    pnl.add(jt);
}


public int posx()
{
    int [] pom = b.lastPosSelected(); 
    posx = pom[0];        
    return posx;
}

public int posy()
{
    int [] pom = b.lastPosSelected();   
    posy = pom[1];        
    return posy;
}

public void actionPerformed(ActionEvent e)
{
    String action = e.getActionCommand();  
    if(action.equals("New Map"))
    {
        createGui(null);
        b.revalidate();
        b.repaint();
        c.repaint();
    }
    else if(action.equals("Change XY to House 1"))
    {
        b.changexy("house");
        b.revalidate();
        b.repaint();
        c.repaint();
    }
    else if(action.equals("Change XY to House 2"))
    {
        b.changexy("house1");
        b.revalidate();
        b.repaint();
        c.repaint();
    }
    else if(action.equals("Change XY to Road 1"))
    {
        b.changexy("road");
        b.revalidate();
        b.repaint();
        c.repaint();
    }
    else if(action.equals("Change XY to Road 2"))
    {
        b.changexy("road1");
        b.revalidate();
        b.repaint();
        c.repaint();
    }
    else if(action.equals("Change XY to Road 3"))
    {
        b.changexy("road2");
        b.revalidate();
        b.repaint();
        c.repaint();
    }
    else if(action.equals("Change XY to Grass"))
    {
        b.changexy("grass");
        b.revalidate();
        b.repaint();
        c.repaint();
    }
    else if(action.equals("Select more = false"))
    {
        b.selMore();
        b.repaint();
        selectMore = "Select more = true";
        btns[9].setText(selectMore);
    }
    else if(action.equals("Select more = true"))
    {
        b.selMore();
        b.repaint();
        selectMore = "Select more = false";
        btns[9].setText(selectMore);
    }
    else if(action.equals("Get info of selected"))
    {
        jt.setText(" Name: "+b.getSelected()+"\n Costs: "+b.getSelectedCost());
    }
}    

public static void main(String[] args) {
    new Game(null);
}

}

Answer 1

按钮的“ Action 命令”不一定是按钮的文本。尝试用

替换文本测试

if (e.getSource() == button[1])

或类似的有一个可靠的测试。如果这有效，那更有可能是运气——规范中没有任何内容表明按钮文本将是操作命令，事实上，如果您使用 javax.swing.Action，它通常不会。

此代码还有许多其他问题，使得阅读和预测其功能变得更加困难:

• 如果测试字符串常量，请始终使用 "THE CONSTANT".equals(something) 的形式，因为您可以保证该常量不为空；另一种方法是空指针异常。
  • 在这种情况下，如果您保留测试代码，则可以使用 switch-over-strings
• 您对 pnl 进行空检查，然后尝试分配它。事实上它不会为空。将初始化代码移至构造函数，去掉 createGui，并使 Pnl 字段成为最终字段。这样代码就更简单了，你可以让编译器证明 pnl 不能为 null，从而从可能的领域中消除整个错误类别。
• 重构GUI时，如果在运行时删除和添加组件，则需要invalidate();重新验证()； repaint(); 确保显示更新的咒语 - 这里的代码是否有效将因 JDK 和外观而异。实际上，您根本不需要重建 GUI。