Это совсем плохое решение или для данного уровня нормально?

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

Решен

if (a1 > b1 && a1 >= c1 || a1 > c1 && a1 >= b1)
  {
      if (b1 >= c1)
      System.out.println(a1 + " " + b1 + " " + c1);
      else
      System.out.println(a1 + " " + c1 + " " + b1);
  }
else if (b1 > c1 && b1 >= a1 || b1 >= c1 && b1 > a1)
  {
      if (c1 >= a1)
      System.out.println(b1 + " " + c1 + " " + a1);
      else
      System.out.println(b1 + " " + a1 + " " + c1);
  }
else if (c1 > b1 && c1 >= a1 || c1 >= b1 && c1 > a1)
  {
      if (a1 >= b1)
      System.out.println(c1 + " " + a1 + " " + b1);
      else
      System.out.println(c1 + " " + b1 + " " + a1);
  }

Прошу поделиться идеальным решением этой задачи. Спасибо!

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

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

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

Pavel KurchavovSupport Engineer в Ozon.ru

17 июля 2020, 12:22

Мое решение на данный уровень знаний, лучше не придумал.

BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

        List<Integer> list = new ArrayList<>();

        for(int i = 0; i < 3; i++) {
            int number = Integer.parseInt(reader.readLine());
            list.add(number);
        }
        Collections.sort(list);
        Collections.reverse(list);

        for(Integer number: list) {
            System.out.println(number);
        }

Никита

Уровень 31

Expert

17 июля 2020, 20:00

наверное самое простое.

Ivan Sushkov

Уровень 8

17 июля 2020, 21:31

Спасибо!

Pavel KurchavovSupport Engineer в Ozon.ru

17 июля 2020, 12:14

Первое что бросается в глаза, это названия переменных. У тебя есть a1, b1, c1. Не проще ли для читаемости назвать их тогда a, b, c?) И также выражения в условии оператора if еще разграничить скобками , например:

((a1 > b1 && a1 >= c1)  ||  (a1 > c1 && a1 >= b1))

Так будет лучше читаться. Работающее решение - это хорошо, но еще важно, чтобы другой человек мог без особых проблем понять, что тут происходит. А так молодец!))))

Никита

Уровень 31

Expert

17 июля 2020, 11:41

полезный

если решение работает, то оно уже неплохое))) а лучшим будет через массивы) на данном уровне наверное сделать метод min для двух чисел, и попарно сравнить