JavaRush/Помощь по Java задачам/Валидатор не принимает, хотя всё корректно работает.

17 уровень

09.04.2019
869просмотров
20комментариев

Валидатор не принимает, хотя всё корректно работает.

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

Решен

Пробовал и через if-else. Не принимает ни тот, ни другой вариант. Хотя при запуске всё работает корректно. Но валидатор не принимает.

Продолжим реализацию движения ракеты. Она должна двигаться и по вертикали, и по горизонтали.
Давай внесем соответствующие изменения в метод move: теперь он будет принимать в качестве параметров состояния нажатия клавиш. Первый параметр — это isUpPressed, второй — isLeftPressed, третий — isRightPressed.
Для установки скорости движения по оси X создадим переменную speedX. Значение этой переменной, как и speedY, будет изменяться в методе move в зависимости от нажатых клавиш. Также не забудь об изменении самих координат(x и y).

Требования:

В классе Rocket должно существовать приватное поле speedX типа double, проинициализированное при объявлении значением 0.
В классе Rocket должен существовать публичный метод void move(boolean isUpPressed, boolean isLeftPressed, boolean isRightPressed).
В классе Rocket не должен существовать публичный метод void move().
В методе move(boolean, boolean, boolean) значение поля speedY должно уменьшаться на значение boost, если параметр метода isUpPressed равен true.
В методе move(boolean, boolean, boolean) значение поля speedY должно увеличиваться на значение boost, если параметр метода isUpPressed равен false.
В методе move(boolean, boolean, boolean) значение поля y должно быть увеличено на значение speedY после того, как поле speedY было изменено.
В методе move(boolean, boolean, boolean) значение поля speedX должно уменьшаться на значение boost, если параметр метода isLeftPressed равен true.
В методе move(boolean, boolean, boolean) значение поля speedX должно увеличиваться на значение boost, если параметр метода isRightPressed равен true.
В методе move(boolean, boolean, boolean) значение поля x должно быть увеличено на значение speedX после того, как поле speedX было изменено.
В методе onTurn(int) класса MoonLanderGame у объекта rocket должен быть вызван метод move(boolean, boolean, boolean) с параметрами: isUpPressed, isLeftPressed, isRightPressed.

package com.javarush.games.moonlander; public class Rocket extends GameObject { private double speedY = 0; private double boost = 0.05; private double speedX = 0; public Rocket(double x, double y) { super(x, y, ShapeMatrix.ROCKET); } public void move(boolean isUpPressed, boolean isLeftPressed, boolean isRightPressed){ if (isUpPressed) speedY -= boost; else speedY += boost; y += speedY; if (isLeftPressed) speedX -= boost; if (isRightPressed) speedX += boost; x = x + speedX; } }

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

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

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

Wrell

Уровень 40

20 июня 2019, 12:08

Такое решение проходит, хотя ракета начинает управляться... странно.

public void move(boolean isUrPressed, boolean isLeftPressed, boolean isRightPressed) {
    speedY += isUrPressed ? -boost : boost;
    y += speedY;
    speedX += isLeftPressed || isRightPressed ? (isLeftPressed ? -boost : boost) : 0;
    x += isLeftPressed || isRightPressed ? speedX : 0;
}

Николай

Уровень 36

Expert

9 апреля 2019, 19:02

решение

Возможно ответ кроется в этом: "значение поля x должно быть увеличено на значение speedX после того, как поле speedX было изменено" Судя по вашему коду - вы и "x" меняете при каждом вызове метода move. А нужно проверять на выполнение условий нажатия кнопок.

Алексей

Уровень 17

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

поле Х должно меняться при каждом вызове move(), т.к. скорость ракеты по горизонтали постоянна, при нажатии кнопок она увеличивается или уменьшается. Я так понял правила игры.

Николай

Уровень 36

Expert

9 апреля 2019, 19:25

решение

Вы можете считать все, что угодно, но я лишь смотрю задание и ваш код. Вот логика: 1. значение поля speedX должно уменьшаться на значение boost, если параметр метода isLeftPressed равен true 2. значение поля speedX должно увеличиваться на значение boost, если параметр метода isRightPressed равен true 3. значение поля x должно быть увеличено на значение speedX после того, как поле speedX было изменено

Алексей

Уровень 17

9 апреля 2019, 19:51

Спасибо. Помогло. Мне казалось, что ракета должна иметь постоянную скорость по оси Х.

VIKTOR NEZHELSKIY

Уровень 27

Expert

9 апреля 2019, 18:01

Попробуй

if (isLeftPressed) {
    speedX -= boost;
    x += speedX;
} else if (isRightPressed) {
    speedX += boost;
    x += speedX;
} else {
    speedX = 0;
}
x += speedX;

Алексей

Уровень 17

9 апреля 2019, 18:04

В методе move(boolean, boolean, boolean) значение поля speedX должно уменьшаться на значение boost, если параметр метода isLeftPressed равен true. Убедись, что в методе move поле speedX не уменьшается, если параметр метода isLeftPressed равен false. Добавилась еще и такая ошибка.

VIKTOR NEZHELSKIY

Уровень 27

Expert

9 апреля 2019, 18:12

Может ты неправильно вставил этот фрагмент метода в свой метод? Дай полный текст твоего метода move(...) на данном этапе.

Алексей

Уровень 17

9 апреля 2019, 18:20

public void move(boolean isUpPressed, boolean isLeftPressed, boolean isRightPressed){
    if (isUpPressed) speedY -= boost;
    else speedY += boost;
    y += speedY;
    speedX += (isLeftPressed ? -boost : 0);
    speedX += (isRightPressed ? boost : 0);
    x += speedX;
}

Алексей

Уровень 17

9 апреля 2019, 18:21

public void move(boolean isUpPressed, boolean isLeftPressed, boolean isRightPressed){
        if (isUpPressed) speedY -= boost;
        else speedY += boost;
        y += speedY;
//        speedX += (isLeftPressed ? -boost : 0);
//        speedX += (isRightPressed ? boost : 0);
//        x += speedX;
        if (isLeftPressed) {
            speedX -= boost;
            x += speedX;
        } else if (isRightPressed) {
            speedX += boost;
            x += speedX;
        } else {
            speedX = 0;
        }
        x += speedX;
    }

Алексей

Уровень 17

9 апреля 2019, 18:22

public void move(boolean isUpPressed, boolean isLeftPressed, boolean isRightPressed){
    if (isUpPressed) speedY -= boost;
    else speedY += boost;
    y += speedY;
    if (isLeftPressed) {
        speedX -= boost;
        x += speedX;
    } else if (isRightPressed) {
        speedX += boost;
        x += speedX;
    }
    x += speedX;
}

Алексей

Уровень 17

9 апреля 2019, 18:22

ни один не проходит. А второй создает вторую ошибку валидатора. Скорость не должна становиться нулевой.

Алексей

Уровень 17

9 апреля 2019, 18:34

Поделитесь своим вариантом, я лишнее вырежу.

VIKTOR NEZHELSKIY

Уровень 27

Expert

9 апреля 2019, 19:34

public void move(boolean isUpPressed, boolean isLeftPressed, boolean isRightPressed) {
    if (isUpPressed) {
        speedY -= boost;
    } else {
        speedY += boost;
    }
    y += speedY;

    if (isLeftPressed) {
        speedX -= boost;
        x += speedX;
    } else if (isRightPressed) {
        speedX += boost;
        x += speedX;
    } else if (speedX > slowdown) {
        speedX -= slowdown;
    } else if (speedX < -slowdown) {
        speedX += slowdown;
    } else {
        speedX = 0;
    }
    x += speedX;
    checkBorders();
    switchFire(isUpPressed, isLeftPressed, isRightPressed);
}

VIKTOR NEZHELSKIY

Уровень 27

Expert

9 апреля 2019, 17:48

Попробуй просто заменить x = x + speedX; на x += speedX; То что принял валидатор лучше не меняй.

Алексей

Уровень 17

9 апреля 2019, 17:53

Спасибо за совет, но и его валидатор не принимает. Уже вроде все варианты попробовал. Второй день на месте топчусь.

Алексей

Уровень 17

9 апреля 2019, 17:58

а как у вас этот метод написан?

VIKTOR NEZHELSKIY

Уровень 27

Expert

9 апреля 2019, 18:16

У меня нет этого метода на том шаге, где ты сейчас. На следующих шагах надо будет добавлять в метод дополнительные действия, которые тебе сейчас, что бы пройти валидатор, не нужны.

Алексей

Уровень 17

9 апреля 2019, 18:29

а именно эта часть как написана?

Алексей

Уровень 17

9 апреля 2019, 17:42

пробовал такой вариант, тот же эффект. Может ошибка валидатора?

speedX = speedX + (isLeftPressed ? -boost : 0) + (isRightPressed ? boost : 0);
x = x + speedX;