Ошибка

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

Решен

первый раз пробую применить метод в решении такого рода задачи.. но не пойму, где ошибка

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

Требования:

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

package com.javarush.task.task04.task0418; import java.util.Scanner; /* Минимум двух чисел */ public class Solution { public static void main(String[] args) throws Exception { //напишите тут ваш код Scanner first = new Scanner(System.in); int a = first.nextInt(); Scanner second = new Scanner(System.in); int b = second.nextInt(); Scanner third = new Scanner(System.in); int c = third.nextInt(); Scanner fourth = new Scanner(System.in); int d = fourth.nextInt(); if ( maxf(a,b) < maxf(b,c) ) System.out.println(maxf(a,b)); else System.out.println(maxf(d,c)); public static int maxf(int x, int y) { int max; if (x < y) max = y; else max = x; return max; } } }

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

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

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

Михаил Гараздюк

Уровень 22

29 марта 2020, 23:23

Возможно компилятору не нравится, что ты используешь Scanner. Попробуй использовать BufferReader, вместо Scanner:

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());

И еще я вижу у тебя опечатку: if ( maxf(a,b) < maxf(b,c) ) - ты во втором вызове b переменную написал вместо d

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

Целое условие вывода у тебя неправильное. Тебе нужно вывести большее число , а ты при сравнении выводишь меньшее. Еще у тебя метод внутри метода. maxf у тебя в main методе. Нужно его вынести за пределы main. И про последнее условие тоже не надо забывать: "Если максимальных чисел несколько, необходимо вывести любое из них."

Андрей Малышкин

Уровень 8

30 марта 2020, 07:01

Спасибо, исправил ошибки, всё заработало.

Павел Безумный учёный

Expert

29 марта 2020, 22:03

Сравните числа попарно при помощи метода maxf( ):

System.out.println(maxf(maxf(a,b), maxf(c,d)));

Андрей Малышкин

Уровень 8

29 марта 2020, 22:15

выглядит компактно и похоже на правду, если работает, то крутое решение! Но ошибку выдаёт в другом месте. в строчке 25

Павел Безумный учёный

Expert

29 марта 2020, 22:20

Сообщение об ошибке в 25 строке выдаётся из-за того, что код метода maxf( ) находится внутри main( ). Переместите его в класс Solution рядом с main( ).

Андрей Малышкин

Уровень 8

29 марта 2020, 22:27

это я исправил, мне уже писали об этом.. всё равно ошибка Отсутствует идентификатор или переменная.

Павел Безумный учёный

Expert

29 марта 2020, 22:30

Проверьте имя пакета в первой строке. Задача Максимум четырех чисел имеет номер 0419:

package com.javarush.task.task04.task0419;

Андрей Малышкин

Уровень 8

29 марта 2020, 22:39

неа package com.javarush.task.task04.task0419; import java.util.Scanner; /* Минимум двух чисел */ public class Solution { public static void main(String[] args) throws Exception { //напишите тут ваш код Scanner first = new Scanner(System.in); int a = first.nextInt(); Scanner second = new Scanner(System.in); int b = second.nextInt(); Scanner third = new Scanner(System.in); int c = third.nextInt(); Scanner fourth = new Scanner(System.in); int d = fourth.nextInt();} /* if ( maxf(a,b) < maxf(c,d) ){ System.out.println(maxf(c,d)); else System.out.println(maxf(a,b)); */ System.out.println(maxf(maxf(a,b), maxf(c,d))); public static int maxf(a, b) { int max; if (a < b) max = a; else max = b; return max; } } }