康威生命游戏中setBackground方法无法正常工作求助

核心问题 #

你的代码里，按下'a'键触发的loop()方法是在**事件调度线程（EDT）**中执行的，而Swing的UI操作必须在EDT中完成，但loop()是一个无限循环，会直接阻塞EDT——这导致UI无法刷新，setBackground()的修改无法生效。移除KeyListener后，你可能是在其他非EDT线程调用循环，但这又会导致跨线程操作UI，其实也不符合规范，只是偶然能工作。

解决步骤 #

1. 使用Swing Timer替代手动while循环：Timer的任务会自动在EDT中执行，既不会阻塞线程，又能保证UI操作的合法性。
2. 修复isTouching()方法的数组越界问题：当前代码在处理边缘细胞时（比如X=arrX-1），X+1会超出数组长度，导致数组越界异常。
3. 调整游戏启停逻辑，通过控制Timer的启动/停止来替代直接调用阻塞式循环。

修改后的代码 #

Main类修改

import javax.swing.JFrame;
import javax.swing.JPanel;
import java.awt.*;
import java.awt.Dimension;
import java.awt.Graphics;
import java.util.Scanner;
import javax.swing.JLabel;
import javax.swing.Timer;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

public class Main
{ 
  public static final int arrY = 28;
  public static final int arrX = 28;
  public static int[][] arr = new int[arrX][arrY];
  public static int[][] endArr = new int[arrX][arrY];
  public static final int CELL = 1;
  public static final int DEADCELL = 0; 
  public static boolean stop = true;
  public static int generations = 0;
  static MyFrame frame = new MyFrame();
  // 游戏Timer，每300ms执行一次一代更新（可调整速度）
  private static Timer gameTimer = new Timer(300, new ActionListener() {
      @Override
      public void actionPerformed(ActionEvent e) {
          updateGeneration();
      }
  });

  public static int isTouching(int X, int Y) {
    int integer = 0;
    // 遍历周围8个方向，先判断边界，避免数组越界
    for (int dx = -1; dx <= 1; dx++) {
        for (int dy = -1; dy <= 1; dy++) {
            // 跳过当前细胞自身
            if (dx == 0 && dy == 0) continue;
            int nx = X + dx;
            int ny = Y + dy;
            // 判断坐标是否在数组范围内
            if (nx >= 0 && nx < arrX && ny >=0 && ny < arrY) {
                if (arr[nx][ny] == CELL) {
                    integer++;
                }
            }
        }
    }
    return integer;
  }
    
  public static void main(String[] args) throws InterruptedException {
    // 初始化数组
    for (int i = 0; i < arrX; i++) {
      for (int j = 0; j < arrY; j++) {
        arr[i][j] = DEADCELL;
        endArr[i][j] = DEADCELL;
      }
    }  
    frame.makeBoard(arr); 
  }

  // 单独提取单代更新逻辑
  public static void updateGeneration() {
      generations++;
      // 计算下一代状态
      for (int i = 0; i < arrX; i++) {
        for (int j = 0; j < arrY; j++) {
          int touch = isTouching(i, j);
          if (arr[i][j] == CELL) {
              if (touch < 2 || touch > 3) {
                  frame.labelWhite(i, j);
                  endArr[i][j] = DEADCELL;
              } else {
                  frame.labelBlack(i,j);
                  endArr[i][j] = CELL;
              }
          } else {
              if (touch == 3) {
                  frame.labelBlack(i,j);
                  endArr[i][j] = CELL;
              } else {
                  endArr[i][j] = DEADCELL;
              }
          }
        }
      }  
      // 更新当前数组为下一代状态
      for (int i = 0; i < arrX; i++) {
        for (int j = 0; j < arrY; j++) {
          arr[i][j] = endArr[i][j];
        }
      }
  }

  public static void stop() {
    if (stop) {
        // 启动游戏
        stop = false;
        gameTimer.start();
    } else {
        // 停止游戏
        stop = true;
        gameTimer.stop();
    }
  } 
}

MyFrame类修改（仅需调整KeyListener相关逻辑，其他不变）

import java.awt.Color;
import java.awt.event.*;
import javax.swing.*;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;

public class MyFrame extends JFrame implements KeyListener{
  public static JLabel[][] labelArr = new JLabel[28][28];

  MyFrame(){
    this.setTitle("Oliver's Game of Life");
    this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    this.setSize(1000,1000);
    this.setLayout(null);
    this.addKeyListener(this);
    // 确保窗口获得焦点，KeyListener才能生效
    this.setFocusable(true);
    this.requestFocusInWindow();
    this.setVisible(true);
  }

  // makeBoard方法保持不变...

  @Override
  public void keyTyped(KeyEvent e) {
    switch(e.getKeyChar()) {
      case 'a': Main.stop();
        break;
    }
  }

  // keyPressed和keyReleased保持不变...

  // labelWhite和labelBlack方法保持不变...
}

额外说明 #

• Swing Timer的延迟时间（300ms）可以根据需要调整，数值越小游戏速度越快。
• 给窗口添加setFocusable(true)和requestFocusInWindow()是为了确保KeyListener能正常接收按键事件，避免窗口未聚焦时按键无效的问题。
• 重构后的isTouching()方法用双层循环遍历周围8个方向，同时做边界判断，更简洁且避免数组越界。

内容的提问来源于stack exchange，提问作者Oliver Bentley