JavaRush/Помощь по Java задачам/Почему решение не принимается?

20 уровень

14.11.2020
552просмотров
10комментариев

Почему решение не принимается?

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

Решен

Я пробовал с фиксированной длиной массивов, потом даже оптимизировал под изменение длины, но решение все равно не принимается :С

Реализовать метод main(String[]), в котором нужно скопировать содержимое массивов firstArray и secondArray в один массив resultArray.
Массив firstArray должен быть в начале нового массива resultArray, а secondArray — после него.
Программа должна правильно обрабатывать массивы любой длины.

Требования:

В классе Solution должна быть публичная статическая переменная firstArray типа int[], проинициализированная массивом с 10-ю значениями.
В классе Solution должна быть публичная статическая переменная secondArray типа int[], проинициализированная массивом с 10-ю значениями.
В классе Solution должна быть публичная статическая переменная resultArray типа int[].
Реализуй метод main(String[]) согласно условию.

package com.javarush.task.pro.task05.task0504; import java.util.Arrays; /* Объединяем массивы */ public class Solution { public static int[] firstArray = new int[]{0, 1, 2, 3, 4, 5, 6, 7, 8, 9}; public static int[] secondArray = new int[]{10, 11, 12, 13, 14, 15, 16, 17, 18, 19}; public static int[] resultArray; public static void main(String[] args) { resultArray = new int[firstArray.length + secondArray.length]; int i = 0; while(i<resultArray.length){ if(i<10){ resultArray[i] = firstArray[i]; }else{ resultArray[i] = secondArray[i - 10]; } i++; } System.out.println(Arrays.toString(resultArray)); } }

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

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

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

Mejhoks

Уровень 16

13 декабря 2020, 20:06

Поставь решено - если решил. Я то уже свою решил и не могу проверить почему не засчитало с while, но засчитало for. Хотя одна вещь бросается в глаза - то что ты при использовании while втыкнул с этим местом }else{ resultArray[i] = secondArray[i - secondArray.length]; } а в цикле for исправил }else{ resultArray[i] = secondArray[i - firstArray.length]; } Как бы провтык логики был, хотя и то 10 и тот 10, но если было бы не 10, а 20, номер ячейки был бы отрицательный. Не знаю может ли задачник проверить такое, типа тонкий случай... Разве что он проверяет пару разных скрытых выходных данных... И у тебя так как раз не стыковка была на которою он напоролся...

hidden #2322530

Уровень 41

14 ноября 2020, 16:08

решение

под разную длину массива у тебя ничего не адаптировано а с чего ты взял, что например у тебя firstArray будет всегда при тестах 10 элементов?

if(i<10){

прохождение массива в цикле с индексами лучше делать так:

for (int i = 0; i < resultArray.length; i++) { // проходим новый массив
    if (i < firstArray.length) { // если  i < длины первого массива
        resultArray[i] = firstArray[i]; // заполняем первым массивом
    } else { // иначе заполняем вторым массив, где индекс считается i - длина первого
                // тогда для 10 элемента будет брать 0й из второго массива
        resultArray[i] = secondArray[i - firstArray.length];
    }
}

PS. а массивы изначальное были static переменными?

Ilnar_Bajteev

Уровень 20

14 ноября 2020, 16:10

да, статик был изначально. Оптимизировал, но все равно не принимает

package com.javarush.task.pro.task05.task0504;

import java.util.Arrays;

/*
Объединяем массивы
*/

public class Solution {
    public static int[] firstArray = new int[]{0, 1, 2, 3, 4, 5, 6, 7, 8, 9};
    public static int[] secondArray = new int[]{10, 11, 12, 13, 14, 15, 16, 17, 18, 19};
    public static int[] resultArray;

    public static void main(String[] args) {
        resultArray = new int[firstArray.length + secondArray.length];
        int i = 0;
        while(i<resultArray.length){
            if(i< firstArray.length){
                resultArray[i] = firstArray[i];
            }else{
                resultArray[i] = secondArray[i - secondArray.length];
            }
            i++;
        }
        System.out.println(Arrays.toString(resultArray));
    }
}

Ilnar_Bajteev

Уровень 20

14 ноября 2020, 16:16

Использовал ваше предложение, по поводу цикла for и все получилось, но все таки, в чем была проблема старого решения?

package com.javarush.task.pro.task05.task0504;

import java.util.Arrays;

/*
Объединяем массивы
*/

public class Solution {
    public static int[] firstArray = new int[]{0, 1, 2, 3, 4, 5, 6, 7, 8, 9};
    public static int[] secondArray = new int[]{10, 11, 12, 13, 14, 15, 16, 17, 18, 19};
    public static int[] resultArray;

    public static void main(String[] args) {
        resultArray = new int[firstArray.length + secondArray.length];

        for(int i = 0;i< resultArray.length;i++){
            if(i< firstArray.length){
                resultArray[i] = firstArray[i];
            }else{
                resultArray[i] = secondArray[i- firstArray.length];
            }
        }
        System.out.println(Arrays.toString(resultArray));
    }
}

hidden #2322530

Уровень 41

14 ноября 2020, 16:17

можно ещё так попробовать:

resultArray = new int[firstArray.length + secondArray.length];
System.arraycopy(firstArray,0,resultArray,0,firstArray.length);
System.arraycopy(secondArray,0,resultArray,firstArray.length,secondArray.length);
System.out.println(Arrays.toString(resultArray));

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

resultArray[i] = secondArray[i - secondArray.length];

Ilnar_Bajteev

Уровень 20

14 ноября 2020, 16:20

я понял, но я ведь потом исправил и отталкивался только от array.length, разве нет?(простите, если прозвучало грубо с; )

hidden #2322530

Уровень 41

14 ноября 2020, 16:21

полезный

суть в том, что я отнимал длину первого ( мы же первый скопировали уже)

resultArray[i] = secondArray[i- firstArray.length];

а ты отнимал от второго.

resultArray[i] = secondArray[i - secondArray.length];

а представь если у первого 10 элементов, а у второго 20? попробуй ради интереса, там будет исключение

Ilnar_Bajteev

Уровень 20

14 ноября 2020, 16:22

это понял, не заметил, спасибо большое! <3

hidden #2322530

Уровень 41

14 ноября 2020, 16:22

а так я потом написал ещё вариант, можно было в 3 строки решить всё с помощью метода

System.arraycopy()

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

Ilnar_Bajteev

Уровень 20

14 ноября 2020, 16:24

Понял, учту и буду его теперь использовать)