2012-04-01 17 views
1

私はConwayのGame of Lifeのグラフィカルなモデルを作っているプログラムに取り組んでいますが、起動したら何もしないようにしています。ボタンは機能せず、グリッドは変化しません。私は間違って何をしていますか?Java Game of Lifeプログラムが動作しないのはなぜですか?

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 

    } 
}  

誰かがそれに間違っていると教えてもらえれば、それはすばらしいでしょう。

+0

アプレットは[イベントディスパッチスレッド](http://download.oracle.com/javase/tutorial/uiswing/concurrency/initial.html)のGUIオブジェクトでなければなりません。 – trashgod

+0

この[Game of Life](http://stackoverflow.com/a/8200046/418556)のバージョンも参照してください。アプレットは高度なトピックです。 'JFrame'ベースのアプリケーションをコードします。今のところは。 –

答えて

5

一つあなたが長いプロセスを実行しようとしているということですEvent Dispatch ThreadまたはEDTとも呼ばれるSwingイベントスレッドは、実際にプログラムをフリーズします。

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を使用する必要があります。

スイングイベントスレッドの詳細については、Concurrency in Swingを参照してください。

また、(1)では、セルの初期化をどこで有効にすることができますか?すべての世代から始まる生存細胞がなければ、空のグリッドだけが表示されますか? 1つまたは複数のコンポーネントにMouseListenerを追加する必要がありますか?これは良いアイデアだと思う。

また、(2)ボタンは、Swing button tutorialで概説されているように、ActionListenersを追加すると、よりよく機能します。あなたはSwingチュートリアルを終えましたか?そうでない場合は、それらをチェックアウトしてください(here)あなたがかなり手伝ってくれると思います。

また、(3)一度にあまりに多くの問題を解決しようとしているということで、あなたが噛んでいる以上に噛んでいるかもしれません。これと同様のGUIを作成すると、プログラムの各部分を孤立して処理してから、すべてを1つの大きなプログラムにまとめる前に作業したいと思っています。たとえば、まずGUI以外のモデルを作成し、モデルのメソッドを呼び出すテストコードを実行して世代を動作させます。 JButtons、次にMouseListenerを含むGUIの各部分の次の作業、次に生命のゲームの表示、そして世代の実行。

小さなテストプログラムをデバッグしてシバン全体をデバッグしようとする方がずっと簡単です。信頼してください。

+0

助けてくれてありがとう。これは間違いなく私がそれを働かせるのを助けるでしょう。 – Areth

+0

@Areth:ようこそ。また、「(3)」もお読みください。 –

+1

@HovercraftFullOfEelsあなたは 'while(paused = false)'が 'while(true)'に等しいと確信していますか?これは代入であり、比較ではありません( '==')。私はこれが有効なJava構文であることを知らなかった。私のIDEでは、while(false)isoと同等であることを示唆しています。while(true) ' – Robin

0

おそらく、init()でsuper.init()を試してみてください。

+2

この回答はどのように役立つでしょうか?真剣に。 –

+1

@RayTayek - 回答を投稿する前に、本当に確認してください。 –

1

あなたのプログラムは私のために始まりますが、他の問題があります。

cell = new cellClass(); 

変更: これはあなたの問題を修正し始めるでしょう、あなたの主な問題の

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

ありがとう、私はそれを新しいクラス(新しいプロジェクト)にC&Pしてロードしました。おそらく私が気づいていたはずのことを指摘してくれてありがとう。 – Areth

関連する問題