Is this bugged ?

Question about the task

Resolved

Everything is as per requirements still not passing. It keeps saying to call move(0,-1) in case SnakeDirection.UP and other requirements which I am satisfying. Only three requirements pass. I am adding this code in Snake class, as requirement says.

public void move(){

        if(isAlive()){
            if(getDirection().equals(SnakeDirection.UP)) move(0,-1);
            else if(getDirection().equals(SnakeDirection.RIGHT)) move(1,0);
            else if(getDirection().equals(SnakeDirection.DOWN)) move(0,1);
            else if(getDirection().equals(SnakeDirection.LEFT)) move(-1,0);
        }

    }

    public void move(int a, int b){

    }

Comments (9)

Popular
New
Old

You must be signed in to leave a comment

Andrei

Level 41

2 April 2021, 12:47

I just had the same issue for about 20 minutes trying to figure it out...

Maria

Level 34 , Utrecht, Netherlands

5 October 2020, 16:48

You should use variable direction directly, without getDirection() method It worked for me

if (isAlive()) {
            if (direction == SnakeDirection.UP) {

Tumelo Mohlala

Level 31 , Johannesburg, South Africa

10 August 2020, 08:39

What is wrong with this code?

if (isAlive) {
            SnakeDirection direction = getDirection();

            switch (direction) {
                case UP:
                    move(0, -1);
                    break;
                case RIGHT:
                    move(1,0);
                    break;
                case DOWN:
                    move(0,1);
                    break;
                case LEFT:
                    move(-1, 0);
                    break;
            }
        }

Tumelo Mohlala

Level 31 , Johannesburg, South Africa

10 August 2020, 08:38

I'm having this same issue but my code seems as per requirement.

if (isAlive) {
            SnakeDirection direction = getDirection();

            if (direction == SnakeDirection.UP) {
                move(0, -1);
            } else if (direction == SnakeDirection.RIGHT) {
                move(1, 0);
            } else if (direction == SnakeDirection.DOWN) {
                move(0, 1);
            } else {
                move(-1,0);
            }
        }

Lawson

Level 29 , Lagos, Nigeria

23 September 2020, 16:03

I thimk u should have used equals

Guadalupe Gagnon

Level 37 , Tampa, United States

8 April 2019, 21:44

solution

Try comparing like for them all like this:

direction /*or getDirection()*/ == SnakeDirection.DOWN

I don't know if this is your issue or not, but it is the only difference between my code and yours. I read somewhere that '==' can check for null values.

Guadalupe Gagnon

Level 37 , Tampa, United States

8 April 2019, 22:22

also the last statement i just went with 'else' and not 'else if()'

Ankush Rajput

Level 25 , Bhopal, India

9 April 2019, 06:02

it passed when using direction and was not passing with getDirection() ? Any idea why ? For null check, I also tried using Objects.equals() and deepEquals() methods. Still failed.

Guadalupe Gagnon

Level 37 , Tampa, United States

9 April 2019, 13:08

I couldn't tell you why. direction and getDirection() return the same exact value. Just another codegym quark to add to the list.