Java Swing : Issue painting a grid

Question

该程序应该在绘制的网格上运行元胞自动机模拟(想想康威的生命游戏)，并有一个开始/暂停按钮来开始/暂停模拟，该模拟以 1 秒的间隔运行。据我所知，除了绘制网格(处理、GUI 的其余部分)之外，其他所有内容都工作正常。

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ConcurrentModificationException;

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


public class CA_DriverV2 extends JFrame{

    private static final Color white = Color.WHITE, black = Color.BLACK;

    private Board board;
    private JButton start_pause;

    public CA_DriverV2(){

        board = new Board();
        board.setBackground(white);

        start_pause = new JButton("Start");
        start_pause.addActionListener(board);

        this.add(board, BorderLayout.NORTH);
        this.add(start_pause, BorderLayout.SOUTH);
        this.setLocationRelativeTo(null);
        this.setDefaultCloseOperation(EXIT_ON_CLOSE);
        this.setSize(300, 300);
        this.setVisible(true);

    }

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

    private class Board extends JPanel implements ActionListener{

        private final Dimension DEFAULT_SIZE = new Dimension(5, 5);
        private final int DEFAULT_CELL = 10, DEFAULT_INTERVAL = 1000, DEFAULT_RATIO = 60;

        private Dimension board_size;
        private int cell_size, interval, fill_ratio;
        private boolean run;
        private Timer timer;

        private Color[][] grid;

        public Board(){
            board_size = DEFAULT_SIZE;
            cell_size = DEFAULT_CELL;
            interval = DEFAULT_INTERVAL;
            fill_ratio = DEFAULT_RATIO;
            run = false;

            //Initialize grid with random values
                //NOTE: Add JOptionPane for option to define fill rate and board size?
                //ALT: Have a resize(int h, int w) method to resize grid when needed.
                //ALT: Have refill(int r) method to restart with different fill ratio.
            grid = new Color[board_size.height][board_size.width];
            for (int h = 0; h < board_size.height; h++)
                for (int w = 0; w < board_size.width; w++){
                    int r = (int)(Math.random() * 100);
                    if (r >= fill_ratio)
                        grid[h][w] = black;
                    else grid[h][w] = white;
                }

            timer = new Timer(interval, this);
        }

        @Override
        public Dimension getPreferredSize(){
            return new Dimension(board_size.height, board_size.width);
        }

        @Override
        public void paintComponent(Graphics g){
            for (int h = 0; h < board_size.height; h++)
                for (int w = 0; w < board_size.width; w++){
                    try{
                        if (grid[h][w] == black)
                            g.setColor(black);
                        else g.setColor(white);
                        g.fillRect(h * cell_size, w * cell_size, cell_size, cell_size);
                    } catch (ConcurrentModificationException cme){}
                }
        }

        public void actionPerformed(ActionEvent e) {

            //Timer tick processing
            if (e.getSource().equals(timer)){
                repaint();
                Color[][] newGrid = new Color[board_size.height][board_size.width];
                for (int h = 1; h < board_size.height; h++)
                    for (int w = 1; w < board_size.height; w++) {
                        int surrounding = 0;
                        //Count black neighbors
                        for (int i = -1; i <= 1; i++)
                            for (int j = -1; j <= 1; j++){
                                if(i != 0 && j != 0){
                                    try{
                                        if(grid[h + i][w + j] == black)
                                            surrounding++;
                                    } catch(ArrayIndexOutOfBoundsException ae){}
                                }
                            }



                        //Generate next iteration
                        if (surrounding > 5 || surrounding < 2)
                            newGrid[h][w] = black;
                        else newGrid[h][w] = white;
                    }
                for (int h = 1; h < board_size.height; h++){
                    for (int w = 1; w < board_size.height; w++){
                        grid[h][w] = newGrid[h][w];
                        System.out.print(grid[h][w] + " ");
                    }
                    System.out.println();
                }
                System.out.println();
            }

            //Start-Pause button processing
            else if(e.getSource().equals(start_pause)){
                if(run){
                    timer.stop();
                    start_pause.setText("Pause");
                }
                else {
                    timer.restart();
                    start_pause.setText("Start");
                }
                run = !run;

            }
        }
    }
}

它在最顶部打印一些内容，看起来像是初始网格的一小部分被按钮的一小部分覆盖，其余部分是默认的灰色。

Answer 1

您的板 Board 变量添加为 BorderLayout.NORTH，而不是 BorderLayout.CENTER，因此它仅填充顶部 5 个像素。

根据我的评论，你的程序中永远不应该有这样的代码:

catch(ArrayIndexOutOfBoundsException ae){}

您不仅不应该忽略异常，而且甚至不应该捕获这种类型的异常。相反，请稍微小心地创建 for 循环，以便它们可以处理边缘。

另外，不要忘记在类的重写中调用 super.paintComponent(g) 方法。