JavaRush/Помощь по Java задачам/не пойму в чем проблема, помогайте по логике всэ работает...

5 уровень

04.06.2019
571просмотров
10комментариев

не пойму в чем проблема, помогайте по логике всэ работает, а проверку не проходит

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

Обсуждается

/*
Ближайшее к 10
*/

public class Solution {
    public static void main(String[] args) {
        displayClosestToTen(9, -1);
        displayClosestToTen(20, -5);
        displayClosestToTen(10,10);
        displayClosestToTen(-10,10);
        displayClosestToTen(0,-5);
    }

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


    }

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

Напиши метод 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(9, -1); displayClosestToTen(20, -5); displayClosestToTen(10,10); displayClosestToTen(-10,10); displayClosestToTen(0,-5); } public static void displayClosestToTen(int a, int b) { int c, d; c = abs(abs(a)- 10); d = abs(abs(b)- 10); if (c < d) System.out.println(a); else if (c>d) System.out.println(b); else System.out.println(a); } public static int abs(int a) { if (a < 0) { return -a; } else { return a; } } }

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

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

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

Denis

Уровень 5

8 июня 2019, 12:02

Спасибо всем за намеки, переделал, теперь вроде правильно считает а валидацию пройти не могу. Что не так???

public static void displayClosestToTen(int a, int b) {
        int c, d;
        if (a<0)c = abs(a)+10;
        else c = abs(a-10);

        if (b<0)d = abs(b)+10;
        else d = abs(b-10);

        if (c<d) System.out.println(a);
        else if (c>d) System.out.println(b);
        else System.out.println(a);

displayClosestToTen(9, -1); displayClosestToTen(20, -5); displayClosestToTen(10,10); displayClosestToTen(-10,10); displayClosestToTen(0,20); displayClosestToTen(-10,30); displayClosestToTen(0,-5); --------- 9 20 10 10 0 -10 0 Process finished with exit code 0

Hukiro

Уровень 9

5 июня 2019, 16:35

Как уже говорили, ошибка в логике: отнимая 10 от модуля переменных (a, b) и сравнивая (a ,b) друг с другом, мы получаем некорректное значение, в случае если переменные были отрицательными, поскольку модуль числа будет уже положительным. Одним из выходов из данной ситуации будет отнять 10 до того, как обращаться к методу abs

Алексей

Уровень 41

4 июня 2019, 11:32

Проблема здесь в самой логике. На неё правильно указал пользователь с ником Sergei (Санкт-Петербург). В вашей логике для отрицательных чисел будет неверно высчитываться "расстояние до числа 10" (например, для "-14" до "10" будет не "24", а "4"). Поэтому рекомендую поменять логику: сделайте случаи для отрицательного числа и положительного числа отдельно. Да, по два ветвления на каждое число, но не так много, как казалось бы.

Владимир Созанский

Уровень 24

4 июня 2019, 10:00

я думаю что ошибок 2 1. обращение к методу без имени класса нужно Solution.abs(a) вместо abs(a) 2. никто не указывал что нужно получать модуль числа, поэтому правильнее написать c = abs(a - 10); d = abs(b - 10); -10 -10 = 20 10-10 = 0 0<20 результат 10 (а не -10)

hidden #598481

Уровень 27

4 июня 2019, 10:35

Первое - это одно и то же в рамках класса.

Владимир Созанский

Уровень 24

4 июня 2019, 10:54

Как знаете. Так как мы обучаемся, а не пишем программу в продакшен, то желательно писать в правильном стиле.

Sergei

Уровень 24

4 июня 2019, 09:16

c = abs(abs(a)- 10);
d = abs(abs(b)- 10);
if (c < d) System.out.println(a);

по моему в логике не всё верно a = -14, b = 15 c = 4, d = 5 c < d, поэтому -14 ближе к 10, чем 15 0o может наоборот, если c > d?

Евгений

Уровень 20

4 июня 2019, 09:03

Скорее всего не выполнено условие если числа на равной длине.

IceBerg

Уровень 41

Master

4 июня 2019, 08:57

displayClosestToTen(20, -5); //ожидаем 20
displayClosestToTen(-10,10); //10

Denis

Уровень 5

8 июня 2019, 10:58

spasibo