为什么我的Java游戏生活程序不工作？

Question

我正在制作康威生命游戏图形模型的程序，但它一旦启动就不会让我做任何事情;按钮不起作用，网格不变。我究竟做错了什么？

import java.awt.*; 
import javax.swing.*; 
import java.awt.event.ActionEvent; 
import java.awt.event.ActionListener; 
import java.awt.event.MouseEvent; 


public class gameOfLife extends JApplet{ 
    private static final long serialVersionUID = 1L; 

cellClass cell; 

public void init() { 
    Container contentWindow = getContentPane(); 
    cell = new cellClass();{{ 
    setLayout(new FlowLayout()) }}; 
    contentWindow.add(cell);  
    } 
} 

class grid extends JComponent{ 
    private static final long serialVersionUID = 2L; 

    int XSIZE = 500; 
    int YSIZE = 500; 
    private int row; 
    private int col; 
    private int size = 5; 
    private cellClass c; 
    private Dimension preferredSize = new Dimension(XSIZE, YSIZE); 

    public void paint(Graphics a) { 
    int x, y; 
     for(x=0; x<row; x++){ 
     for(y=0; y<col; y++){ 
      if(c.grid[x][y] != 0){ 
       a.drawRect(x * size, y * size, 5, 5); 
      } 
     } 
    } 
    a.drawRect(0, 0, XSIZE, YSIZE); 

} 

public grid(cellClass newGrid, int newRow, int newCol, int newSize) { 
    setMinimumSize(preferredSize); 
    setMaximumSize(preferredSize); 
    setPreferredSize(preferredSize); 
    this.row = newRow; 
    this.col = newCol; 
    this.size = newSize; 
    this.c = newGrid; 

} 
} 

class cellClass extends JPanel implements ActionListener{ 
private static final long serialVersionUID = 3L; 

static final int ROW = 100; 
static final int COL = 100; 
static final int SIZE = 5; 
static final int min = 2; 
static final int max = 3; 
static final int birth = 3; 
public int genCount = 0; 

public int[][] grid; 
private int[][] nextGrid; 

private GridBagLayout gridBag = new GridBagLayout(); 
private GridBagConstraints c = new GridBagConstraints(); 

JLabel title; 
JLabel genCounter; 
JButton oneGen; 
JButton contPlay; 
JButton stop; 
public grid board; 
public boolean paused = true; 
public boolean canChange = true; 

cellClass() { 
    grid = new int [ROW][COL]; 
    nextGrid = new int[ROW][COL]; 

    makeGrid(grid); 

    setLayout(gridBag); 

    title = new JLabel("Game of Life Applet"); 
    c.gridx = 0; 
    c.gridy = 0; 
    c.gridwidth = 2; 
    c.insets = new Insets(2,0,0,0); 
    c.anchor = GridBagConstraints.WEST; 
    add(title); 

    board = new grid(this,ROW,COL,SIZE); 
    c.gridx = 0; 
    c.gridy = 2; 
    c.gridwidth = 1; 
    gridBag.setConstraints(board, c); 
    add(board); 

    oneGen = new JButton("Move one Generation"); 
    c.gridx = 0; 
    c.gridy = 3; 
    c.gridwidth = 1; 
    gridBag.setConstraints(oneGen, c); 
    add(oneGen); 

    contPlay = new JButton("Play"); 
    c.gridx = 1; 
    c.gridy = 3; 
    c.gridwidth = 1; 
      contPlay.setVisible(true); 
    gridBag.setConstraints(contPlay, c); 
    add(contPlay); 

    stop = new JButton("Stop"); 
    c.gridx = 2; 
    c.gridy = 3; 
    c.gridwidth = 1; 
      stop.setVisible(false); 
    gridBag.setConstraints(stop, c); 
    add(stop); 

    genCounter = new JLabel("Generation: 0"); 
    c.gridx = 0; 
    c.gridy = 1; 
    c.gridwidth = 1; 
    gridBag.setConstraints(genCounter, c); 
    add(genCounter); 
} 

class ButtonListener { 
    public void addActionListener(ActionEvent e) throws InterruptedException { 
     JButton source = (JButton)e.getSource(); 

     if(source == oneGen){ 
      nextGen(); 
     } 
     if(source == contPlay){ 
      paused = false; 
      canChange = false; 
          contPlay.setVisible(false); 
          stop.setVisible(true); 
      while (paused = false) { 
       nextGen(); 
       Thread.sleep(1000); 
      } 
     } 
     if(source == stop) { 
      paused = true; 
      canChange = false; 
          stop.setVisible(false); 
          contPlay.setVisible(true); 
     } 
    } 
} 

public void mouseClicked(MouseEvent e){ 
    int xco = e.getX() - board.getX(); 
    int yco = e.getY() - board.getY(); 
    if((e.getComponent() == board) && (paused == true)){ 
     if(grid[xco/5][yco/5] == 1){ 
      grid[xco/5][yco/5] = 0; 
      board.repaint(); 
     }else if(grid[xco/5][yco/5] == 0){ 
      grid[xco/5][yco/5] = 1; 
      board.repaint(); 
     } 
    } 
} 

public void makeGrid(int[][] emptyGrid) { 
    int x, y; 
    for(x = 0; x < ROW; x++){ 
     for(y = 0; y < COL; y++){ 
      emptyGrid[x][y] = 0; 
     } 
    } 
} 

public void nextGen() { 
    getNextGen(); 
    board.repaint(); 
    genCount++; 
    genCounter.setText("Generation: " + Integer.toString(genCount));   
} 

public void getNextGen() { 
    int x, y, neighbor; 
    makeGrid(nextGrid); 
    for(x = 0; x < ROW; x++){ 
     for(y=0; y<COL; y++){ 
      neighbor = calculate(x,y); 

      if(grid[x][y] != 0){ 
       if((neighbor >= min) && (neighbor <= max)) { 
        nextGrid[x][y] = neighbor; 
       } 
      }else { 
       if(neighbor == birth){ 
        nextGrid[x][y] = birth; 
       } 
      } 
     } 
    } 
    makeGrid(grid); 
    copyGrid(nextGrid,grid); 
} 

public void copyGrid(int[][] source, int[][] newGrid) { 
    int x, y; 
    for(x=0; x<ROW; x++){ 
     for(y=0; y<COL; y++){ 
      newGrid[x][y] = source[x][y]; 
     } 
    } 
} 

private int calculate(int x, int y){ 
    int a, b, total; 

    total = (grid[x][y]); 
    for (a = -1; a<= 1; a++) { 
     for (b = -1; b <= 1; b++){ 
      if(grid[(ROW + (x + a)) % ROW][(COL + (y + b)) % COL] != 0) { 
       total++; 
      } 
     } 
    } 
    return total; 
} 

@Override 
public void actionPerformed(ActionEvent arg0) { 
    // TODO Auto-generated method stub 

    } 
}  

如果有人能告诉我它有什么问题，那就太棒了。

Answer 1

你的一个主要问题是，你尝试在运行一个漫长的过程Swing事件线程，也被称为Event Dispatch Thread或EDT，这实际上会冻结您的程序。我看到发生这个问题在这里：

class ButtonListener { 
    public void addActionListener(ActionEvent e) throws InterruptedException { 
    JButton source = (JButton) e.getSource(); 

    // ... 

     while (paused = false) { // ******* 
      nextGen(); 
      Thread.sleep(1000); // ****** 
     } 
    } 

    // ... 

} 

你同时拥有while(true)环路和Thread.sleep(...)这两者都不应该在事件线程中调用。您可以使用Swing Timer。

有关Swing事件线程的更多信息，请阅读Concurrency in Swing。 （1），你在哪里允许单元格初始化为活着？如果没有活细胞开始，所有世代都将只显示一个空格子？您是否需要将MouseListener添加到一个或多个组件中？我认为这将是一个好主意。

另外（2）按钮通常在将动作侦听器添加到它们时会更好，这在Swing button tutorial中有很好的概述。你经历过Swing教程吗？如果没有，请检查他们（找到他们here），因为他们会帮助你很多，我想。 （3）你可能会咬掉更多的东西，因为你一次试图解决太多的问题。当我创建一个类似于此的GUI时，我喜欢单独对程序的每个部分进行处理，并在将它整合到一个大程序中之前先让它工作。举例来说，首先在非GUI模型上工作，并通过调用模型方法的测试代码来运行代码。接下来，每次在GUI的每个部分工作一次，包括JButtons，然后是MouseListener，然后显示生命游戏，然后执行这些世代。

调试较小的测试程序然后试图调试整个shebang相当容易，相信我。

Answer 2

也许尝试init（）中的super.init（）？

Answer 3

你的程序启动了我，但有其他问题。 这将帮助您开始使用固定的问题：

变化：

cell = new cellClass(); 

要：

cell = new cellClass(){{ 
    setLayout(new FlowLayout()); 
}};