JavaRush/Помощь по Java задачам/Хвост змейки

22 уровень

12.02.2019
1561просмотров
17комментариев

Хвост змейки

Вопрос по задаче

Решен

что-то не так с итератором, но что - не понимаю

Движение змейки очень простое: в соседней с головой ячейке создается новая голова, а последний элемент (хвост) змейки удаляется.

В классе Snake реализуй метод createNewHead(), который создает новый элемент GameObject и возвращает его. Новый элемент GameObject должен создаваться рядом с элементом, в котором на данный момент находится голова змейки (элемент с индексом 0 из snakeParts).

Новый элемент GameObject создается с той стороны, куда движется змейка (поле direction).

Пример:

- если змейка движется влево, new GameObject(headX-1, headY);
- если змейка движется вниз, new GameObject(headX, headY + 1)

Здесь headX — координата головы змеи x, headY — координата головы змеи y.

Примечание: координаты могут быть отрицательные и/или быть за границами игрового поля. Пока что это нормально, в дальнейших частях задачи исправим.

В классе Snake создай метод removeTail() и реализуй его.

Требования:

В классе Snake должен существовать публичный метод GameObject createNewHead().
Метод createNewHead() должен быть реализован согласно условию.
В классе Snake должен существовать публичный метод void removeTail().
Метод removeTail() должен удалять последний элемент из списка snakeParts.

package com.javarush.games.snake; import com.javarush.engine.cell.*; public class SnakeGame extends Game{ public static final int WIDTH = 15; public static final int HEIGHT = 15; @Override public void initialize() { setScreenSize(WIDTH,HEIGHT); createGame(); } private Snake snake; private int turnDelay; @Override public void onTurn(int a ){ snake.move(); drawScene(); } private void createGame(){ Snake snake1 = new Snake(WIDTH/2,HEIGHT/2); snake = snake1; drawScene(); turnDelay = 300; setTurnTimer(turnDelay); } private void drawScene(){ for(int i = 0; i < WIDTH; i++){ for(int j = 0; j < HEIGHT; j++){ setCellColor(i,j,Color.GREEN); } } snake.draw(this); } }

Комментарии (17)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Дмитрий

Уровень 15

19 апреля 2019, 09:10

с хвостом можно вообще в 1 строчку:

public void removeTail(){
        snakeParts.remove(snakeParts.get(snakeParts.size() - 1));
    }

Михаил Евреинов Android Developer

24 ноября 2020, 12:52

public void removeTail() {
        snakeParts.remove(snakeParts.size()-1);
    }

ти передаешь в метод remove объект, а нужно индекс

Anika

Уровень 22

12 февраля 2019, 22:48

может кто-то по-другому делает??

Denys

Уровень 34

12 февраля 2019, 22:49

а можете показать где собственно проблема? в каком методе? в какой строке? не понятно куда нужно смотреть...

Anika

Уровень 22

12 февраля 2019, 22:51

метод removeTail() в классе Snake

Denys

Уровень 34

12 февраля 2019, 22:55

А если snakeParts.remove(snakeParts.size() - 1); ?

Denys

Уровень 34

12 февраля 2019, 23:02

Но поинтересуюсь спросить, что у вас вообще происходит с 14 по 22 строку?)

Anika

Уровень 22

12 февраля 2019, 23:08

пытаюсь удалять хвост змеи)

Denys

Уровень 34

12 февраля 2019, 23:10

прошел вариант с

snakeParts.remove(snakeParts.size() - 1);

Anika

Уровень 22

12 февраля 2019, 23:13

ругается на 16 строку этого же класса

Denys

Уровень 34

12 февраля 2019, 23:15

Я думаю что ожидалось что-то типа этого кода, в вашем предоставленном варианте:

Iterator<GameObject> snakeIterator = snakeParts.iterator();
while (true) {
    if (snakeIterator.hasNext()) {
        snakeIterator.next();
    } else {
        snakeIterator.remove();
        break;
    }
}

Конечно же там ругается... Вы совсем не понятно что хотите сделать в цикле for, наверно вместо оператора сравнения ожидалось двоеточие?) Получается два варианта (либо for (true) либо for (false)). В данном случае тело цикла не будет выполнено никогда. Следовательно хвост не удаляется и задача работает некорректно.

Anika

Уровень 22

12 февраля 2019, 23:19

" Проверь, что после вызова метода removeTail() размер змейки уменьшается на 1."

Denys

Уровень 34

12 февраля 2019, 23:20

полезный

Я имел ввиду перепишите весь метод либо так

public void removeTail() {
    snakeParts.remove(snakeParts.size() - 1);
}

либо так

public void removeTail() {
    Iterator<GameObject> snakeIterator = snakeParts.iterator();
    while (true) {
        if (snakeIterator.hasNext()) {
            snakeIterator.next();
        } else {
            snakeIterator.remove();
            break;
        }
    }
}

Anika

Уровень 22

12 февраля 2019, 23:26

так как раз работает, спасибо, но не пропускает, говорит : " Проверь, что после вызова метода removeTail() размер змейки уменьшается на 1."

Denys

Уровень 34

12 февраля 2019, 23:35

решение

На данном этапе, оба метода дают правильный результат. Метод createNewHead() правильный. Скиньте пожалуйста исходный код (можете скинуть лишь класс Snake) который вы уже изменили. Если ваши методы выглядят следующим образом:

public void removeTail() {
    snakeParts.remove(snakeParts.size() - 1);
}

public GameObject createNewHead() {
    GameObject gameObject = null;
    if(direction == Direction.UP)
        gameObject =new GameObject(snakeParts.get(0).x, snakeParts.get(0).y - 1);
    if(direction == Direction.DOWN)
        gameObject = new GameObject(snakeParts.get(0).x, snakeParts.get(0).y + 1);
    if(direction == Direction.RIGHT)
        gameObject = new GameObject(snakeParts.get(0).x+1, snakeParts.get(0).y);
    if(direction == Direction.LEFT)
        gameObject = new GameObject(snakeParts.get(0).x-1, snakeParts.get(0).y );
    return gameObject;
}

то должно проходить проверку однозначно.

Anika

Уровень 22

12 февраля 2019, 23:54

все валидуется,спасибо большое за помощь ♥ ♥ ♥

Denys

Уровень 34

12 февраля 2019, 23:57

Ну вот ;) пожалуйста. Успехов в обучении!)