Подскажите, почему не проходит проверку?

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

Решен

Вбиваю числа от 1 до 20, запускаю - всё правильно, выводит с 11 по 20. Даже специально выводил и первый маленький массив, и второй, выводит с 1 до 10 и с 11 до 20. А проверку не проходит...

1. Создать массив на 20 чисел.
2. Ввести в него значения с клавиатуры.
3. Создать два массива на 10 чисел каждый.
4. Скопировать большой массив в два маленьких: половину чисел в первый маленький, вторую половину во второй маленький.
5. Вывести второй маленький массив на экран, каждое значение выводить с новой строки.

Требования:

Программа должна создавать большой массив на 20 целых чисел.
Программа должна считывать с клавиатуры 20 чисел для большого массива.
Программа должна создавать два маленьких массива на 10 чисел каждый.
Программа должна скопировать одну половину большого массива в первый маленький, а вторую - во второй и вывести второй маленький массив на экран.

package com.javarush.task.task07.task0705; import java.io.BufferedReader; import java.io.InputStreamReader; import java.util.Arrays; /* Один большой массив и два маленьких */ public class Solution { public static void main(String[] args) throws Exception { //напишите тут ваш код BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); int[] listBig = new int[20]; for (int i = 0; i < listBig.length; i++) listBig[i] = Integer.parseInt(reader.readLine()); int[] list1 = Arrays.copyOf(listBig, 10); int[] list2 = Arrays.copyOfRange(listBig, 10, listBig.length); /*System.out.println("первый массив"); for(int k = 0; k<list2.length; k++) System.out.println(list1[k]); System.out.println("второй массив");*/ for(int j = 0; j<list2.length; j++) System.out.println(list2[j]); } }

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

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

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

Галкин Юрий

Уровень 41

26 августа 2021, 10:45

Потестировал ваше решение. Все ок. Пишите администрации.

public class App {

    public static void main(String[] args) throws Exception {
        boolean testGoesWell = true;
        for (int i = 0; i < 100 && testGoesWell; i++) {
            int[] listBig = listBig();
            int[][] littleArrays = process(listBig);

            final boolean containsFirstArray = arrayContainsSubarray(littleArrays[0], 0, listBig);
            final boolean containsSecondArray = arrayContainsSubarray(littleArrays[1], 10, listBig);
            testGoesWell = containsFirstArray && containsSecondArray;

            System.out.println("big array contains first  array: " + containsFirstArray);
            System.out.println("big array contains second array: " + containsSecondArray);
            System.out.println("End of test " + i);
        }

        System.out.println("TEST RESULT: " + (testGoesWell ? "PASSED" : "FAILED"));

    }

    private static boolean arrayContainsSubarray(int[] subarray, int offset, int[] array) {
        boolean isContaining = true;
        for (int i = 0; i < 10 && isContaining; i++) {
            isContaining = subarray[i] == array[i + offset];
        }
        return isContaining;
    }

    private static int[][] process(int[] listBig) {
        int[] list1 = Arrays.copyOf(listBig, 10);
        int[] list2 = Arrays.copyOfRange(listBig, 10, listBig.length);
        int[][] littleArrays = new int[][]{list1, list2};

        for (int j = 0; j < list2.length; j++) {
            System.out.println(list2[j]);
        }
        return littleArrays;
    }

    private static int[] listBig() {
        final int[] listBig = new int[20];
        final Random random = new Random();

        for (int i = 0; i < 20; i++) {
            listBig[i] = random.nextInt();
        }
        return listBig;
    }
}

Александр Горохов

Уровень 28

26 августа 2021, 10:53

Ого, спасибо! Хоть я ничего почти не понял из кода:)

Галкин Юрий

Уровень 41

26 августа 2021, 12:17

Можете запустить. Это баловство. Я тут насоздавал входящих больших массивов (вместо чтения из файла), и использовал ваш код (метод process) для разбиения на подмассивы. Ну и дальше сравнил, содержаться ли малые массивы один за другим в изначальном большом. И всё. Возможно есть случаи, которые я не рассмотрел. Тогда предлагаю дописать тест.

Ирина Алымова Java Developer в Nerdysoft

26 августа 2021, 10:20

У тебя программа работает правильно, но думаю валидатор проверяет что маленькие массивы будут размером в 10 элементов вне зависимости от размера большого массива. Попробуй инициализировать размер массивов, и только потом скопировать туда данные.

Александр Горохов

Уровень 28

26 августа 2021, 10:25

Спасибо, видимо, так и есть

Анна Яковленко

Уровень 21

26 августа 2021, 10:05

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

Александр Горохов

Уровень 28

26 августа 2021, 10:24

возможно, Вы правы. получилось так:

int[] list1 = new int[10];
int[] list2 = new int[10];

        System.arraycopy(listBig, 0, list1, 0, 10);
        System.arraycopy(listBig, 10, list2, 0, 10);

        for(int j = 0; j<list2.length; j++)
            System.out.println(list2[j]);

Гофф

Уровень 20

Master

26 августа 2021, 10:04

А какой длины у тебя list2[] ? Ну и логическая неувязочка. Ты. с одной стороны, используешь в коде listBig.length, а не число 20. Но при делении массива используешь именно число 10, а не listBig.length / 2. Уж либо так, либо этак.

Александр Горохов

Уровень 28

26 августа 2021, 10:12

lits2.length получается 10. Попробовал изменить на: int[] list1 = Arrays.copyOf(listBig, listBig.length/2); int[] list2 = Arrays.copyOfRange(listBig, listBig.length/2, listBig.length); Результат такой же - условие "Программа должна создавать два маленьких массива на 10 чисел каждый." не выполнено.