JavaRush/Помощь по Java задачам/Всё выводится корректно, но тз не проходит. Помогите поня...

35 уровень

23.07.2019
787просмотров
12комментариев

Всё выводится корректно, но тз не проходит. Помогите понять в чём проблема

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

Решен

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

Напиши метод displayClosestToTen. Метод должен выводить на экран ближайшее к 10 из двух чисел, записанных в аргументах метода.
Например, среди чисел 8 и 11 ближайшее к десяти 11. Если оба числа на равной длине к 10, то вывести на экран любое из них.

Подсказка:
используй метод public static int abs(int a), который возвращает абсолютную величину числа.

Требования:

Программа должна выводить числа на экран.
Метод main не должен вызывать System.out.println() или System.out.print().
Метод main должен вызывать метод displayClosestToTen.
Метод displayClosestToTen должен вызывать метод abs.
Метод displayClosestToTen должен выводить число на экран согласно заданию.

package com.javarush.task.task04.task0409; /* Ближайшее к 10 */ public class Solution { public static void main(String[] args) { displayClosestToTen(8, 11); displayClosestToTen(7, 14); } public static void displayClosestToTen(int a, int b) { int x = 0; int y = 0; int absX = abs(a); int absY = abs(b); //Проверка на отрицательность if (a < 0) { x = absX + 10; //проверка на диапозон от 0 до 10 } else if (absX >= 0 && absX <= 10){ x = (absX - 10) * -1; //если больше 10 } else x = absX - 10; if (b < 0) { y = absY + 10; } else if (absY >= 0 && absY <= 10){ y = (absY - 10) * -1; } else y = absY - 10; if (x > y) { System.out.println(b); } else if (x < y) { System.out.println(a); } else if (x == y) { System.out.println(a); } } public static int abs(int a) { if (a < 0) { return -a; } else { return a; } } }

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

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

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

hidden #598481

Уровень 27

23 июля 2019, 11:50

полезный

Ошибок в результатах я не нашел, но почему ты ищешь модуль числа, а не сразу длину отрезков от a до 10, от б до 10? Ведь геометрический смысл модуля это и есть расстояние между двумя точками. Можно было бы избавиться от всего колхоза в решении: тебя не будет волновать знак переменных a и b, не нужны будут все эти ифы. И здесь

} else if (x < y) {
    System.out.println(a);
} else if (x == y) {
    System.out.println(a);
}

Если у тебя при обоих условиях требуется сделать одно и то же, почему не объединить их в один блок, зачем два? особенно учитывая, что в совокупности они представляют собой все, что не подходит под условие "if (x > y)" - можно же просто написать else.

darkSide

Уровень 35

23 июля 2019, 12:53

Из твоего комментария я понял, что это мне не надо

if (a < 0) {
            x = absX + 10;
            //проверка на диапозон от 0 до 10
        } else if (absX >= 0 && absX <= 10){
            x = (absX - 10) * -1;
            //если больше 10
        } else
            x = absX - 10;
        if (b < 0) {
            y = absY + 10;
        } else if (absY >= 0 && absY <= 10){
            y = (absY - 10) * -1;
        } else
            y = absY - 10;

И Сразу получая значения absX b absY должен их считать x = absX - 10 y = absY - 10 Потом ифом посмотреть кто ближе, но как я узнаю что это отрицательное число? Ведь если у меня будет -11 и 8, то значение -11 будет ближе по логике

darkSide

Уровень 35

23 июля 2019, 13:09

Я правда не могу понять, уже загугли и про геометрический смысл модуля и понял что такое абсолютная величина и слегка догадываюсь о чём ты говоришь, но не понимаю что именно от меня требуется Сижу уже второй час и лишь на этот кусок кода меня хватило:

public static void displayClosestToTen(int a, int b) {
        int x = 0;
        int y = 0;
        int absX = abs(a);
        int absY = abs(b);

        if (a < 0) {
            x = absX + 10;
        } else
            x = abs(absX - 10);

        if (b < 0) {
            y = absY + 10;
        } else
            y = abs(absY - 10);

        if (x > y) {
            System.out.println(b);
        } else
            System.out.println(a);
    }

Радует, что ещё остаётся боевой настрой, буду думать дальше Не талантом, так усердием...))

darkSide

Уровень 35

23 июля 2019, 13:33

решение

Не знаю считается ли это победой, но я нашёл решение в интернете на stackOverflow И вроде бы понятно как всё делается, но почему именно так?

public static void displayClosestToTen(int a, int b) {
        if (abs(10-a) < abs(10-b)) {
            System.out.println(a);
        } else
            System.out.println(b);
    }

    public static int abs(int a) {
        if (a < 0) {
            return -a;
        } else {
            return a;
        }
    }

Мы ведь получаем новое значение из метода abs и условие в иве не должно быть равно if (2 < 21) {на вывод должно идти первое условие и переменная a } Но идёт второе else с переменной b У меня есть догадка, что в методе abs действия происходят с копией переменной

hidden #598481

Уровень 27

23 июля 2019, 14:27

Че-че-че?) Попробуй спросить еще раз со знаками препинания; укажи, для каких a и b ты проверяешь? Суть модуля понимаешь?

Ореховый торт без выпечки

Уровень 26

24 июля 2019, 06:06

вы не совсем поняли решение, т.к. до конца так и не поняли задачу. "Ведь если у меня будет -11 и 8, то значение -11 будет ближе по логике" - это не верно)) в этом случае это должно быть 8. Потому что как писали ниже про шаги от 8 до 10 два шага, а от -11 до 10 аж целых 21 шаг. И соответсвенно if (2 < 21) мы естественно должны выбрать тот вариант, где два шага, а else идет только в том случае, если условие в ифе не выполняется. Надеюсь хоть чуть-чуть стало понятнее...

darkSide

Уровень 35

24 июля 2019, 07:29

У меня получилось разобраться ещё вчера) Я когда пытался написать свою мысль vk, то в самом конце понял о чём идёт речь и как это всё работает, мне не дал сайт вывести сообщение, так как слишком много было от меня комментариев на моём уровне Но спасибо ещё раз за объяснение :)

Aleksei Ilin Software Developer в Axelot

23 июля 2019, 10:51

Нужно не любое меньшее или большее 10 число а именно ближайшее, т.е. найди самое большое число меньшее 10 и самое меньшее из больших 10 ) . Например представь, что 10 - это 0 (ноль) и тебе надо найти -1 и 1 (если бы такие были в списке).

darkSide

Уровень 35

23 июля 2019, 11:11

Я пытаюсь вчитаться в ваше объяснение, но у меня не получается его понять))

darkSide

Уровень 35

23 июля 2019, 11:20

Я не понимаю дальше: "т.е. найди самое большое число меньшее 10 и самое меньшее из больших 10 )" А пример я вообще не понял))

Руслан Full Stack Developer в novacom

23 июля 2019, 13:14

как я понял его то алексея он имел ввиду тебя нужно найти растояние, что такое растояние это грубо говоря количество шагов до 10 у числа 9 растояние будет 1 у числа 8 растояние будет 2..

darkSide

Уровень 35

23 июля 2019, 13:37

Теперь я понял что он хотел мне сказать, спасибо)