Добрый день, подскажите пожалуйста что не так?

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

2048 (6)

Java Multithreading, 10 уровень, 15 лекция

Архивный

Добрый день, вроде все условия выполняются, подскажите пожалуйста где ошибка в логике или подкиньте пример для проверки.

Итак, ты реализовал сжатие и слияние плиток, что в комбинации дает нам возможность осуществить движение влево.
Отлично! Но нам нужно еще и добавлять новую плитку в случае, если после передвижения игровое поле изменилось.

Давай сделаем так:
1. Изменим метод compressTiles, чтобы он возвращал true в случае, если он вносил изменения во входящий массив, иначе - false.
2. То же самое сделаем и для метода mergeTiles.
3. Реализуем метод left, который будет для каждой строки массива gameTiles вызывать методы compressTiles и mergeTiles и добавлять одну плитку с помощью метода addTile в том случае, если это необходимо.
4. Метод left не должен быть приватным, т.к. вызваться он будет, помимо прочего, из класса Controller.

Требования:

Метод compressTiles должен возвращать true в случае, если им были внесены изменения во входящий массив.
Метод compressTiles должен возвращать false в случае, если им НЕ были внесены изменения во входящий массив.
Метод mergeTiles должен возвращать true в случае, если им были внесены изменения во входящий массив.
Метод mergeTiles должен возвращать false в случае, если им НЕ были внесены изменения во входящий массив.
Метод left должен перемещать элементы массива gameTiles влево в соответствии с правилами игры и добавлять плитку с помощью метода addTile, если это необходимо.
Метод left НЕ должен изменять массив gameTiles, если перемещение влево невозможно.

package com.javarush.task.task35.task3513; import java.util.Arrays; public class Main { public static void main(String[] args) { } }

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

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

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

ApplejackНастраиваю телевизор в кругу друзей

26 сентября 2020, 07:42

решение

Привет. Метод left() у тебя правильный. Проверь методы compress и merge. Я сильно не всматривался (особенно в merge), но, по-моему, у тебя compress возвращает неправильный boolean. В частности посмотри на вторую строку compress();

hidden #2322530

Уровень 41

21 сентября 2020, 13:09

при каких условиях надо добавлять плитку? если mergeTiles = true если compressTiles = true и getEmptyTiles.size > 0

Eidalon

Уровень 39

21 сентября 2020, 15:51

Добрый день, спасибо за ответ. Если я правильно понимаю то если mergeTiles или compressTiles хоть один раз дало true и getEmptyTiles.size > 0 - тогда добавляем addTile. Ниже мой код, Если я все правильно понял, то это не сработало.

boolean flag = false;
           for (int i = 0; i < FIELD_WIDTH; i++) {
               if (compressTiles(gameTiles[i]) | mergeTiles(gameTiles[i]))
                   flag = true;
           }
           if (flag && getEmptyTiles().size() > 0 )
               addTile();

hidden #2322530

Уровень 41

21 сентября 2020, 15:58

потому что ты проверяешь вначале все 4 строки, а добавляешь только одну плитку) надо в каждом проходе от gameTiles.length , проверять те условия что я написал, и добавлять плитку. если во всех 4х строках будет true везде, значит надо добавить 4 плитки, а ты добавляешь одну

-2

Eidalon

Уровень 39

22 сентября 2020, 14:32

Добрый день, я специально скачал игру, да и коменты тут почитал, надо добавлять только одну плитку, если можно добавить. Так что дело не в этом

hidden #2322530

Уровень 41

22 сентября 2020, 14:40

у меня добавляется для каждой строки

for (int i = 0; i < gameTiles.length; i++) {
            boolean compress = compressTiles(gameTiles[i]);
            boolean merge = mergeTiles(gameTiles[i]);
            if (compress && merge && getEmptyTiles().size() > 0) {
                addTile();
            }
        }

-2