Question

我有一个java swing项目，如下所示：

import javax.swing.JFrame;
import javax.swing.JButton;
import javax.swing.JPanel;

import javax.swing.ImageIcon;
import javax.swing.Box;
import javax.swing.BoxLayout;
import java.awt.BorderLayout;
import java.awt.Color;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Random;


public class DiceGameReplaceDice extends JFrame
{  
    private JFrame gameFrame;
    private JPanel mainPanel;
    private JPanel centerPanel = new JPanel();
    private JButton diceArray[];
    private DiceListener diceListener = new DiceListener();
    private ButtonListener buttonListener = new ButtonListener();
    private Random rand = new Random();
    private int NUM_DICE = 2;


    private String diceImages[] = {"./src/1.png", "./src/2.png", "./src/3.png",
            "./src/4.png", "./src/5.png", "./src/6.png"};

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

    public DiceGameReplaceDice()    
    {
        // Initialize the frame that holds the game
        gameFrame = new JFrame();
        gameFrame.setSize(800, 600);
        gameFrame.setLocation(300, 100); 
        gameFrame.setTitle("Dice Game");
        gameFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

        // Add Panel
        mainPanel = new JPanel();
        mainPanel.setLayout(new BorderLayout());

        // Instantiate a ButtonListener
        buttonListener = new ButtonListener();


        // Add 1 Button and 1 Label to a newly created panel 
        // Add new panel to West 
        JButton buttonW1 = new JButton("Add Die");
        buttonW1.setName("W1");
        buttonW1.addActionListener(buttonListener);
        JPanel panelWest = new JPanel();
        panelWest.setLayout(new BoxLayout(panelWest,BoxLayout.Y_AXIS ));
        panelWest.setBackground(new Color(0, 0, 122));  // set to blue
        panelWest.add(Box.createVerticalGlue());
        panelWest.add(buttonW1);
        panelWest.add(Box.createVerticalGlue());

        mainPanel.add(panelWest, BorderLayout.WEST);


        // Create and display center panel with dice
        displayCenterPanel();

        // Add mainPanel to frame and display the frame
        gameFrame.add(mainPanel);
        gameFrame.setVisible(true);
    }

    private  void displayCenterPanel()
    {
        centerPanel = new JPanel();
        centerPanel.setLayout(new BoxLayout(centerPanel,BoxLayout.X_AXIS ));
        centerPanel.setBackground(new Color(0, 122, 0));  // set to green
        centerPanel.add(Box.createHorizontalGlue());

        diceArray = new JButton[NUM_DICE];

        // Add 2 Buttons to center panel with images of 2 random dice 
        for (int i=0; i<NUM_DICE; i++)
        {
            // Create dice button
            int dieNum = rand.nextInt(6)+1;
            diceArray[i] = new JButton(new ImageIcon(diceImages[dieNum-1]));
            diceArray[i].setName("Dice" + i); 
            diceArray[i].addActionListener(diceListener);

            // Add to center panel
            centerPanel.add(diceArray[i]);
            centerPanel.add(Box.createHorizontalGlue());
        }

        mainPanel.add(centerPanel, BorderLayout.CENTER);

        // Add mainPanel to frame and display the frame
        gameFrame.add(mainPanel);
        gameFrame.setVisible(true);
    }

    // Implement an (inner) class that implements ActionListener
    class DiceListener implements ActionListener
    {

        public void actionPerformed(ActionEvent e) 
        {
            NUM_DICE -= 1;
            displayCenterPanel();
        }
    }

    // Implement an (inner) class that implements ActionListener
    class ButtonListener implements ActionListener
    {

        public void actionPerformed(ActionEvent e) 
        {
            String button = ((JButton)e.getSource()).getName();
            System.out.println("Button Pressed: " + button);
            if (button.equals("W1"))
                NUM_DICE ++;
            System.out.println(NUM_DICE);
            displayCenterPanel();
        }
    }
}

单击“添加模具”按钮时，模具将添加到屏幕并正确格式化。但是，当按下骰子并且NUM_DICE减小时，单击骰子按钮会导致奇怪的重叠和“鬼按钮”。你是如何解决这个问题的？

Answer 1

快速解决方法是在添加新版本之前删除centerPanel。添加行

mainPanel.remove(centerPanel);

是你在displayCenterPanel内做的第一件事。

但是，动态操作布局的方式还有很多不足之处。而不是每次都创建一个新面板，只需修改现有面板：

public class DiceGame {

    private JPanel centerPanel = new JPanel();
    private Random rand = new Random();
    private int numDice = 2;

    public static void main(String[] args) {

        SwingUtilities.invokeLater(() -> new DiceGame());
    }

    public DiceGame() {

        JFrame gameFrame = new JFrame();
        gameFrame.setSize(800, 600);
        gameFrame.setTitle("Dice Game");
        gameFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

        JPanel mainPanel = new JPanel();
        mainPanel.setLayout(new BorderLayout());

        JButton buttonW1 = new JButton("Add Die");
        buttonW1.addActionListener(e -> {
            numDice++;
            updateCenterPanel();
        });
        JPanel panelWest = new JPanel();
        panelWest.setLayout(new BoxLayout(panelWest, BoxLayout.Y_AXIS));
        panelWest.setBackground(new Color(0, 0, 122));
        panelWest.add(Box.createVerticalGlue());
        panelWest.add(buttonW1);
        panelWest.add(Box.createVerticalGlue());

        centerPanel.setLayout(new BoxLayout(centerPanel, BoxLayout.X_AXIS));
        centerPanel.setBackground(new Color(0, 122, 0));

        mainPanel.add(panelWest, BorderLayout.WEST);
        mainPanel.add(centerPanel);

        gameFrame.add(mainPanel);
        gameFrame.setVisible(true);
    }

    private void updateCenterPanel() {

        centerPanel.removeAll();
        centerPanel.add(Box.createHorizontalGlue());

        JButton[] diceArray = new JButton[numDice];

        for (int i = 0; i < numDice; i++) {
            diceArray[i] = new JButton(Integer.toString(rand.nextInt(6) + 1));
            diceArray[i].setName("Dice" + i);
            diceArray[i].addActionListener(e -> {
                numDice--;
                updateCenterPanel();
            });
            centerPanel.add(diceArray[i]);
            centerPanel.add(Box.createHorizontalGlue());
        }
        centerPanel.revalidate();
        centerPanel.repaint();
    }
}

在使组件层次结构无效后，必须重新验证和重新绘制。

备注：

当局部变量发生时，不要创建字段。

NUM_DICE不是final，因此应将其命名为numDice。

当gameFrame.setVisible(true);已经可见时调用它不会做任何事情。

如果您有正在使用的工作版本，请将setSize(...)上的JFrame替换为pack，并确保正确计算其子项的大小。

删除JLabel中的按钮

1 个答案: