我的代码中的按钮有问题,该按钮旨在循环多个速度倍增器

Having issues with a button in my code that is meant to cycle through many speed multipliers

我正在为学校完成一个期末项目,在该项目中我对制作内容具有充分的创造力。我决定制作一个底部有几个按钮的简单 Conway 模拟器。一到play/pause,一到步,一到清除,一到变速。它是在处理过程中制作的,我尝试过实现速度按钮,但每当我单击它时,它总是恢复到 1 倍速度,即使这不是预期的行为。我的代码如下(处理 3):

void setup() {
   size(550, 615);
   pixelDensity(2);
   frameRate(60);
}

//initial array, set as 2 glider guns ----------------------------
boolean[][] colour =  {{false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false, true,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false, true,false,false,false, true,false,false, true, true, true,false,false,false,false,false,false,false,false,false,false, true,false,false,false,false,false,false, true, true,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false,false,false,false,false,false, true, true,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false, true, true,false,false,false,false, true,false,false,false,false, true,false,false,false,false,false,false,false, true, true,false,false, true,false,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false, true, true,false,false,false,false,false, true,false,false,false,false,false,false,false, true,false, true,false,false, true,false,false, true, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false, true,false,false,false, true,false,false,false,false,false, true, true, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false, true,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false, true,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true, true, true,false,false,false,false,false, true,false,false,false, true,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true, true,false,false, true,false,false, true,false, true,false,false,false,false,false,false,false, true,false,false,false,false,false, true, true,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false,false, true,false,false, true, true,false,false,false,false,false,false,false, true,false,false,false,false, true,false,false,false,false, true, true,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false, true, true,false,false,false,false,false,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false, true, true,false,false,false,false,false,false, true,false,false,false,false,false,false,false,false,false,false, true, true, true,false,false, true,false,false,false, true,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false, true,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false}
};
//----------------------------------------------------------------

//game variables -------------------------------------------------
int xposGame = 25;
int yposGame = 25;
int[] arraySize = {50, 50};
int slowDown = 4;
int speedMultiplier = slowDown;
//----------------------------------------------------------------

//play button variables ------------------------------------------
int xposPlay = xposGame;
int yposPlay = 550;
int playWidth = 40;
boolean playActive = false;
//----------------------------------------------------------------

//clear button variables -----------------------------------------
int clearWidth = 40;
int xposClear = 485;
int yposClear = 550;
//----------------------------------------------------------------

//step button variables ------------------------------------------
int stepWidth = 40;
int xposStep = xposGame + xposPlay + 25;
int yposStep = 550;
//----------------------------------------------------------------

//speed multiplier button variables ------------------------------
int speedWidth = 40;
int xposSpeed = xposGame + xposPlay + xposStep;
int yposSpeed = 550;
//----------------------------------------------------------------

//clear the board ------------------------------------------------
void clearArr(){
  for(int i = 0; i < arraySize[0]; i++){
    for(int j = 0; j < arraySize[1]; j++){
      colour[i][j] = false;
    }
  }
}
//----------------------------------------------------------------

//iteration function ---------------------------------------------
void iteration(){
  
 //counter for # of neighboring cells
 int neighborNum = 0;
  
 //next generations array
 boolean[][] newColour =  new boolean[arraySize[0]][arraySize[1]];
  
 //read the current generation's array
 for(int i = 0; i < arraySize[0]; i++){
   for(int j = 0; j < arraySize[1]; j++){
      
    //search all values around the cell (excluding those outside the array) and count how many neighbors are alive
    if(i != -0){
      if(j != 0){
        if(colour[i-1][j-1]==true){neighborNum++;}
      }
      if(colour[i-1][j]==true){neighborNum++;}
      if(j != arraySize[1]-1){
        if(colour[i-1][j+1]==true){neighborNum++;}
      }
    }
      
    if(j != 0){
      if(colour[i][j-1]==true){neighborNum++;}
    }
    if(j != arraySize[1]-1){
      if(colour[i][j+1]==true){neighborNum++;}
    }
      
    if(i != arraySize[0]-1){
      if(j != 0){
        if(colour[i+1][j-1]==true){neighborNum++;}
      }
      if(colour[i+1][j]==true){neighborNum++;}
      if(j != arraySize[1]-1){
        if(colour[i+1][j+1]==true){neighborNum++;}
      }
    }
        
    //check if the current cell is alive. if it has 2-3 neighbors, it survives, else it doesnt
    if(colour[i][j]==true){
      if(neighborNum==3||neighborNum==2){
        newColour[i][j] = true;
      }
      else{
        newColour[i][j] = false;
      }
    }
        
    //check if the current cell is dead. if it has 3 neighbors, birth it next round
    if(colour[i][j]==false){
      if(neighborNum==3){
        newColour[i][j] = true;
      }
      else{
        newColour[i][j] = false;
      }
    }
       
    //set the number of neighbors on the current cell to 0 for the next cell
    neighborNum = 0;
   }
 }
  
  //set the new array to be the current array
  colour = newColour;
}
//----------------------------------------------------------------

//iteration drawing function -------------------------------------
void drawIteration(){
  for(int i = 0; i < arraySize[0]; i++){
    for(int j = 0; j < arraySize[1]; j++){
      if(colour[i][j] == true){
        fill(255);
      }
      else{fill(0);}
      strokeWeight(2);
      stroke(50);
      rect(10*j+xposGame, 10*i+yposGame, 10, 10);
    }
  }
}
//----------------------------------------------------------------

//play/pause button function -------------------------------------
void drawPlay(){
  noStroke();
  fill(0);
  rect(xposPlay, yposPlay, playWidth, playWidth, 5);
  fill(255);
  if(!playActive){
    triangle(xposPlay + (playWidth/3), yposPlay + (playWidth/3.3), xposPlay + (playWidth/3), yposPlay + playWidth - (playWidth/3.3), xposPlay + (playWidth*3.5/5), yposPlay + (playWidth/2));
  }
  if(playActive){
    rect(xposPlay + (playWidth/3), yposPlay + (playWidth/3.3), playWidth/9, playWidth-(2*(playWidth/3.3)), 2);
    rect(xposPlay + playWidth - (playWidth/3) - (playWidth/9), yposPlay + (playWidth/3.3), playWidth/9, playWidth-(2*(playWidth/3.3)), 2);
  }
}
//----------------------------------------------------------------

//clear button function ------------------------------------------
void drawClear(){
  noStroke();
  fill(0);
  rect(xposClear, yposClear, clearWidth, clearWidth, 5);
  strokeWeight(clearWidth/9);
  stroke(255);
  line(xposClear+clearWidth/3, yposClear + clearWidth/3, xposClear + 2 * clearWidth/3, yposClear + 2 * clearWidth/3);
  line(xposClear+clearWidth/3, yposClear + 2 * clearWidth/3, xposClear + 2 * clearWidth/3, yposClear + clearWidth/3);
}
//----------------------------------------------------------------

//step button function -------------------------------------------
void drawStep(){
  noStroke();
  fill(0);
  rect(xposStep, yposStep, stepWidth, stepWidth, 5);
  strokeWeight(stepWidth/9);
  stroke(255);
  line(xposStep + stepWidth/3, yposStep + stepWidth/3, xposStep + (stepWidth*3.5/5), yposStep + (stepWidth/2));
  line(xposStep + stepWidth/3, yposStep + 2*stepWidth/3, xposStep + (stepWidth*3.5/5), yposStep + (stepWidth/2));
}
//----------------------------------------------------------------

//speed multiplier button ----------------------------------------
void drawSpeed(){
  noStroke();
  fill(0);
  rect(xposSpeed, yposSpeed, speedWidth, speedWidth, 5);
  fill(255);
  textAlign(CENTER);
  textSize(25);
  if(slowDown == 12){
    text("1x", xposSpeed + (speedWidth/2), yposSpeed + (speedWidth/1.4));
  }
  if(slowDown == 6){
    text("2x", xposSpeed + (speedWidth/2), yposSpeed + (speedWidth/1.4));
  }
  if(slowDown == 4){
    text("3x", xposSpeed + (speedWidth/2), yposSpeed + (speedWidth/1.4));
  }
  if(slowDown == 3){
    text("4x", xposSpeed + (speedWidth/2), yposSpeed + (speedWidth/1.4));
  }
}
//----------------------------------------------------------------

//all mousePressed events ----------------------------------------
void mousePressed(){
  
  //play button
  if(mouseX > xposPlay && mouseX < xposPlay + playWidth && mouseY > yposPlay && mouseY < yposPlay + playWidth){
    playActive = !playActive;
  }
  
  //clear button
  if(mouseX > xposClear && mouseX < xposClear + clearWidth && mouseY > yposClear && mouseY < yposClear + clearWidth){
    clearArr();
    playActive = false;
  }
  
  //step button
  if(mouseX > xposStep && mouseX < xposStep + stepWidth && mouseY > yposStep && mouseY < yposStep + stepWidth){
    iteration();
    playActive = false;
  }
  
  //speed multiplier button
  if(mouseX > xposSpeed && mouseX < xposSpeed + speedWidth && mouseY > yposSpeed && mouseY < yposSpeed + speedWidth){
    if(slowDown == 12){
      slowDown = 6;
    }
    if(slowDown == 6){
      slowDown = 4;
    }
    if(slowDown == 4){
      slowDown = 3;
    }
    if(slowDown == 3){
      slowDown = 2;
    }
    if(slowDown == 2){
      slowDown = 12;
    }
  }
  
  //adding cells
  if(!playActive && mouseX > xposGame && mouseX < xposGame + 10*arraySize[0] && mouseY > yposGame && mouseY < yposGame + 10*arraySize[1]){
      colour[(mouseY-yposGame)/10][(mouseX-xposGame)/10] = !colour[(mouseY-yposGame)/10][(mouseX-xposGame)/10];
  }
}
//----------------------------------------------------------------

void draw() {
  background(50);
  noStroke();
  fill(50);
  
  drawPlay();
  drawClear();
  drawStep();
  drawSpeed();
  
  if(speedMultiplier == 0){
    if(playActive){ 
      iteration();
    }
    speedMultiplier = slowDown;
  }
  
  speedMultiplier --;
  
  drawIteration();
}

您的这部分逻辑看起来过于复杂且容易出错:

if(slowDown == 12){
      slowDown = 6;
    }
    if(slowDown == 6){
      slowDown = 4;
    }
    if(slowDown == 4){
      slowDown = 3;
    }
    if(slowDown == 3){
      slowDown = 2;
    }
    if(slowDown == 2){
      slowDown = 12;
    }

一个基本的例子是,如果 slowDown6,您将其设置为 4,但是在下一个条件中,您检查它是否是 4 并将其更改为 3 所以因为它是 6,它不仅会变成 4,而且会立即变成 3。你所有的条件都滴下来,你最终在 12 循环中。

试试看:

if(mouseX > xposSpeed && mouseX < xposSpeed + speedWidth && mouseY > yposSpeed && mouseY < yposSpeed + speedWidth){
    println("before", slowDown);
    if(slowDown == 12){
      slowDown = 6;
    }
    if(slowDown == 6){
      slowDown = 4;
    }
    if(slowDown == 4){
      slowDown = 3;
    }
    if(slowDown == 3){
      slowDown = 2;
    }
    if(slowDown == 2){
      slowDown = 12;
    }
    println("after", slowDown);
  }

我能想到的最简单的方法就是将 slowDown 设置为一个值:

if(mouseX > xposSpeed && mouseX < xposSpeed + speedWidth && mouseY > yposSpeed && mouseY < yposSpeed + speedWidth){
    slowDown = 1;
  }

如果你想要多个速度倍增器,也许你打算使用 switch: ?

if(mouseX > xposSpeed && mouseX < xposSpeed + speedWidth && mouseY > yposSpeed && mouseY < yposSpeed + speedWidth){
    println("before", slowDown);
    switch(slowDown){
      case 12:
        slowDown = 6;
        break;
      case 6:
        slowDown = 4;
        break;
      case 4:
        slowDown = 3;
        break;
      case 3:
        slowDown = 2;
        break;
      case 2:
        slowDown = 12;
        break;
    }
    println("after", slowDown);
  }

就实际行为而言,根据按钮标签,如果看起来您想要循环浏览速度预设:1x、2x、3x、4x,然后返回到 1x。

或者,您可以管理一组预设选项并简单地递增(和循环)索引。您也可以选择使用预设文本标签数组。

例如:

int speedPresetIndex = 0;
int[] speedPresetValues = {12, 6, 4, 3, 2};
String[] speedPresetLabels = {"1x","2x","3x","4x",""};
...
if(mouseX > xposSpeed && mouseX < xposSpeed + speedWidth && mouseY > yposSpeed && mouseY < yposSpeed + speedWidth){
    // increment index (looping back to start using %)
    speedPresetIndex = (speedPresetIndex + 1) % speedPresetValues.length; 
    // update slowDown based on preset
    slowDown = speedPresetValues[speedPresetIndex];
  }
...
text(speedPresetLabels[speedPresetIndex], xposSpeed + (speedWidth/2), yposSpeed + (speedWidth/1.4));

这是经过上述更改后的代码修改版本:

void setup() {
   size(550, 615);
   pixelDensity(2);
   frameRate(60);
   println(colour.length, colour[0].length);
}

int speedPresetIndex = 0;
int[] speedPresetValues = {12, 6, 4, 3, 2};
String[] speedPresetLabels = {"1x","2x","3x","4x",""};


//initial array, set as 2 glider guns ----------------------------
boolean[][] colour =  {{false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false, true,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false, true,false,false,false, true,false,false, true, true, true,false,false,false,false,false,false,false,false,false,false, true,false,false,false,false,false,false, true, true,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false,false,false,false,false,false, true, true,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false, true, true,false,false,false,false, true,false,false,false,false, true,false,false,false,false,false,false,false, true, true,false,false, true,false,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false, true, true,false,false,false,false,false, true,false,false,false,false,false,false,false, true,false, true,false,false, true,false,false, true, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false, true,false,false,false, true,false,false,false,false,false, true, true, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false, true,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false, true,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true, true, true,false,false,false,false,false, true,false,false,false, true,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true, true,false,false, true,false,false, true,false, true,false,false,false,false,false,false,false, true,false,false,false,false,false, true, true,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false,false, true,false,false, true, true,false,false,false,false,false,false,false, true,false,false,false,false, true,false,false,false,false, true, true,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false, true, true,false,false,false,false,false,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false, true, true,false,false,false,false,false,false, true,false,false,false,false,false,false,false,false,false,false, true, true, true,false,false, true,false,false,false, true,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false, true,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true,false,false, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false, true, true,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false},
                      {false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false,false}
};
//boolean[][] colour = new boolean[50][50];
//----------------------------------------------------------------

//game variables -------------------------------------------------
int xposGame = 25;
int yposGame = 25;
int[] arraySize = {50, 50};
int slowDown = 4;
int speedMultiplier = slowDown;
//----------------------------------------------------------------

//play button variables ------------------------------------------
int xposPlay = xposGame;
int yposPlay = 550;
int playWidth = 40;
boolean playActive = false;
//----------------------------------------------------------------

//clear button variables -----------------------------------------
int clearWidth = 40;
int xposClear = 485;
int yposClear = 550;
//----------------------------------------------------------------

//step button variables ------------------------------------------
int stepWidth = 40;
int xposStep = xposGame + xposPlay + 25;
int yposStep = 550;
//----------------------------------------------------------------

//speed multiplier button variables ------------------------------
int speedWidth = 40;
int xposSpeed = xposGame + xposPlay + xposStep;
int yposSpeed = 550;
//----------------------------------------------------------------

//clear the board ------------------------------------------------
void clearArr(){
  for(int i = 0; i < arraySize[0]; i++){
    for(int j = 0; j < arraySize[1]; j++){
      colour[i][j] = false;
    }
  }
}
//----------------------------------------------------------------

//iteration function ---------------------------------------------
void iteration(){
  
 //counter for # of neighboring cells
 int neighborNum = 0;
  
 //next generations array
 boolean[][] newColour =  new boolean[arraySize[0]][arraySize[1]];
  
 //read the current generation's array
 for(int i = 0; i < arraySize[0]; i++){
   for(int j = 0; j < arraySize[1]; j++){
      
    //search all values around the cell (excluding those outside the array) and count how many neighbors are alive
    if(i != -0){
      if(j != 0){
        if(colour[i-1][j-1]==true){neighborNum++;}
      }
      if(colour[i-1][j]==true){neighborNum++;}
      if(j != arraySize[1]-1){
        if(colour[i-1][j+1]==true){neighborNum++;}
      }
    }
      
    if(j != 0){
      if(colour[i][j-1]==true){neighborNum++;}
    }
    if(j != arraySize[1]-1){
      if(colour[i][j+1]==true){neighborNum++;}
    }
      
    if(i != arraySize[0]-1){
      if(j != 0){
        if(colour[i+1][j-1]==true){neighborNum++;}
      }
      if(colour[i+1][j]==true){neighborNum++;}
      if(j != arraySize[1]-1){
        if(colour[i+1][j+1]==true){neighborNum++;}
      }
    }
        
    //check if the current cell is alive. if it has 2-3 neighbors, it survives, else it doesnt
    if(colour[i][j]==true){
      if(neighborNum==3||neighborNum==2){
        newColour[i][j] = true;
      }
      else{
        newColour[i][j] = false;
      }
    }
        
    //check if the current cell is dead. if it has 3 neighbors, birth it next round
    if(colour[i][j]==false){
      if(neighborNum==3){
        newColour[i][j] = true;
      }
      else{
        newColour[i][j] = false;
      }
    }
       
    //set the number of neighbors on the current cell to 0 for the next cell
    neighborNum = 0;
   }
 }
  
  //set the new array to be the current array
  colour = newColour;
}
//----------------------------------------------------------------

//iteration drawing function -------------------------------------
void drawIteration(){
  for(int i = 0; i < arraySize[0]; i++){
    for(int j = 0; j < arraySize[1]; j++){
      if(colour[i][j] == true){
        fill(255);
      }
      else{fill(0);}
      strokeWeight(2);
      stroke(50);
      rect(10*j+xposGame, 10*i+yposGame, 10, 10);
    }
  }
}
//----------------------------------------------------------------

//play/pause button function -------------------------------------
void drawPlay(){
  noStroke();
  fill(0);
  rect(xposPlay, yposPlay, playWidth, playWidth, 5);
  fill(255);
  if(!playActive){
    triangle(xposPlay + (playWidth/3), yposPlay + (playWidth/3.3), xposPlay + (playWidth/3), yposPlay + playWidth - (playWidth/3.3), xposPlay + (playWidth*3.5/5), yposPlay + (playWidth/2));
  }
  if(playActive){
    rect(xposPlay + (playWidth/3), yposPlay + (playWidth/3.3), playWidth/9, playWidth-(2*(playWidth/3.3)), 2);
    rect(xposPlay + playWidth - (playWidth/3) - (playWidth/9), yposPlay + (playWidth/3.3), playWidth/9, playWidth-(2*(playWidth/3.3)), 2);
  }
}
//----------------------------------------------------------------

//clear button function ------------------------------------------
void drawClear(){
  noStroke();
  fill(0);
  rect(xposClear, yposClear, clearWidth, clearWidth, 5);
  strokeWeight(clearWidth/9);
  stroke(255);
  line(xposClear+clearWidth/3, yposClear + clearWidth/3, xposClear + 2 * clearWidth/3, yposClear + 2 * clearWidth/3);
  line(xposClear+clearWidth/3, yposClear + 2 * clearWidth/3, xposClear + 2 * clearWidth/3, yposClear + clearWidth/3);
}
//----------------------------------------------------------------

//step button function -------------------------------------------
void drawStep(){
  noStroke();
  fill(0);
  rect(xposStep, yposStep, stepWidth, stepWidth, 5);
  strokeWeight(stepWidth/9);
  stroke(255);
  line(xposStep + stepWidth/3, yposStep + stepWidth/3, xposStep + (stepWidth*3.5/5), yposStep + (stepWidth/2));
  line(xposStep + stepWidth/3, yposStep + 2*stepWidth/3, xposStep + (stepWidth*3.5/5), yposStep + (stepWidth/2));
}
//----------------------------------------------------------------

//speed multiplier button ----------------------------------------
void drawSpeed(){
  noStroke();
  fill(0);
  rect(xposSpeed, yposSpeed, speedWidth, speedWidth, 5);
  fill(255);
  textAlign(CENTER);
  textSize(25);
  text(speedPresetLabels[speedPresetIndex], xposSpeed + (speedWidth/2), yposSpeed + (speedWidth/1.4));
}
//----------------------------------------------------------------

//all mousePressed events ----------------------------------------
void mousePressed(){
  
  //play button
  if(mouseX > xposPlay && mouseX < xposPlay + playWidth && mouseY > yposPlay && mouseY < yposPlay + playWidth){
    playActive = !playActive;
  }
  
  //clear button
  if(mouseX > xposClear && mouseX < xposClear + clearWidth && mouseY > yposClear && mouseY < yposClear + clearWidth){
    clearArr();
    playActive = false;
  }
  
  //step button
  if(mouseX > xposStep && mouseX < xposStep + stepWidth && mouseY > yposStep && mouseY < yposStep + stepWidth){
    iteration();
    playActive = false;
  }
  
  //speed multiplier button
  if(mouseX > xposSpeed && mouseX < xposSpeed + speedWidth && mouseY > yposSpeed && mouseY < yposSpeed + speedWidth){
    // increment index (looping back to start using %)
    speedPresetIndex = (speedPresetIndex + 1) % speedPresetValues.length; 
    // update slowDown based on preset
    slowDown = speedPresetValues[speedPresetIndex];
  }
  
  //adding cells
  if(!playActive && mouseX > xposGame && mouseX < xposGame + 10*arraySize[0] && mouseY > yposGame && mouseY < yposGame + 10*arraySize[1]){
      colour[(mouseY-yposGame)/10][(mouseX-xposGame)/10] = !colour[(mouseY-yposGame)/10][(mouseX-xposGame)/10];
  }
}
//----------------------------------------------------------------

void draw() {
  background(50);
  noStroke();
  fill(50);
  
  drawPlay();
  drawClear();
  drawStep();
  drawSpeed();
  
  if(speedMultiplier == 0){
    if(playActive){ 
      iteration();
    }
    speedMultiplier = slowDown;
  }
  
  speedMultiplier --;
  
  drawIteration();
}

希望这能解决您的主要问题。

此外,我还有一些可能有用的建议:

  • avoid repeating yourself in code. all mouse / button rectangle boundary checks could be encapsulate the logic in a reusable function. (If class at school went over classes you could implement and reuse a Button class). You can see example in 。尽管它在 p5.js 中,但语法非常相似。
  • 同样,如果 classes 被教导,2D CA 可以被封装到可重复使用的 class
  • 最好有这样的建议:您可以使用 .png 图像,其中 true 标记为黑色背景上的白色像素,而不是像这样手动定义大型 2D 布尔数组背景。使用这种方法,未​​来的版本可能不仅可以绘制,还可以保存/加载 2D CA 配置。

其余的一点都不差(相当干净的代码,一致的命名等)!干得漂亮!