添加paint方法后我的JPanel不显示

Question

我最近为我的第一个棋盘游戏创建了我的第一个棋盘，并在我的 JPanel 上以多种方式绘制棋盘。我最终决定使用 Label，但由于我需要将图片添加到该标签，我不得不将其更改为 JLabel。语法正确，但 Jlabels 不显示。 我尝试清理我的绘画方法，它开始显示，但标签的背景没有改变。

我尝试将代码放入我的绘制方法中，但没有成功。 我还尝试在我的 draw 方法之后将容器添加到框架中，但它也不起作用。

以下是我的代码的一些重要部分：

    ////////////////////////////////////
    //The constructor, Creates the frame.
    ////////////////////////////////////
    public SecondFrame() {
        setTitle("Counter Strike");
        setIconImage(Toolkit.getDefaultToolkit().getImage(SecondFrame.class.getResource("/cs/resources/icon2.png")));

        setResizable(false);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setBounds(150, 10, WIDTH, HEIGHT);
        contentPane = new JPanel();
        //contentPane.setBackground(Color.CYAN);
        contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
       
        setContentPane(contentPane);
        GroupLayout gl_contentPane = new GroupLayout(contentPane);
        gl_contentPane.setHorizontalGroup(
            gl_contentPane.createParallelGroup(Alignment.LEADING)
                .addGap(0, 984, Short.MAX_VALUE)
        );
        gl_contentPane.setVerticalGroup(
            gl_contentPane.createParallelGroup(Alignment.LEADING)
                .addGap(0, 662, Short.MAX_VALUE)
        );
        contentPane.setLayout(gl_contentPane);
        
    
        
        

        //Starting the game.
        start();
        
        
        
          //Initializing the label "boardParts" 2D array.
            frame();


    }
    
    
    
    
    
    //////////////////
    //Draws the board
    //////////////////
    
    public void frame() {
        
        boardParts=new JLabel[rows][columns];
        
        for(int i=0;i<rows;i++)
            for(int j=0;j<columns;j++) {
                
                boardParts[i][j]=new JLabel();
                boardParts[i][j].setBounds(100*i+100, 100*j+100, tilesize, tilesize);
                boardParts[i][j].setBorder(new LineBorder(new Color(0, 0, 0)));

                if(randomBarrier()) 
                     boardParts[i][j].setIcon(new ImageIcon(FirstFrame.class.getResource("/cs/resources/boundIcon.png")));
                else
                    boardParts[i][j].setBackground(Color.yellow);
                 contentPane.add(boardParts[i][j]);
                 
            }
        
    }
    
  

我还在我的另一个类中创建了这个类的一个新对象，当我运行它时，它显示错误大约 1 秒，然后将它们擦除，所以我不知道这些错误是什么原因。

这是我的简化代码：

import java.awt.Color;
import java.awt.EventQueue;
import java.awt.Toolkit;

import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.GroupLayout;
import javax.swing.Icon;
import javax.swing.ImageIcon;
import javax.swing.GroupLayout.Alignment;
import javax.swing.border.EmptyBorder;
import javax.swing.border.LineBorder;

import javax.swing.JButton;
import java.awt.event.ActionListener;
import java.util.Timer;
import java.util.TimerTask;
import java.awt.event.ActionEvent;

public class Frame1 {

    private JFrame frame;

    /**
     * Launch the application.
     */
    public static void main(String[] args) {
        EventQueue.invokeLater(new Runnable() {
            public void run() {
                try {
                    Frame1 window = new Frame1();
                    window.frame.setVisible(true);
                } catch (Exception e) {
                    e.printStackTrace();
                }
            }
        });
    }

    /**
     * Create the application.
     */
    public Frame1() {
        initialize();
    }

    /**
     * Initialize the contents of the frame.
     */
    private void initialize() {
        frame = new JFrame();
        frame.setBounds(100, 100, 450, 300);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        
        JButton btnNewButton = new JButton("New button");
        btnNewButton.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent arg0) {
                frame.setVisible(false);
                SecondFrame frame2 = new SecondFrame();
                frame2.setVisible(true);
                
            }
        });
        GroupLayout groupLayout = new GroupLayout(frame.getContentPane());
        groupLayout.setHorizontalGroup(
            groupLayout.createParallelGroup(Alignment.LEADING)
                .addGroup(groupLayout.createSequentialGroup()
                    .addGap(42)
                    .addComponent(btnNewButton)
                    .addContainerGap(303, Short.MAX_VALUE))
        );
        groupLayout.setVerticalGroup(
            groupLayout.createParallelGroup(Alignment.LEADING)
                .addGroup(Alignment.TRAILING, groupLayout.createSequentialGroup()
                    .addContainerGap(183, Short.MAX_VALUE)
                    .addComponent(btnNewButton)
                    .addGap(56))
        );
        frame.getContentPane().setLayout(groupLayout);
    }
}


package hello;

import java.awt.Color;
import java.awt.Toolkit;

import javax.swing.GroupLayout;
import javax.swing.ImageIcon;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.GroupLayout.Alignment;
import javax.swing.border.EmptyBorder;
import javax.swing.border.LineBorder;

public class SecondFrame extends JFrame implements Runnable{

    /**
     * 
     */
    private static final long serialVersionUID = 1L;
    private JPanel contentPane;
    private static int WIDTH=1000,HEIGHT=700;
    private static int boardWidth=500,boardHeight=500;  //The width and height of the game board.
    private Thread thread;
    private boolean isRunning;
    private BoardParts barriers;
    private int rows=8,columns=5,tilesize=100;
    private JLabel[][] boardParts;
    private boolean[][] notBarrier;

    
    
    

    ////////////////////
    //Creates the frame.
    ////////////////////
    public SecondFrame() {
        setTitle("Counter Strike");
        setIconImage(Toolkit.getDefaultToolkit().getImage(SecondFrame.class.getResource("/cs/resources/icon2.png")));

        setResizable(false);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setBounds(150, 10, WIDTH, HEIGHT);
        contentPane = new JPanel();
        contentPane.setBackground(Color.CYAN);
        contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
       
        setContentPane(contentPane);
        GroupLayout gl_contentPane = new GroupLayout(contentPane);
        gl_contentPane.setHorizontalGroup(
            gl_contentPane.createParallelGroup(Alignment.LEADING)
                .addGap(0, 984, Short.MAX_VALUE)
        );
        gl_contentPane.setVerticalGroup(
            gl_contentPane.createParallelGroup(Alignment.LEADING)
                .addGap(0, 662, Short.MAX_VALUE)
        );
        contentPane.setLayout(gl_contentPane);
        
    
        
        

        //Starting the game.
        start();
        
        
        
        //Initializing the label "boardParts" 2D array.
        frame();
        


    }
    

    
    
    //////////////////
    //Draws the board
    //////////////////
    
    public void frame() {
        
        boardParts=new JLabel[rows][columns];
        notBarrier=new boolean[rows][columns];
        
        for(int i=0;i<rows;i++)
            for(int j=0;j<columns;j++) {
                
                boardParts[i][j]=new JLabel();
                boardParts[i][j].setBounds(100*i+100, 100*j+100, tilesize, tilesize);
                boardParts[i][j].setBorder(new LineBorder(new Color(0, 0, 0)));

                if(randomBarrier()) {
                     boardParts[i][j].setIcon(new ImageIcon(Frame1.class.getResource("/cs/resources/boundIcon.png")));
                     notBarrier[i][j]=false;
                }
                else {
                    boardParts[i][j].setBackground(Color.yellow);
                    notBarrier[i][j]=true;
                }
                 contentPane.add(boardParts[i][j]);
                 
            }
    

    }
    
    


    
    ///////////////////////////////////////////////////////////////////////////
    //finds a random place for the barrier objects in the beginning of the game.
    ///////////////////////////////////////////////////////////////////////////

    public static boolean randomBarrier() {  //Should put all the parts to this method to see if the are barrier material or not.
        
        
        int row = WIDTH/100;
        int column = HEIGHT/100;
        
        int min = 0;
        int max = row*column;
        
        double random = Math.random();
        
        if(random<0.4)
            return true;
        
        else if(random>=0.4)
           return false;
        
        return true;
        
    }


    
    
    
    
    //////////////////
    //Starts the game.
    /////////////////
    public void start() {
        
        isRunning = true;
        thread = new Thread(this);
        thread.start();
        
    }
    
    
    
    
    ////////////////
    //Stops the game.
    ///////////////
    public void stop() {
        
        isRunning = false;
        try {
            thread.join();
        } catch (InterruptedException e) {
            System.out.println("An error occured...");
            e.printStackTrace();
        }
        
    }
    
    
    
    public void tick() {
        
        
    }
    

    
    
    @Override
    public void run() {


        while(isRunning) {
            tick();
            repaint();
        }
        
    }
    
    
}


package hello;

public class BoardParts {

}

Answer 1

1. 发布代码时不要使用“代码片段”。相反，您粘贴代码，选择代码，然后使用{} 按钮突出显示代码。

2. 发布代码时发布正确的minimal reproducible example。这是直接演示所述问题的最小代码。代码应该在一个文件中，我们应该能够复制/粘贴/编译和测试代码。这样做的目的是迫使您消除所有不必要的代码，以便于理解问题。大多数时候你会发现自己的问题。您已被要求提供 MRE 是以前的问题。每个问题都应该有一个 MRE，这样我们就不必猜测你在做什么。

3. 在我看来你有一个 2D 网格。不要使用 GroupLayout。这告诉我您正在使用 IDE 生成代码。您花时间学习 IDE 而不是学习 Swing。您可以轻松地将GridLayout 用于二维网格。

4. 不要使用静态变量。不需要您的 width、height、boardWidth 和 boardHeight 变量。每个 Swing 组件都应该负责确定自己的首选大小。然后在将所有组件添加到框架之后，在使其可见之前打包（）框架。然后框架将确定其适当的大小。在这种情况下，您可以对每个 JLabel 使用 setPreferredSize(...) 以使它们成为您的 tile 的大小。因此，在循环之外，您创建一个 Dimension 对象的单个实例，供所有标签共享。

5. 不要在 setBounds() 方法中使用幻数。在您的最后一个问题中，您在不使用幻数的情况下获得了解决方案。事实上，您甚至不应该使用 setBounds() 方法。布局管理器的工作是设置组件的大小/位置。

6. 不要将线程用于动画。动画应该使用Swing Timer 完成。 Swing 组件的所有更新都应在事件调度线程 (EDT) 上完成。 Swing Timer 将在 EDT 上执行。

7. 不要创建Icon 的多个实例。一个图标可以被多个组件共享。因此，您在循环代码之外创建 Icon 的单个实例，并将该实例用于所有组件。

8. 与LineBorder 相同。您只需要一个实例。

9. 在您上一个问题的 cmets 中，您获得了有关如何使用单个语句编写 randomBarrier() 方法的建议。您在此处发布的代码完全没有必要。甚至没有使用前 4 条语句中的变量。

但 Jlabels 不显示。

正如我之前建议的那样，您可以在面板上使用GridLayout 并将标签添加到面板并将面板添加到框架。阅读Layout Manager 上的 Swing 教程，了解更多信息和工作示例。

我尝试清理我的油漆方法

没有理由使用自定义绘制方法。您现在正在使用 Swing 组件 (JLabel)，Swing 将为您完成所有绘制工作。此外，正如您在上一个问题中提到的，您是否确实需要进行自定义绘画，您会覆盖 paintComponent() 方法。我们不应该一直重复同样的建议。

标签的背景没有改变。

JLabel 是唯一默认不透明的 Swing 组件。所以你需要使用：

label.setOpaque( true );

当您创建每个 JLabel 时。

Answer 2

我认为，您必须为您的电路板覆盖paintComponent(Graphics g)。 您必须将要绘制的所有内容保存在 Collection 中，然后在paintComponent override 中绘制它们。