JavaRush/Помощь по Java задачам/Вот смотрю предыдущую лекцию и все равно чего-то не догон...

10 уровень

26.01.2019
1010просмотров
17комментариев

Вот смотрю предыдущую лекцию и все равно чего-то не догоняю, что не так делаю?

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

Решен

public class Solution {
    public static int min(int a, int b, int c) {

    int z;

    if (a < b && a < c)
    z = a;
    else
    if (b < a && b < c)
    z = b;
    else
    if (c < a && c < b)
    z = c;

    return z;
    //напишите тут ваш код
    }

Кажется, что все сделал так как понимаю, можете кто-нибудь обьяснить чего именно я не допираю? уже по всякому пробовал и ниче блин не выходит

Написать функцию, которая вычисляет минимум из трёх чисел.

Подсказка:
Нужно написать тело существующей функции min.

Требования:

Программа должна выводить текст на экран.
Метод min не должен выводить текст на экран.
Метод main должен вызвать метод min четыре раза.
Метод main должен выводить на экран результат работы метода min. Каждый раз с новой строки.
Метод min должен возвращать минимальное значение из чисел a, b и с.

package com.javarush.task.task02.task0216; /* Минимум трёх чисел */ public class Solution { public static int min(int a, int b, int c) { int z; if (a < b && a < c) z = a; else if (b < a && b < c) z = b; else if (c < a && c < b) z = c; return z; //напишите тут ваш код } public static void main(String[] args) throws Exception { System.out.println(min(1, 2, 3)); System.out.println(min(-1, -2, -3)); System.out.println(min(3, 5, 3)); System.out.println(min(5, 5, 10)); } }

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

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

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

Александр QA Automation Engineer в Alfa Bank

4 февраля 2019, 13:12

1) Используйте <=, поскольку при выполнении команды

System.out.println(min(5, 5, 10));

Компилятор не знает, что делать, когда a=b 2) Зачем дополительная переменная? Сделаем рефакторинг?) if (**********) { return a; } else if (**********) { return b; } else { return c; }

Дмитрий

Уровень 10

4 февраля 2019, 18:30

Все верно, я уже это усвоил)

Ren4T

Уровень 14

28 января 2019, 23:20

условие (a < b && a < c) и (b < a && b < c) в последнем случае не выполняются т.к a=b

Ren4T

Уровень 14

28 января 2019, 23:23

короче условие (a < b && a < c) работает только если a b c не равны друг другу

Дмитрий

Уровень 10

29 января 2019, 19:10

с этим вроде разобрался но зато встрял в другой задаче...

Riccio

Уровень 35

Master

26 января 2019, 23:11

Во-первых: возвращаемое значение - z - точно, стопроцентно должно быть инициализировано. У вас же все равно стоит условие else if (). Во-вторых: у вас не все ситуации рассмотрены в условиях.

Дмитрий

Уровень 10

26 января 2019, 23:32

''инициализировано'' это как?не совсем понял. И не понял, какие ситуации не рассмотрены?

Riccio

Уровень 35

Master

26 января 2019, 23:55

решение

Вы должны вернуть переменную типа int, а если z не будет инициализировано, оно будет NULL, а int не может быть нуллом. Так что для начала давайте изменим строчку вашего кода и посмотрим на результат:

int z = -999999; // число "из головы", просто для того, чтобы код заработал

Дмитрий

Уровень 10

27 января 2019, 00:05

Как Вы и сказали я подставил число в значение переменной z. Все заработало. Но главное я не понял почему... скажем тут (ур 2 лекция 7) public class MethodCall { public static void main(String[] args) { int a = 5, b = 7; int c = a, d = b; int m2; if (c < d) m2 = c; else m2 = d; int m = m2; System.out.println("Minimum is "+ m); разве м2 не инициализируется как null?

Riccio

Уровень 35

Master

27 января 2019, 00:10

Нет. Просто здесь объявление переменной и её инициализация разделены:

int m2; //объявлена переменная м2
//блок ее инициализации:
      if (c < d) // при соблюдении этого неравенства она равна переменной с
           m2 = c;
      else // и В ЛЮБОМ ДРУГОМ случае она равна d
             // тут нет условий после else, которые могут быть не выполнены
           m2 = d;

И что, правильно заработал код? И валидатор принял?

Дмитрий

Уровень 10

27 января 2019, 00:20

Да, теперь все работает, спасибо!) И все же я до конца не понял, если у меня после else есть еще другие условия(не одно) каким образом это мешает переменной z принять значение?

Riccio

Уровень 35

Master

27 января 2019, 00:26

Ваш код

public static int min(int a, int b, int c) {
    int z = -99999;

    if (a < b && a < c) z = a;
    else if (b < a && b < c) z = b;
    else if (c < a && c < b) z = c;

    return z;
    //напишите тут ваш код
}

Вывод:

1
-3
-99999 //ошибка
-99999 //ошибка

Тем, что у вас не все случаи охвачены. То есть z может быть не инициализировано. Если бы там просто было else, то говоря по-русски это звучало бы так: "В этом случае зет такая, в этом такая, а если эти случаи не сработают - она вот такая". А у вас последнее условие все равно не охватывает весь диапазон и зет может быть не инициализирована.

Дмитрий

Уровень 10

27 января 2019, 00:36

''не охватывает весь диапазон'' вот тут не понял почему не охватывает. Вроде кажется не сложно когда уже решено, но еще не знаю/понимаю всех правил и условий языка программирования. И извиняюсь за столь тупые вопросы, просто хочется понять своей головой, на своих словах) иначе далеко не уеду

Riccio

Уровень 35

Master

27 января 2019, 00:39

Покажите новый вариант кода.

Дмитрий

Уровень 10

27 января 2019, 00:47

public class Solution { public static int min(int a, int b, int c) { int z = 11; if (a <= b && a <= c) z = a; else if (b <= a && b <= c) z = b; else if (c <= a && c <= b) z = c; return z; Все прошло

Riccio

Уровень 35

Master

27 января 2019, 00:52

Ну, за самостоятельное осознание < и <= даю прямую подсказку:

...

А можно еще лаконичнее:

...

Дмитрий

Уровень 10

27 января 2019, 00:57

Низкий поклон вам!