Не проходит по последним трём пунктам

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

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; public class Main { public static void main(String[] args) { } }

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

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

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

Влад

Уровень 35

9 апреля 2019, 13:44

полезный

а, ещё инициализацию полей score=0 u maxTile=2 в метод resetGameTiles переместил, надеюсь кому-нибудь это поможет.

Влад

Уровень 35

9 апреля 2019, 13:40

решение

В последствии ряда изменений мой метод mergeTiles стал выглядеть так:

boolean changed = false;
for (int i = 0; i < tiles.length - 1; i++) {
            if (tiles[i].isEmpty()) break;
            for (int j = tiles.length - 1; j > i; j--) {
                if (tiles[i].value == tiles[j].value) {
                    changed = true;
                    tiles[i].value <<= 1;
                    tiles[j].value = 0;
                    if (j < tiles.length - 1) {
                        for (int l = j; l < tiles.length - 1; l++) {
                            tiles[l].value = tiles[l + 1].value;
                        }
                    }
                    if (tiles[i].value > maxTile) maxTile = tiles[i].value;
                    score += tiles[i].value;
                    break;
                }
            }
        }
return changed;

Своровал неверную реализацию с комментов - валидатор пропустил, при том что при подаче в метод ряда {2, 4, 2, 0} он его вообще не меняет. Собственно, метод:

boolean changed = false;
for (int i = 0; i < tiles.length; i++) {
            try {
                if (tiles[i].value > 0 && tiles[i].value == tiles[i + 1].value) {

                    tiles[i].value += tiles[i + 1].value;

                    if (tiles[i].value > maxTile) {
                        maxTile = tiles[i].value;
                    }
                    score += tiles[i].value;
                    changed = true;

                    for (int j = i + 1; j < tiles.length; j++) {
                        if (j != tiles.length - 1) {
                            tiles[j].value = tiles[j + 1].value;
                        } else tiles[j].value = 0;
                    }
                }
            }catch (ArrayIndexOutOfBoundsException ex) {
                //NOP
            }
        }
return changed

Дмитрий

Уровень 35

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

полезный

Ряд {2, 4, 2, 0} и не должен быть изменен, все верно.

Влад

Уровень 35

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

да, это я правила не до конца изучил, неверным был мой метод на самом деле