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

16 уровень

18.03.2022
449просмотров
10комментариев

Код работает правильно, но не проходит проверку.

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

Решен

Предыдущий товарищ столкнулся с такой же проблемой, и ему подсказали про локальную переменную, которую создавать не надо. Но я же, вроде, не создаю новую переменную, а прописываю параметр массива, как написано в лекции... Потом я перенес объявление массива в main, опять не прошел проверку, но уже с другим советом от ментора. Пожалуйста, помогите разобраться.

Создай треугольный массив, где значение каждого элемента — это сумма его индексов. Например:
array[7][3] = 7 + 3 = 10, array[3][0] = 3 + 0 = 3.

Выведи массив на экран в следующем виде:
0 1 2 2 3 4 3 4 5 6 4 5 6 7 8 5 6 7 8 9 10 ...
Числа в строке разделены пробелом.

Задать треугольный массив можно следующим образом:
int[][] array = new int[10][]; array[0] = new int[1]; array[1] = new int[2]; array[2] = new int[3]; ...

Требования:

В методе main(String[]) двумерный массив result заполни числами согласно условию.
Выведенный текст должен содержать 10 строк.
Выведенные числа должны соответствовать условию.

package com.javarush.task.pro.task05.task0510; /* Треугольный массив */ public class Solution { //public static int[][] result = new int[10][]; public static void main(String[] args) { int[][] result = new int[10][]; for (int y = 0; y < 10; y ++){ for (int x = 0; x < (y + 1); x ++){ result[y] = new int [x+1]; result[y][x] = (x + y); System.out.print(result[y][x] + " "); } System.out.println(); } } }

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

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

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

Владимир

Уровень 30

18 марта 2022, 17:44

Евгений, товарищ vk правильно говорит об ошибке. Просто он великодушно оставляет Вам возможность решить самому ;) По поводу "проверки по двум пунктам" - это конечно любопытно (вопрос к валидатору JR). НО ваш код все же не верный и вы в этом можете легко убедиться, проверив/написав после него:

// показывает содержимое двумерного массива int [][] result
for (int[] array : result) {
            System.out.println(Arrays.toString(array));
        }

Т.о. вывод №1: то, что мы видим снаружи/в консоли не всегда прямо соответствует тому, что внутри. Обратите внимание на строку во внутреннем цикле:

result[y] = new int [x+1];

Каждый новый проход от 0 до x < (y + 1) заново пересоздает один и тот же массив и в итоге остается тот, который последний! С заполненным последним элементом! Ответьте себе, что значит "10" в:

=new int[10][];

И для чего нужен внешний цикл? P.S. Если решение все же не найдете, напишите в лс ;)

Евгенийрежиссер в киностудия Лисоцвет

19 марта 2022, 09:59

Владимир, спасибо большое за время, которое Вы тратите на мой вопрос. Пользуясь Вашим любезным предложением, прошу, разъясните мне три момента: 1. result[y] = new int [x+1]; Я делаю так, как описано в лекции, т.е. формирую длину строки y массива result [y][x] (6 уровень, 6 лекция) 2. result[y][x] = (x + y); Я формирую каждый конкретный элемент массива, согласно условию. Получается, что алгоритм правильный... 3. Таким образом получается, что и массив result инициализируется в main, ведь это всё в main происходит... Объясните, пожалуйста, где ошибки в моих рассуждениях. Буду Вам чрезвычайно признателен.

Владимир

Уровень 30

19 марта 2022, 14:06

решение

3. Массив int[][] result был создан в классе, к которому принадлежит main (). Поэтому пересоздавать его в main () не нужно! Остальное все происходит в main () - это бесспорно.

1. result[y] = new int [x+1]; //"Формирую длину строки"

Точнее создаете массив [x] в ячейке [y].

2. result[y][x] = (x + y); //"Формирую каждый конкретный элемент"

НЕ ВСЕ элементы, а только один последний! Потому что в следующей итерации этого цикла вы заново пересоздаете массив [x] в ячейке [y] и т.д... Пока в конце не выйдите из внутреннего цикла (в итоге в элементе [y] останется последний созданный массив с последним заполненным элементом). Далее во внешнем цикле повысится счетчик и вы опять во внутреннем будете пересоздавать кучу массивов и заполнять только один элемент! Вероятно вас сбивает с толку System.out.print(result[y][x] + " "). Так он только показывает промежуточную стадию. В следующей итерации "х" вы создаете новый массив на том же месте "y", старый теряется безвозвратно. Двумерный массив [][], в 1й паре скобок указывается количество одномерных массивов, во 2й - длина каждого одномерного массива. Внешний цикл служит для того, чтобы проходить по массивам, внутренний - чтобы проходить по элементам каждого массива. Таким образом логично создавать массивы во внешнем цикле, а инициализировать/заполнять каждый массив во внутреннем:

for (int i = 0; i < result.length; i++) {
           result[i] = new int[i + 1];
           for (int j = 0; j < result[i].length; j++) {
               result[i][j] = i + j;

Заметьте, как обозначены границы "условной части". Это более гибкий/универсальный способ, чем просто задавать жесткие значения.

Евгенийрежиссер в киностудия Лисоцвет

19 марта 2022, 14:37

Алиллуйя!!!!! Я разобрался. Я не там создавал "внешний массив"!!! Я всё понял!!!! Нужно было создать строку во внешнем цикле (это и есть инициализация массива в методе мэйн), а затем создавать элементы этой строки во внутреннем цикле. Огромное Вам спасибо, Владимир!

Владимир

Уровень 30

19 марта 2022, 15:08

Успехов ;) P.S. Проверяйте в конце то, что написали. И не забывайте о такой полезной штуке, как Debugger

hidden #598481

Уровень 27

18 марта 2022, 07:15

1. Если переменная public static int[][] result была дана изначально, то нужно использовать её. Только инициализировать её нужно внутри метода main. 2. То, что вывод на экран соответствует желаемому, ещё не означает, что массив заполнен правильно. Выведи его на экран ещё раз после завершения работы цикла:

for (int i = 0; i < result.length; i++) {
    for (int j = 0; j < result[i].length; j++) {
        System.out.print(result[i][j] + " ");
    }
    System.out.println();
}

И я думаю, ты удивишься содержимому массива. Потому что в алгоритме тоже есть ошибка, которую ты не замечаешь.

Евгенийрежиссер в киностудия Лисоцвет

18 марта 2022, 09:03

Спасибо большое за ответ. Вы привели фрагмент не моего кода. Мой код проходит проверку по двум условиям, т.е. алгоритм правильный. Непонятно только, как объявлять структуру массива. Как и в каком месте.

hidden #598481

Уровень 27

18 марта 2022, 11:17

решение

> Непонятно только, как объявлять структуру массива. Как и в каком месте.

public static int[][] result; // это объявление переменной
result = new int[10][]; // это инициализация

инициализация должна быть в мейне. Ваш код работает неправильно. Он выводит то, что вы хотите увидеть, но при этом содержимое массива отличается от того, что видно в консоли. > Вы привели фрагмент не моего кода. Я этого и не говорил ) Я предложил вам добавить этот вывод на экран в свой метод мейн, чтобы ещё раз вывести на экран содержимое массива и своими глазами убедиться, что массив заполнен неправильно. Потом перед проверкой этот второй цикл нужно убрать, разумеется. Просто это поможет найти вторую ошибку, не связанную с объявлением массива result.

Евгенийрежиссер в киностудия Лисоцвет

19 марта 2022, 07:17

Спасибо большое, сейчас попробую.

Евгенийрежиссер в киностудия Лисоцвет

19 марта 2022, 14:39

Спасибо большое, я понял свою ошибку.