Какие еще варианты есть компактнее

@3vik · Регистрация: 12.02.2018

Правильно ли я сделал? Есть ли еще варианты, поскидывайте пожалуйста я поразбираюсь

Программа вводит пять чисел с клавиатуры и выводит минимальное из них на экран.
Требования:
1. Программа должна считывать числа с клавиатуры.
2. В классе должен быть метод public static void main.
3. В классе должен быть метод public static min, принимающий 5 параметров типа int.
4. Метод min должен возвращать минимум из 5 переданных чисел. Если минимальных чисел несколько - вернуть любое из них.
5. Программа должна выводить строку, которая начинается на "Minimum = ".
6. Программа должна выводить строку, которая заканчивается минимальным из 5 введенных чисел.

Java

    public static void main(String[] args) throws Exception {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        int a = Integer.parseInt(reader.readLine());
        int b = Integer.parseInt(reader.readLine());
        int c = Integer.parseInt(reader.readLine());
        int d = Integer.parseInt(reader.readLine());
        int n = Integer.parseInt(reader.readLine());
 
        int minimum = min(a, b, c, d, n);
 
        System.out.println("Minimum = " + minimum);
    }
 
 
    public static int min(int a, int b, int c, int d, int n) {
        int[] arr = {a, b, c, d, n};
        for (int i = 0; i < arr.length; i++){
            arr[0] = a;
            arr[1] = b;
            arr[2] = c;
            arr[3] = d;
            arr[4] = n;
 
 
        }
        Arrays.sort(arr);
        for (int i = 0; i < arr.length; i++) {
 
        }
        return arr[0];
    }
}

Добавлено через 42 секунды
можно ли мой код оптимизировать?

@iSmokeJC · 22.03.2018, 21:45

Сообщение от 3vik

можно ли мой код оптимизировать?

Можно.
Вместо 5 переменных задай массив из 5 элементов, заполняй его в цикле. Вместо BufferedReader используй сканер и метод nextInt(). В метод min передавай этот массив, там сортируй и возвращай минимум. Профит

Добавлено через 53 секунды

Сообщение от 3vik

int minimum

переменную можно вообще не использовать

@xoraxax · 22.03.2018, 21:58

вместо сортировки просто найди минимум, остальное без разницы

@3vik · 22.03.2018, 22:50 **[ТС]**

iSmokeJC,

Java

        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        int a;
        int arr [] = new int[5];
        for (int i = 0; i < arr.length; i++){
           a = Integer.parseInt(reader.readLine());
           arr[i] = a;
 
 
        }
 
        System.out.println(min(arr));
    }
 
 
    public static int min(int [] arr) {
        Arrays.sort(arr);
    
        return arr[0];
    }
}

"В классе должен быть метод public static min, принимающий 5 параметров типа int."
Правильно? вроде работает, но все равно не по условию, я без сканера сделал, т. к. к Buffer что то привык уже.

@iSmokeJC · 22.03.2018, 22:56

3vik, цикл - то зачем в методе min?

Сообщение от 3vik

принимающий 5 параметров типа int

Если это обязательно, то придется таки делать как в твоем 1-м варианте и передавать туда arr[0], arr[1]...

@3vik · 22.03.2018, 22:59 **[ТС]**

iSmokeJC, по условию да, было обязательно, спасибо!

@iSmokeJC · 22.03.2018, 22:59

Java

1
2
3

for (int i = 0; i < arr.length; i++){
     arr[i] = Integer.parseInt(reader.readLine());
}

Переменная а не нужна

@3vik · 22.03.2018, 23:05 **[ТС]**

iSmokeJC, тут сразу получается и заполняет тем что с клавиатуры, а у меня было разбито на 2 шага?

@iSmokeJC · 22.03.2018, 23:06

3vik, ага

@3vik · 22.03.2018, 23:10 **[ТС]**

iSmokeJC, Спасибо огромное!!!!

Добавлено через 2 минуты
xoraxax, минимум путем сравнения переменных в массиве? или как

@korvin_ · 23.03.2018, 10:51

Сообщение от 3vik

или как

Java

1	IntStream.of(arr).min();

@ArtemFM · 26.03.2018, 01:55

Java

import java.util.Scanner;
 
public class Task004 {
    private static final int AMOUNT_NUMBERS = 5;
 
    public static void main(String[] args) {
        int[] numbers = readArrayFromConsole(AMOUNT_NUMBERS);
 
        int minNumber = getMinNumber(numbers);
 
        System.out.printf("Minimum = %d;", minNumber);
    }
 
    public static int getMinNumber(int[] array) {
        int min = 0;
        if (array != null) {
            for (int number : array) {
                if (number < min) {
                    min = number;
                }
            }
        }
        return min;
    }
 
    public static int[] readArrayFromConsole(int amountNumber) {
        int[] result = amountNumber > 0 ? new int[amountNumber] : new int[0];
        if (amountNumber > 0) {
            Scanner read = new Scanner(System.in);
            for (int number = 1; number <= amountNumber; number++) {
                System.out.printf("Input number №%d: ", number);
                result[number - 1] = read.nextInt();
            }
        }
        return result;
    }
}

@Aviz__ · 26.03.2018, 11:44

Вот, держи, завалялось похожее:

Java

import java.util.Scanner;
 
public class CyberMinNum {
 
    private static int getMin(final int ... nums) {
        if (nums == null)
            return Integer.parseInt(null);
        if (nums.length == 1)
            return nums[0];
        if (nums.length == 2)
            return Math.min(nums[0], nums[1]);
        int minForRet = nums[0];
        for (int i = 1; i < nums.length; i++) {
            if (nums[i] < minForRet)
                minForRet = nums[i];
        }
        return minForRet;
    }
 
    public static void main(String args []){
        Scanner inp = new Scanner(System.in);
        System.out.printf("Введите через пробел 5 целых чисел%n");
        System.out.printf("Minimum = %d",getMin(inp.nextInt(), inp.nextInt(), inp.nextInt(),
                inp.nextInt(), inp.nextInt()));
    }
 
}

возможный вариант работы:

Bash

1
2
3

Введите через пробел 5 целых чисел
1 2 3 0 -1
Minimum = -1

@iSmokeJC Супер-модератор 6374 / 3630 / 1387 Регистрация: 21.10.2017 Сообщений: 10,496
	22.03.2018, 21:45	2
	Сообщение от 3vik можно ли мой код оптимизировать? Можно. Вместо 5 переменных задай массив из 5 элементов, заполняй его в цикле. Вместо BufferedReader используй сканер и метод nextInt(). В метод min передавай этот массив, там сортируй и возвращай минимум. Профит Добавлено через 53 секунды Сообщение от 3vik int minimum переменную можно вообще не использовать 0

@xoraxax 2879 / 2460 / 756 Регистрация: 05.07.2013 Сообщений: 11,903
	22.03.2018, 21:58	3
	вместо сортировки просто найди минимум, остальное без разницы 0

@iSmokeJC Супер-модератор 6374 / 3630 / 1387 Регистрация: 21.10.2017 Сообщений: 10,496
	22.03.2018, 22:56	5
	3vik, цикл - то зачем в методе min? Сообщение от 3vik принимающий 5 параметров типа int Если это обязательно, то придется таки делать как в твоем 1-м варианте и передавать туда arr[0], arr[1]... 0

@3vik 0 / 0 / 1 Регистрация: 12.02.2018 Сообщений: 142
	22.03.2018, 22:59 [ТС]	6
	iSmokeJC, по условию да, было обязательно, спасибо! 0

@3vik 0 / 0 / 1 Регистрация: 12.02.2018 Сообщений: 142
	22.03.2018, 23:05 [ТС]	8
	iSmokeJC, тут сразу получается и заполняет тем что с клавиатуры, а у меня было разбито на 2 шага? 0

@iSmokeJC Супер-модератор 6374 / 3630 / 1387 Регистрация: 21.10.2017 Сообщений: 10,496
	22.03.2018, 23:06	9
	3vik, ага 0

@3vik 0 / 0 / 1 Регистрация: 12.02.2018 Сообщений: 142
	22.03.2018, 23:10 [ТС]	10
	iSmokeJC, Спасибо огромное!!!! Добавлено через 2 минуты xoraxax, минимум путем сравнения переменных в массиве? или как 0