Не выполняет сортировку по условию, хотя IntelliJ IDEA её делает, проверял, в чём причина?

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

Решен

package com.javarush.task.task04.task0420;

/*
Сортировка трех чисел
*/

import java.io.*;

public class Solution {
    public static void main(String[] args) throws Exception {

     BufferedReader r = new BufferedReader(new InputStreamReader(System.in));
        int a = Integer.parseInt(r.readLine());
        int b = Integer.parseInt(r.readLine());
        int c = Integer.parseInt(r.readLine());
        int max, mid, min;
        max = max(a, b, c);
        mid = mid(a, b, c);
        min = min(a, b, c);
        System.out.print(max + " ");
        System.out.print(mid + " ");
        System.out.print(min);
        //Напишите тут ваш код
    }

    public static int max(int a, int b, int c) {
        if (a > b && a > c) return a;
        if (b > a && b > c) return b;
        if (c > b && c > a) return c;
        return 0;
    }

    public static int min(int a, int b, int c) {
        if (a < b && a < c) return a;
        if (b < a && b < c) return b;
        if (c < b && c < a) return c;
        return 0;
    }

    public static int mid(int a, int b, int c) {
        if (a > b && a < c) return a;
        if (b > a && b < c) return b;
        if (c < b && c > a) return c;
        return 0;
    }
}

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

Требования:

Программа должна считывать числа c клавиатуры.
Программа должна выводить числа на экран.
Программа должна выводить три числа разделенных пробелами.
Программа должна выводить числа в порядке убывания.

package com.javarush.task.task04.task0420; /* Сортировка трех чисел */ import java.io.*; public class Solution { public static void main(String[] args) throws Exception { BufferedReader r = new BufferedReader(new InputStreamReader(System.in)); int a = Integer.parseInt(r.readLine()); int b = Integer.parseInt(r.readLine()); int c = Integer.parseInt(r.readLine()); int max, mid, min; max = max(a, b, c); mid = mid(a, b, c); min = min(a, b, c); System.out.print(max + " "); System.out.print(mid + " "); System.out.print(min); //Напишите тут ваш код } public static int max(int a, int b, int c) { if (a > b && a > c) return a; if (b > a && b > c) return b; if (c > b && c > a) return c; return 0; } public static int min(int a, int b, int c) { if (a b && a < c) return a; if (a c) return a; if (b > a && b < c) return b; if (b < a && b > c) return b; if (c a) return c; if (c > b && c < a) return c; return 0; } }

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

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

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

Александр Андреев

Уровень 17

9 марта 2022, 07:06

int a = Integer.parseInt(reader.readLine()); int b = Integer.parseInt(reader.readLine()); int c= Integer.parseInt(reader.readLine()); int x1 = 0, x2 = 0, x3 = 0; if (a>b){ x1 = a; x2 = b; } else { x1 = b; x2 = a; } if (x2<c) { x3 = x2; x2 = c; if (x1<x2) {c=x1;x1=x2; x2=c; } } else { x3 = c; } System.out.println(x1 +" "+ x2 +" "+ x3);

Bogdan

Уровень 9

17 июня 2020, 15:11

нужно использовать >= и else, ведь числа могут быть равными

Даниил

Уровень 5

22 октября 2019, 20:31

BufferedReader reader = new BufferedReader( new InputStreamReader(System.in)); String s = reader.readLine(); String s1 = reader.readLine(); String s2 = reader.readLine(); int n = Integer.parseInt(s); int n1 = Integer.parseInt(s1); int n2 = Integer.parseInt(s2); if (n1 >= n && n1 >= n2){ if (n>=n2) System.out.println(n1+" "+n+" "+n2); else System.out.println(n1+" "+n2+" "+n); } else if (n>=n1&&n>=n2){ if (n1>=n2) System.out.println(n+" "+n1+" "+n2); else System.out.println(n+" "+n2+" "+n1); } else if (n2>=n1&&n2>=n){ if (n1>=n) System.out.println(n2+" "+n1+" "+n); else System.out.println(n2+" "+n+" "+n1); }

Insaner

Уровень 39

23 сентября 2017, 15:49

Неужели пропустило такое решение? Возвращать 0 - плохая идея, особенно когда все 3 числа равны.

Артём Лозовой

Уровень 21

23 сентября 2017, 12:28

Сам нашёл) public static int mid(int a, int b, int c) { if (a > b && a < c) return a; if (a c) return a; if (b > a && b < c) return b; if (b < a && b > c) return b; if (c a) return c; if (c > b && c < a) return c; return 0; в миде условий больше, чем 3.

Эдуард

Уровень 17

27 августа 2019, 07:33

как то громоздко, не проще? if (a < b) { if (a > c) { System.out.println((max = b)+" "+(midl = a)+" "+(min = c)); } else { if (b > c) { System.out.println((max = b) + " " + (midl = c) + " " + (min = a));// } else {System.out.println((max = c) + " " + (midl = b) + " " + (min = a));} }} else { if(a<c) { System.out.println((max = c) + " " + (midl = a) + " " + (min = b)); } else { if(b>c){ System.out.println((max = a) + " " + (midl = b) + " " + (min = c)); } else { System.out.println((max = a) + " " + (midl = c) + " " + (min = b)); } } }

Василий Александров

Уровень 5

31 января 2020, 12:14

У вас тоже, на мой субъективный взгляд, заморочено... И тяжело читабельно, особенно в таких строках else { if (b > c) { System.out.println((max = b) + " " + (midl = c) + " " + (min = a));// } else {System.out.println((max = c) + " " + (midl = b) + " " + (min = a));} }} 1. Можно легко запутаться в фигурных скобках; 2.Зачем вам // в коде?

Василий Александров

Уровень 5

31 января 2020, 12:16

public static void descrease (int a, int b, int c) {
        if (a>=b && b>=c)
            System.out.println(a + " " + b + " " + c);
        else if (a>=c && c>=b)

и т.д. На мой взгляд он более компактный и более проще читается.