java - Action 监听器无法正常工作

Question

我认为 Action 监听器遇到了问题。奇怪的是它在某些计算机上有效，但在其他计算机上无效。所有计算机都运行 JE7.1。因此，在某些系统上，它可以完美地工作，用户输入文本，按 Enter 键，文本将附加到 JTextPane 并更新静态 userInput 变量。但在其他计算机上，会附加文本，但变量似乎永远不会更新。

例如:用户输入“Hello”"Hello 被附加到 JtextPane 和静态变量 userInput = "Hello"

我在这里错过了一些非常愚蠢的事情吗？预先感谢您。

package com.core;
import java.awt.*;
import java.awt.event.*;
import java.util.ArrayList;

import javax.swing.*;
import javax.swing.text.*;


public class GUI {
    public static JFrame f;
    private static JPanel topPanel = new JPanel();
    private static JTextPane textArea = new JTextPane();
    public static JTextField inputText = new JTextField();
    private static String userInput = "";
    private static Player player;

    public static JPanel sidePanel = new JPanel();

    public JFrame buildFrame(){
        f = new JFrame("AMSA World");
        f.setSize(800, 600);
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        f.setLayout(new GridBagLayout());

        textArea.setFocusable(false);
        JScrollPane scrollPane = new JScrollPane(textArea);
        scrollPane.setHorizontalScrollBarPolicy(ScrollPaneConstants.HORIZONTAL_SCROLLBAR_NEVER);

        ActionListener listener = new ActionListener() {  
            public void actionPerformed(ActionEvent evt) {
                String text = inputText.getText();
                if(!text.equals("")){
                    appendToPane(text + "\n\n", Color.blue);
                    inputText.setText("");
                    if(text.startsWith("use ")){
                        ArrayList<Item> inventory = player.getInventory();
                        String key = text.substring(4);
                        boolean found = false;
                        for(Item i : inventory){
                            if(i.getName().equalsIgnoreCase(key)){
                                found = true;
                                i.use();
                                break;
                            }
                        }
                        if(!found){
                            appendToPane("Item does not exist in your iventory.\n\n", Color.black);
                        }
                    }
                    if(text.startsWith("inventory")){
                        ArrayList<Item> inventory = player.getInventory();
                        if(inventory.size() > 0){
                            for(Item i : inventory){
                                appendToPane(i.getName() +"\n", Color.darkGray);
                            }
                        }
                        else
                            appendToPane("Your iventory is empty.\n\n", Color.black);
                        appendToPane("#", Color.blue);
                    }
                    else{
                        userInput = text;
                    }
                }
            } 
        };
        inputText.addActionListener(listener);

        GridBagConstraints c = new GridBagConstraints();
        c.fill = GridBagConstraints.HORIZONTAL;
        c.insets = new Insets(5,5,5,5);
        c.weightx = 1.0;
        c.gridwidth = 3;
        c.weighty = 0.025;
        c.gridx = 0;
        c.gridy = 0;
        f.add(topPanel, c);

        c.fill = GridBagConstraints.BOTH;
        c.weighty = 1.0;
        c.gridwidth = 2;
        c.gridx = 0;
        c.gridy = 1;
        f.add(scrollPane, c);

        c.fill = GridBagConstraints.BOTH;
        c.weighty = 1.0;
        c.gridwidth = 1;
        c.gridx = 2;
        c.gridy = 1;
        f.add(sidePanel, c);
        sidePanel.setVisible(false);

        c.fill = GridBagConstraints.HORIZONTAL;
        c.weighty = 0.025;
        c.gridwidth = 3;
        c.gridx = 0;
        c.gridy = 2;
        f.add(inputText, c);

        f.setVisible(true);
        return f;
    }
    public static JTextField getTextField(){
        return inputText;
    }
    public static void toggleTextField(boolean value){
        inputText.setEnabled(value);
    }
    public static String getUserInput(){
        return userInput;
    }
    public static void setUserInput(String s){
        userInput = s;
    }
    public static JPanel getTopPanel(){
        return topPanel;
    }
    public static JPanel getSidePanel(){
        return sidePanel;
    }
    public static void setPlayer(Player p){
        player = p;
    }

    public static void appendToPane(String msg, Color c){
        if(inputText.isEnabled()){
        StyleContext sc = StyleContext.getDefaultStyleContext();
        AttributeSet aset = sc.addAttribute(SimpleAttributeSet.EMPTY, StyleConstants.Foreground, c);

        textArea.setCharacterAttributes(aset, false);
        textArea.replaceSelection(msg);
        textArea.setCaretPosition(textArea.getDocument().getLength());
        }
    }
}

Answer 1

为了(尝试)回答您的问题，我猜您遇到了多线程问题。 Swing 事件在它们自己的线程中运行，Event Dispatch Thread 。如果您从另一个线程访问该变量，偶尔会遇到变量在线程之间不同步的情况。

从更大的角度来看，在编写 Java 代码时走捷径通常最终会花费您大量的时间和精力。将整个 UI 放置在具有静态方法的静态变量中并不是一个好主意。花点时间实例化一个实例。除此之外，我猜您没有使用 SwingUtilities.invokeLater() 启动 UI。阅读我上面链接的 Swing 教程。 Swing 效果很好，但并不简单。