# Need help on beginner project (Snake)

please format code with </> button * homework policy * asking questions

Hi i’m working on making snake with a 2d array. Right now i’m trying to move a single square but i can’t get it to work. When moving up or left everthing works fine but when moving the square down or to the right it dosn’t draw the differences in the squares position. Can someone please take a look at my code below and tell me what i’m doing wrong.

Here is the main tab code:

``````int rows= 15;
int columns= 15;
String direction = "none";

sqaure[][] sqaurematrix = new sqaure[rows][columns];

void setup() {
size (375, 375);
for (int y=0; y<columns; y++) {
for (int x=0; x<rows; x++) {
sqaurematrix[x][y]= new sqaure(x, y);
println(sqaurematrix[x][y].xpos, sqaurematrix[x][y].ypos );
}
}
}

void draw() {

for (int y=0; y<columns; y++) {
for (int x=0; x<rows; x++) {

sqaurematrix[x][y].display();
}
}

for (int y=0; y<columns; y++) {
for (int x=0; x<rows; x++) {
sqaurematrix[x][y].move();

}
}
}

void keyPressed() {
if (key == 'w') {
direction = "up";
}
if (key == 's') {
direction = "down";
}
if (key == 'a') {
direction = "left";
}
if (key == 'd') {
direction = "right";
}
}

``````

Here is the square object code:

``````class sqaure {
int xpos, ypos;
color c = color(0, 255, 0);

sqaure(int xpos, int ypos) {
this.xpos = xpos;
this.ypos = ypos;
}

void display() {
fill(0, 0, 255);
} else {
fill(c);
}
rect(xpos*25, ypos*25, 25, 25);
}

void move() {
if (direction=="up" && ypos>0) {
}
if (direction=="down" && ypos<columns-1) {
}
if (direction=="right" && xpos<rows-1) {
}
if (direction=="left" && xpos>0) {
}
delay(500);
println(xpos,ypos);
}
}
}

``````

better check String with equals please:

``````direction.equals("left")
``````

It’s `square with -ua-`

Okay thanks for the help, i have chanced everything that you mentioned but my problem still occurs.

``````int rows= 15;
int columns= 15;
String direction = "none";

square[][] squarematrix = new square[rows][columns];

void setup() {
size (375, 375);
for (int y=0; y<columns; y++) {
for (int x=0; x<rows; x++) {
squarematrix[x][y]= new square(x, y);
println(squarematrix[x][y].xpos, squarematrix[x][y].ypos );
}
}
}

void draw() {
println(direction);

for (int y=0; y<columns; y++) {
for (int x=0; x<rows; x++) {
squarematrix[x][y].display();
}
}

for (int y=0; y<columns; y++) {
for (int x=0; x<rows; x++) {
squarematrix[x][y].move();

}
}
}

void keyPressed() {
if (key == 'w') {
direction = "up";
}
if (key == 's') {
direction = "down";
}
if (key == 'a') {
direction = "left";
}
if (key == 'd') {
direction = "right";
}
}
class square {
int xpos, ypos;
color c = color(0, 255, 0);

square(int xpos, int ypos) {
this.xpos = xpos;
this.ypos = ypos;
}

void display() {
fill(0, 0, 255);
} else {
fill(c);
}
rect(xpos*25, ypos*25, 25, 25);
}

void move() {
if (direction.equals("up") && ypos>0) {
}
if (direction.equals("down") && ypos<columns-1) {
}
if (direction.equals("right") && xpos<rows-1) {
}
if (direction.equals("left") && xpos>0) {
}
delay(500);
println(xpos,ypos);
}
}
}
``````
``````

int rows= 15;
int columns= 15;
String direction = "none";

Square[][] squarematrix = new Square[rows][columns];

void setup() {
size (375, 375);
for (int y=0; y<columns; y++) {
for (int x=0; x<rows; x++) {
squarematrix[x][y]= new Square(x, y);
println(squarematrix[x][y].xpos, squarematrix[x][y].ypos );
}
}

frameRate(9);
}

void draw() {
println(direction);

for (int y=0; y<columns; y++) {
for (int x=0; x<rows; x++) {

squarematrix[x][y].display();
}
}

for (int y=0; y<columns; y++) {
for (int x=0; x<rows; x++) {
if (squarematrix[x][y].move())return;
}
}
}

void keyPressed() {
if (key == 'w') {
direction = "up";
}
if (key == 's') {
direction = "down";
}
if (key == 'a') {
direction = "left";
}
if (key == 'd') {
direction = "right";
}
}

void keyReleased() {
//direction = "";
}

// =================================================================

class Square {
int xpos, ypos;
color c = color(0, 255, 0);

Square(int xpos, int ypos) {
this.xpos = xpos;
this.ypos = ypos;
}//constr

void display() {
fill(0, 0, 255);
} else {
fill(c);
}
rect(xpos*25, ypos*25, 25, 25);
//  direction = "";
}

boolean move() {
if (direction.equals("up") && ypos>0) {
} else if (direction.equals("down") && ypos<columns-1) {
} else if (direction.equals("right") && xpos<rows-1) {
} else if (direction.equals("left") && xpos>0) {
}
delay(5);
println(xpos, ypos);
// direction = "";
return true;
}
return false;
}//method
} //class
//

``````
1 Like

Can you be so kind and explain to me why my code didn’t work and why it behaved different when moving up compared to moving down.

Good question.

It’s a bit tricky.

As you can see, the function `move()` is called again and again for every cell, searching the cell with the snake head and moving it. But when we moved the snake head, we don’t store the information that we are done.

Instead the nested for-loop goes on. What happens now (when we clicked down or right) is that it finds the next cell with (the newly placed) `snakehead` being true in `if (snakehead) {`. So the snake head is moved again. And then again. So it falls down (or to the right).
Because of the search direction is right and down in the nested for loop, this happens.

BUT when the movement was up or left, the snake head is moved once but because the search direction is down, the moved snake head is not found nor moved again.

My solution

I told the function `move() ` to return true once it found something and otherwise false.

And in draw() in the nested for-loop I said

``````if (squarematrix[x][y].move())return;
``````

which means it leaves the nested for-loop (and the function). Hence, we don’t move the snake head again.

Warm regards,

Chrisir

3 Likes