1

Дан на вход программы двумерный массив. Значения из него нужно записать в одномерный массив, по массиву пройтись "змейкой" и вывести на экран.

В моем коде выдает ошибку переполнения массива:

import java.util.Arrays;

public class SnakeArray {


    public static void main(String[] args) {
        SnakeArray snakeArray = new SnakeArray();
        snakeArray.toSnakeArray(new int[][]{{1, 2}, {4, 5}});
    }

    private void toSnakeArray(int[][] largeArray) {

        String[] smallArray = new String[largeArray[0].length + largeArray[1].length];

        for (int i = 0; i < smallArray.length; i++) {
            for (int j=0;j<smallArray.length;j++) {
            smallArray[i] = String.valueOf(largeArray[i][j]);
            }
            }

            System.out.println(Arrays.toString(smallArray));
        }
    }


  Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: Index 2 out of bounds for length 2 
SnakeArray.toSnakeArray(SnakeArray.java:20) 
SnakeArray.main(SnakeArray.java:11)
9
  • Приведите точный текст ошибки
    – Kromster
    4 мая 2020 в 9:51
  • Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: Index 2 out of bounds for length 2 SnakeArray.toSnakeArray(SnakeArray.java:20) SnakeArray.main(SnakeArray.java:11)
    – A.Roy
    4 мая 2020 в 9:53
  • Под вопросом кнопка "править", используйте ее
    – Kromster
    4 мая 2020 в 9:55
  • вы идёте по длинне smallArray в цикле а не по largeArray 4 мая 2020 в 9:57
  • да, но на вход в качестве размерности я передаю размерность большого массива
    – A.Roy
    4 мая 2020 в 9:58

1 ответ 1

1

Старайтесь писать более универсальные методы. Что если массив состоит не из интов или матрица лохматая? Хотелось бы не переписывать код каждый раз. Кроме того, массивы, безусловно, имеют свои преимущества, однако в большинстве случаев использовать коллекции на много проще, особенно с учетом того, что ArrayList в себе по сути инкапсулирует этот самый массив, соответственно, получит массив в нужный момент не составит труда. И последнее... Зачастую проще использовать циклы без индексов либо стримы просто потому, что такой код читается проще и меньше шансов допустить ошибку.

import java.util.ArrayList;
import java.util.Arrays;
import java.util.List;

public class SnakeArray {

    public static void main(String[] args) {
        SnakeArray snakeArray = new SnakeArray();
        List<Integer> result = snakeArray.toSnakeArray(new Integer[][]{{1, 2}, {4, 5}});
        System.out.println(Arrays.toString(result.toArray()));
    }

    private <T> List<T> toSnakeArray(T[][] array) {
        final List<T> result = new ArrayList<>();
        for (T[] arrays : array) result.addAll(Arrays.asList(arrays));
        return result;
    }

}
2
  • Спасибо за совет. Но здесь задание конкретное.
    – A.Roy
    4 мая 2020 в 17:52
  • @Delembech вам абстрактного задания никто никогда и не поставит. задание всегда будет конкретное, а вывести из этого абстракции - и есть ваша задача. в противном случае на выходе процедурная реализация , жуткое дублирование кода дублирование кода и т.д. по списку 4 мая 2020 в 18:07

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.