В чём отличие двух разных способ обхода двумерного массива?

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

2048 (4)

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

Архивный

Подскажите пожалуйста! Не могу понять почему в классе Model, обход массива в не закоментированном коде работает нормально, а реализация, как в закоментированом коде не работает(заполняет, весь массив ссылками на null). //Рабочий код public Model() { this.gameTiles = new Tile[FIELD_WIDTH][FIELD_WIDTH]; for (int i = 0; i < gameTiles.length; i++){ for (int j = 0; j < gameTiles[i].length; j++){ gameTiles[i][j] = new Tile(); } } } // Не рабочий код // public Model() { // this.gameTiles = new Tile[FIELD_WIDTH][FIELD_WIDTH]; // for (Tile[] t :gameTiles){ // for (Tile tile: t){ // tile = new Tile(); // } // } // }

Игра 2048 начинается на поле, где две плитки уже имеют какие-то начальные значения. А наше поле пока пусто :(.

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

Предлагаю создать приватный метод addTile, который будет смотреть какие плитки пустуют и, если такие имеются,

менять вес одной из них, выбранной случайным образом, на 2 или 4 (на 9 двоек должна приходиться 1 четверка).

Получить случайный объект из списка можешь использовав следующее выражение:

(размерСписка * случайноеЧислоОтНуляДоЕдиницы).

Также получение свободных плиток можно вынести в отдельный приватный метод getEmptyTiles, возвращающий список

свободных плиток в массиве gameTiles.

После реализации функционала добавления новых плиток, добавим в конструктор два вызова метода addTile,

выполняя начальное условие задачи.

P.S. Пожалуй стоит весь код из конструктора переместить в метод resetGameTiles, для того, чтобы при необходимости

начать новую игру, не приходилось создавать новую модель, а можно было бы просто вернуться в начальное состояние

вызвав его. Уровень доступа должен быть шире приватного.

P.P.S. Для вычисления веса новой плитки используй выражение (Math.random() < 0.9 ? 2 : 4).

Требования:

Метод getEmptyTiles должен возвращать список пустых плиток в массиве gameTiles.
Метод addTile должен изменять значение случайной пустой плитки в массиве gameTiles на 2 или 4 с вероятностью 0.9 и 0.1 соответственно.
Метод resetGameTiles должен заполнять массив gameTiles новыми плитками и менять значение двух из них с помощью двух вызовов метода addTile.
В конструкторе класса Model должен содержаться вызов метода resetGameTiles.

package com.javarush.task.task35.task3513; import java.util.Arrays; //- будет содержать только метод main и служить точкой входа в наше приложение. public class Main { public static void main(String[] args) { // new Model().test(); } }

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

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

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

Андрей

Уровень 39

12 августа 2019, 14:53

Понял. Спасибо. Но есть тогда второй вопрос. Как тогда через цикл for each заполнить массив, или с его помощью можно только его прочитать?

HellLena

Уровень 41

12 августа 2019, 14:42

Потому что во втором (закомментированном случае) tile - это локальная переменная цикла. Это то же самое, если написать вот так:

for (int i = 0; i < gameTiles.length; i++){
            for (int j = 0; j < gameTiles[i].length; j++){
                Tile tile = gameTiles[i][j];
                tile = new Tile();
            }
        }

Соответственно, присваивание этой переменной какого-либо значения не сохраняется в массиве. Массив у вас изначально заполнен значениями null ещё во время инициализации: this.gameTiles = new Tile[FIELD_WIDTH][FIELD_WIDTH];