Задача 06.Equal Arrays - от Fundamentals module - Софтуерен университет

dZf1aeA-KlimentHristov 56 Точки

Задача 06.Equal Arrays - от Fundamentals module

Здравейте,

немога да разбера, каде ми е грешката в тази задача, уж открих логиката но ми дава 80 точки.

Може ли малко повече разяснение по нея. Благодаря Ви.

package com.company;

import java.lang.reflect.Array;
import java.util.Scanner;

public class EqualArrays {
    public static void main(String[] args) {
        Scanner scan = new Scanner(System.in);

        String[] firstArray = scan.nextLine().split(" ");
        int[] numberFirst = new int[firstArray.length];

        String[] secondArray = scan.nextLine().split(" ");
        int[] numberSecond = new int[secondArray.length];

        int sumFirst = 0;
        int diffrent = 0;
        int position = 0;

        for (int i = 0; i < firstArray.length; i++) {
            numberFirst[i] = Integer.parseInt(firstArray[i]);
            numberSecond[i] = Integer.parseInt(secondArray[i]);
            if (numberFirst[i] != numberSecond[i]){
                diffrent++;
                position = numberFirst[i] -diffrent;
            }
            sumFirst += numberFirst[i];
        }

        if (diffrent == 0){
            System.out.printf("Arrays are identical. Sum: %d",sumFirst);
        }else {
            System.out.printf("Arrays are not identical. Found difference at %d index.",position);
        }
    }
}

Тагове:

13/04/2020 21:20:35 Java Fundamentals

svephoto 1191 Точки

Здравей,

Би ли дал и условието на задачата?

13/04/2020 23:37:17

dZf1aeA-KlimentHristov 56 Точки

Read two arrays and print on the console whether they are identical or not.

Arrays are identical if their elements are equal.

If the arrays are identical find the sum of the first one and print on the console following message:

"Arrays are identical. Sum: {sum}", otherwise find the first index where the arrays differ and print on the console following message: "Arrays are not identical. Found difference at {index} index."

14/04/2020 07:41:40

MartinBG 4803 Точки

Не изчисляваш правилно индекса, на който се открива първата разлика.

Промени:

            if (numberFirst[i] != numberSecond[i]){
                diffrent++;
                position = numberFirst[i] -diffrent;
            }

На:

            if (numberFirst[i] != numberSecond[i]) {
                diffrent++; // това може да е boolean вместо int
                position = i; // i е индекса, на който се намира разликата
                break; // прекъсваме цикъла, защото ни интересува първата разлика
            }

____________

Ето едно решение, което използва неща, които още не сте учили (Stream API), но ще ти е полезно да видиш и по-съвременен подход (for циклите от много години са заменени от по-практични конструкции във всички модерни езици):

import java.util.Arrays;
import java.util.Scanner;
import java.util.stream.IntStream;

public class EqualArrays {
    public static void main(String[] args) {
        Scanner scan = new Scanner(System.in);

        int[] firstArray = getIntArrayFromString(scan.nextLine());
        int[] secondArray = getIntArrayFromString(scan.nextLine());

        String output = IntStream
                .range(0, firstArray.length)
                .filter(index -> firstArray[index] != secondArray[index])
                .mapToObj(index -> String.format(
                        "Arrays are not identical. Found difference at %d index.",
                        index))
                .findFirst()
                .orElse(String.format(
                        "Arrays are identical. Sum: %d",
                        Arrays.stream(firstArray).sum()));

        System.out.println(output);
    }

    private static int[] getIntArrayFromString(String str) {
        return Arrays.stream(str.trim().split("\\s+"))
                .mapToInt(Integer::parseInt)
                .toArray();
    }
}

13/04/2020 23:40:38 13/04/2020 23:55:24

svephoto 1191 Точки

Колегата ти го е дал доста добре описано и с нови полезни неща.

Друг вариант е леко да си рефакторираш твоя код и да махнеш

        int diffrent = 0;
        int position = 0;

- не са ти нужни.

След това да сложиш sumFirst += numberFirst[i]; преди проверката

if (numberFirst[i] != numberSecond[i]) { }

и вътре в нея да си пуснеш печатането на случая, в който масивите не са идентични (тук position просто го заместваш с i), като след това обаче трябва да прекратиш изпълнението на програмата с един return. А печатането на стринга при случая, в който масивите са идентични го оставяш само него накрая без никакви проверки и си готов.

14/04/2020 00:04:19 14/04/2020 00:09:17

dZf1aeA-KlimentHristov 56 Точки

Поправих само и-то и аз не брейквах, а е трябвало да брейкна при първия.

Благодаря колега.

14/04/2020 07:44:32

dZf1aeA-KlimentHristov 56 Точки

Колега, като я решавах вчера задачата не го виждах по този начин решението, малко по подробно пиша задачите все още и не съм мастер де :) Но всеки следващ път се опитвам за по кратки решения и по малко описания.

Благодаря ти.

14/04/2020 07:47:51