Что-то не так

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

Задача по алгоритмам Ӏ Java Syntax: 5 уровень, 12 лекция

Обсуждается

Ребят не пойму что не так, подскажите пожалуйста. Вроде нормально же все должно быть, а получается ввожу любое число и программа выводит изначальный int maximum ( в данном случае "0").

Написать программу, которая:
1. считывает с консоли число N, которое должно быть больше 0
2. потом считывает N чисел с консоли
3. выводит на экран максимальное из введенных N чисел.

Требования:

Программа должна считывать числа с клавиатуры.
Программа должна выводить число на экран.
В классе должен быть метод public static void main.
Нельзя добавлять новые методы в класс Solution.
Программа должна выводить на экран максимальное из введенных N чисел.
Программа не должна ничего выводить на экран, если N меньше либо равно 0.

package com.javarush.task.task05.task0532; import java.io.*; /* Задача по алгоритмам */ public class Solution { public static void main(String[] args) throws Exception { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); int numbers = Integer.parseInt(reader.readLine()); int[] max = new int[numbers]; int maximum = 0; if (numbers > 0) { for (int i : max) { maximum = Math.max(maximum, max[i]); } } System.out.println(maximum); } }

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

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

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

Макс

Уровень 41

28 апреля 2019, 16:41

И еще один момент, у тебя цикл использован неправильно. Такая форма записи, через двоеточие, это так называемый forEach. Суть его в том, что он проходит по всему массиву и на каждом круге переменной присваивается очередное значение этого массива.

for (int i : max)

переменная i при такой записи это не счетчик, а сам элемент массива, т.е. ей на каждом круге поочередно присваивается max[0], потом max[1], max[2] и т.д. Поэтому в теле цикла нужно вместо max[i] писать просто i:

for (int i : max) {
                maximum = Math.max(maximum, i);
            }

Ну или использовать обычную форму цикла for:

for (int i = 0; i < max.length; i++) {
                maximum = Math.max(maximum, max[i]);
            }

Макс

Уровень 41

28 апреля 2019, 16:40

Для начала я попробую объяснить что на самом деле делает твоя программа, тогда возможно ты самостоятельно сможешь разобраться, что в ней не так и как сделать правильно. В начале ты считываешь с консоли одну строку, приводишь ее в числовой формат и присваиваешь это значение переменной numbers, это строчка 12:

int numbers = Integer.parseInt(reader.readLine());

Затем ты создаешь пустой целочисленный массив размером numbers и присваиваешь его переменной max (подчеркиваю, это пустой массив, в нем есть место для numbers элементов, но сами элементы ты в массив не добавлял (а если точнее, то там во всех ячейках массива нули "0", т.к. это массив int который не может принимать пустое значение "null")). Это строчка 13:

int[] max = new int[numbers];

А дальше ты в цикле сравниваешь число maximum (которое равно "0") и число max[0], т.е. первый элемент массива (в котором, как мы уже определили, все элементы тоже равны "0"), это строчка 19:

maximum = Math.max(maximum, max[i]);