Question

此代码是琐事游戏的简单引擎。这个想法是答案出现在JButtons上。为此，我必须设置一个刷新方法，删除所有内容并重新绘制它。似乎每次调用此方法时，它都会变得越来越慢。大约10次按钮点击后它会变得很慢，它会停止响应，我将不得不手动关闭程序。

由于

package mainPackage;

import java.awt.FlowLayout;
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JComponent;
import javax.swing.JFrame;
import javax.swing.JLabel;

public class MainGame{
    static JFrame frame;
    static WindowComp w;

    public static void main(String[] args) {
        frame = new JFrame("Game");
        w = new WindowComp();
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setSize(300, 300);
        frame.setVisible(true);
        frame.setResizable(true);

        WindowComp.setAnswers( "start", "start", "start", "start");
        WindowComp.refreshAll(w, frame);

        WindowComp.setAnswers("final", "final", "final", "final");
        WindowComp.refreshAll(w, frame);
    }
}

public class WindowComp extends JComponent implements ActionListener {
    static JButton [] buttons = new JButton[4];
    static JLabel question = new JLabel("default");

    public WindowComp(){
        setAnswers("default", "default", "default", "default");
    }

    public void paintComponent(Graphics g){
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        if(e.getSource() == buttons[0]){
            setQuestion("button 1");
        }
        if(e.getSource() == buttons[1]){
            setQuestion("button 2");
        }
        if(e.getSource() == buttons[2]){
            setQuestion("button 3");
        }
        if(e.getSource() == buttons[3]){
            setQuestion("button 4");
        }
        refreshAll(MainGame.w, MainGame.frame);
    }

    public void addAll(){
        setLayout(new FlowLayout());
        buttons[0].addActionListener(this);
        buttons[1].addActionListener(this);
        buttons[2].addActionListener(this);
        buttons[3].addActionListener(this);
        add(buttons[0]);
        add(buttons[1]);
        add(buttons[2]);
        add(buttons[3]);
        add(question);
    }

    public static void setAnswers( String ans1, String ans2, String ans3,String ans4){
        buttons[0] = new JButton("Answer 1 : " + ans1);
        buttons[1] = new JButton("Answer 2 : " + ans2);
        buttons[2] = new JButton("Answer 3 : " + ans3);
        buttons[3] = new JButton("Answer 4 : " + ans4);
    }

    public static void setQuestion(String q){
        question = new JLabel("Question: " + q);
    }

    public static void refreshAll(WindowComp w, JFrame frame){
        w.removeAll();
        w.addAll();
        w.revalidate();
        frame.add(w);
    }
}

Answer 1

好的，正如评论中所说，你添加了太多的ActionListeners，这会导致你所描述的问题。

以下是我想给你的一些建议。

首先，每次要更改按钮的文本时，都不必使用关键字new。垃圾收集将摆脱未使用的按钮，但为什么你想要新的按钮，而不是只通过setTest(String)更新按钮上的文本，这不会调用任何垃圾收集。

最后，尝试更多地使用构造函数，实际上可以在调用构造函数时创建所需的一切（在这种情况下，通常不是）。例如，您可以在构造函数中创建所有JButton并将所有侦听器添加到按钮（我将在下面提供一些代码）。

我重写了一下你的代码，它没有你的功能，但它也不会崩溃。

package de;

import javax.swing.JFrame;

public class MainGame{
static JFrame frame;
static WindowComp w;
public static void main(String[] args) {
    frame = new JFrame("Game");
    w = new WindowComp();
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setSize(300, 300);
    frame.add(w);
    frame.setVisible(true);
    frame.setResizable(true);
   /*WindowComp.setAnswers( "start", "start", "start", "start");
    WindowComp.refreshAll(w, frame);

    WindowComp.setAnswers("final", "final", "final", "final");
    WindowComp.refreshAll(w, frame);*/
}

}




package de;

import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JComponent;
import javax.swing.JFrame;
import javax.swing.JLabel;

@SuppressWarnings("serial")
public class WindowComp extends JComponent implements ActionListener {

static JButton [] buttons;
static JLabel question;

public WindowComp(){
    question = new JLabel("default");
    buttons = new JButton[4];
    setLayout(new FlowLayout());
    buttons[0] = new JButton("Answer 1 : " + "default");
    buttons[1] = new JButton("Answer 2 : " + "default");
    buttons[2] = new JButton("Answer 3 : " + "default");
    buttons[3] = new JButton("Answer 4 : " + "default");

    buttons[0].addActionListener(this);
    buttons[1].addActionListener(this);
    buttons[2].addActionListener(this);
    buttons[3].addActionListener(this);
    addAll();

}


@Override
public void actionPerformed(ActionEvent e) {
    if(e.getSource() == buttons[0]){
        setQuestion("button 1");
        setAnswers( "start", "start", "start", "start");
    }
    if(e.getSource() == buttons[1]){
        setQuestion("button 2");
        setAnswers("final", "final", "final", "final"); 
    }
    if(e.getSource() == buttons[2]){
        setQuestion("button 3");
    }
    if(e.getSource() == buttons[3]){
        setQuestion("button 4");
    }
    //refreshAll(MainGame.w, MainGame.frame);

}
public void addAll(){

    add(buttons[0]);
    add(buttons[1]);
    add(buttons[2]);
    add(buttons[3]);
    add(question);

}



public static void setAnswers( String ans1, String ans2, String ans3,String ans4){
    buttons[0].setText("Answer 1 : " + ans1);
    buttons[1].setText("Answer 2 : " + ans2);
    buttons[2].setText("Answer 3 : " + ans3);
    buttons[3].setText("Answer 4 : " + ans4);

}

public static void setQuestion(String q){
    question.setText("Question: " + q);
}

public static void refreshAll(WindowComp w, JFrame frame){
    w.removeAll();
    w.addAll();
    w.revalidate();
    frame.add(w);

}

}

编辑：就当前代码而言，您的函数refreshAll(WindowComp w, JFrame frame)不再被调用，因为当前不需要调用它。在没有调用的情况下测试程序后，我用这个编辑对它进行了评论。

Answer 2

您的问题是ActionListener总是添加到按钮但从未删除。快速解决方法是编写一个方法来删除它们：

public void removeActionListeners(){
        buttons[0].removeActionListener(this);
        buttons[1].removeActionListener(this);
        buttons[2].removeActionListener(this);
        buttons[3].removeActionListener(this);
}

然后在你的＆＃34; refreshAll（）＆＃34;：

中调用它

public static void refreshAll(WindowComp w, JFrame frame){
        w.removeActionListeners();
        w.removeAll();
        w.addAll();
        w.revalidate();
        frame.add(w);
}

...正如所指出的那样，不这是做到这一点的最佳方式，但它不再有任何延迟。

单击按钮多次后，小Swing应用程序崩溃

2 个答案: