ゲーム

Question

import processing.core.PApplet; 

    public class gl extends PApplet { 

    static int neighborCount; 
    static int screenRows; 
    int tNC; // Temporary Neighbor Count 
    int newState; 

    int columns = 960; 
    int rows = 477; 

    int[][] cells = new int[columns][rows]; 
    int[][] newGen = new int[columns][rows]; 

    public static void main(String[] args) { 
     PApplet.main("gl"); 
    } 

    public void settings() { 
     size(1920, 955); 
    } 

    public void setup() { 
     // Set background white and all of cells[][] to 0 or 1 
     screenRows = 0; 
     background(255); 
     for (int j = 0; j < (rows/2); j++) { 
      for (int i = 0; i < (columns/2); i++) { 
       cells[i][j] = (int) random(0, 2); 
      } 
     } 
    } 

    public void draw() { 
     // If program has finished generating this frame, reset everything and set cells[][] equal to newGen[][] 
     if (screenRows > (height/2)) { 
      screenRows = 0; 
      System.out.println("End of generation reached"); 
      background(255); 
      cells = newGen.clone(); 
      for (int i = 0; i < columns; i++) { 
       for (int j = 0; j < rows; j++) { 
        newGen[i][j] = 0; 
       } 
      } 
     } 
     // Go through every element in cells[][], determine it's value, and display it 
     for (int x = 1; x < (width/2) - 1; x++) { 
      for (int y = 1; y < (height/2) - 1; y++) { 

       printCell(x, y); 
      } 
     } 
     screenRows++; 

    } 

    public void printCell(int x, int y) { 
     setCellState(x, y); 

     if (newGen[x][y] == 0) { 
      stroke(255); 
      fill(255); 

     } else if (newGen[x][y] == 1) { 
      stroke(0); 
      fill(0); 

     } 
     System.out.println(x + ", " + y); 
     rect(x, y, 2, 2); 
    } 

    public void setCellState(int x, int y) { 
     tNC = getNeighborCount(x, y); 
     neighborCount = 0; 
     System.out.println(tNC); 

     if (tNC < 2) { // If less than 2 neighbors, cell dead 
      newGen[x][y] = 0; 

     } else if (tNC > 3) { // If more than 3 neighbors, cell dead 
      newGen[x][y] = 0; 

     } else if ((tNC == 2 || tNC == 3) && cells[x][y] == 1) { // If 2 or 3 neighbors and cell is alive, do nothing (unnecessary statement but makes visualizing easier) 

     } else if (tNC == 3 && cells[x][y] == 0) { // If 3 neighbors and cell is dead, cell is alive 
      newGen[x][y] = 1; 

     } else if (tNC == 2 && cells[x][y] == 0) { // If 2 neighbors and cel is dead, do nothing (also unnecessary) 

     } else { 
      System.out.println("Error in setCellState(int, int);"); // In event of none of the conditions being met 
     } 
     tNC = 0; // Reset variable (probably unnecessary but might as well) 
    } 

    public int getNeighborCount(int x, int y) { 
     // Go through each cell adjacent or diagonal to the cell and add it's value (0 or 1) to neighborCount 
     for (int i = -1; i < 2; i++) { 
      for (int j = -1; j < 2; j++) { 
       neighborCount += cells[i + x][j + y]; 
      } 
     } 
     // Subtract the value of the cell being evaluated from neighborCount as that is not a factor in the sum of the neighbors 
     neighborCount -= cells[x][y]; 
     return neighborCount; 
    } 
} 

Pastebin

私はちょうど今のところ、スピードの上に機能性のためつもりです。

私はConwayのGame of LifeをEclipseでの処理を使ってコーディングしようとしています。上記のコードは複数の点で機能不全です：

表示されている世代は、私がしたいと思うよりはるかに小さいです。各セルを2×2ピクセルにし、ウィンドウの高さと幅が半分になるようにすることで、これを相殺する努力をしているにもかかわらず、ウィンドウのほんの一部しか占めていません。

また、数秒後に最初の世代が表示された後、ウィンドウで世代が更新されないように見えます。

私はそれはあなたは三つの主要な問題を持っている0〜7

Answer 1

に任意の数と同じでなければならないとき、変数TNCは、多くの場合、0に等しいことに気づきました。

問題1：あなたはセルをレンダリングするようあなたは大丈夫であるかもしれない、次の世代を生成しているように見える...しかし、あなたはscreenRowsロジック（あなたdraw()機能でifのステートメント）で何をしているの？

私があなただったら、あなたのロジックを2つのセクションに分割します：あなたのボードを描画する1つの関数を書くこと、そして現在のボードに基づいて新しいボードを返すもう1つの関数です。現在の世代を描いているときに次の世代を計算しようとするのをやめてください。これはちょっとした頭痛を与えるだけです。

また、配列を切り替えるロジックが正しいとは思わない。どちらの配列が現在の世代を保持し、次の世代を保持していますか？本気ですか？

問題2：ピクセルサイズと配列座標を切り替えるようです。たとえば、配列インデックス座標で各セルを描画していますが、それらのセルを2x2の長方形として描画しています。とにかく次のセルでその上に描画するだけなので、これは意味がありません。ここでも、ロジックを分離します。ウィンドウwidthとheightに基づいてセルを描画する関数を作成し、配列の位置と配列の長さを作成します。

問題3：あなたのプリントステートメントがあなたのフレームレートを殺しています。 printステートメントは非常に遅いです。あなたのフレームレートは、あなたがやっているすべての計算のためにかなり遅いですが、1フレームごとに印刷するとさらに遅くなります（960 * 477 * 2）。これは実際には論理エラーではありませんが、プログラムが何をしているかを正確に見ることは難しくなります。

解決策：問題を解決するには、コードをかなりリファクタリングすることをお勧めします。私があなただったら、新しいプログラムからやり直すだろう。次に：

ステップ1：次の世代を計算するためにロジックから描画ロジックを分離します。描画用の関数と現在の関数に基づいて新しい配列を返す関数の2つを作成します。

手順2：描画コードで、配列インデックスとピクセル位置を分けてください。多分、セルの位置をとり、ウィンドウのサイズと配列の大きさに基づいて矩形を描く別の関数を書いてください。

PS：あなたはthis personと同じクラスですか？ Daniel Shiffmanのコードも使用していますか？