Третий пункт. Поле passedCarsCount увеличивается по условиям, но валидатор не принимает

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

Обсуждается

Коллеги, почему срабатывает валидатор? Проблема явно не в методе deletePassedItems, а в чем-то еще. Только я не понимаю, чего не так

private void deletePassedItems() {
    Iterator iterator = items.iterator();
    while(iterator.hasNext()){
        RoadObject item = (RoadObject)iterator.next();
        if(item.y >= RacerGame.HEIGHT){
            if (item.type != RoadObjectType.THORN) {
                passedCarsCount++;
            }
            iterator.remove();
        }
    }
}

Чтобы проверять выполнение условий победы, нам нужно будет знать реальное количество машин, с которыми разминулся игрок.
Для этого создай поле passedCarsCount в классе RoadManager и геттер к нему.
Значение этого поля должно увеличиваться на единицу после прохождения каждой машины-препятствия (удаления машины из списка items).

Сравнивать passedCarsCount и RACE_GOAL_CARS_COUNT будем в методе onTurn(int).
Если значение passedCarsCount больше или равно RACE_GOAL_CARS_COUNT, значит цель достигнута и нужно показать финишную линию.

Требования:

В классе RoadManager должно существовать приватное поле passedCarsCount типа int, инициализированное при объявлении значением 0.
В классе RoadManager должен существовать публичный геттер int getPassedCarsCount(), который возвращает значение поля passedCarsCount.
В методе deletePassedItems() класса RoadManager необходимо увеличивать значение поля passedCarsCount на единицу, если из списка items удаляется объект и тип этого объекта — не шипы.
В методе onTurn(int) класса RacerGame необходимо вызвать у объекта finishLine метод show(), если значение, которое вернул метод roadManager.getPassedCarsCount(), больше либо равно RACE_GOAL_CARS_COUNT.
В методе onTurn(int) класса RacerGame метод show() объекта finishLine должен вызываться до moveAll().

package com.javarush.games.racer; public enum Direction { LEFT, RIGHT, NONE }

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

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

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

hidden #2297125

Уровень 18

27 января 2021, 14:51

у меня пропустило только после того как я использовала для сравнения шип это или не шип instanceof

Валерий Емельянов

Уровень 41

19 декабря 2019, 17:49

ну-у, если уж танцы с бубнами, то попробуй поменять местами


                if (item.type != RoadObjectType.THORN) {
                    passedCarsCount++;
                }


                iterator.remove();

в смысле - сначала удалить, потом посчитать (четко по условию)

Валерий Емельянов

Уровень 41

17 декабря 2019, 18:44

Попробуй проверку roadManager.getPassedCarsCount() >= RACE_GOAL_CARS_COUNT вынести в начало метода

@Override
    public void onTurn(int step) {
        if (roadManager.getPassedCarsCount() >= RACE_GOAL_CARS_COUNT) {
            finishLine.show();
        }

        if (roadManager.checkCrush(player)) {
            gameOver();
        } else {
                moveAll();
                roadManager.generateNewRoadObjects(this);
        }
        drawScene();
    }

hidden #2163154

Уровень 22

18 декабря 2019, 16:11

хорошая попытка, но - нет, не помогло :)

Александр Кононенко

Уровень 41

16 декабря 2019, 17:52

может выделываться потому что passedCarsCount не инициализировано 0

hidden #2163154

Уровень 22

17 декабря 2019, 16:33

оно инициализировано сейчас и не проходит все равно

Александр Кононенко

Уровень 41

16 декабря 2019, 16:12

от валидатора можно ждать что угодно... я сделал то же самое только через for{} у меня валидатор принял решение

hidden #2163154

Уровень 22

16 декабря 2019, 17:11

не в этом коде дело, а в чем-то вообще другом. Этот код я раза четыре переписывал по-разному - результат один. Оно пытается о чем-то еще сказать, только я не понимаю.