В чем ошибка?

По разному пытался. Голову сломал. решает правильно и для отрицательных и положительных.

public class Solution {
    public static void main(String[] args) throws Exception {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        int N = Integer.parseInt(reader.readLine());
        int[] num = new int[N];
        int maximum = num[0];
        if (N > 0) {
            for (int i = 0; i < N; i++) {
                num[i] = Integer.parseInt(reader.readLine());
                maximum = Math.max(num[i], maximum);
                if (N == 1){
                    maximum = Math.max(num[0], num[i]);
                }
                }
            }
        System.out.println(maximum);
    }
}

вы умеете запускать свой код? Если нет, то непонятно с чего вы взяли "решает правильно и для отрицательных и положительных.", потому что ваш код не работает. Хотя возможно у вас своя система счисления.

это не ответ, а упрек. Если бы я не был уверен-я бы не спрашивал. Мне нужна помощь, а не насмешки. Есть чем помочь-я слушаю.

тогда скажите что именно вам непонятно? как вы пытались эту задачу решить, на словах опишите свой алгоритм. не зная как вы пытались решать - трудно ответить где у вас ошибка. особенно потому, что я бы решал эту задачу по-другому, и я пока не понимаю логику вашего решения. но ведь одну и ту же задачу можно решить по-разному, поэтому хотелось бы знать что именно у вас не получается, а не просто сказать "сделай так-то - и будет работать", ведь так вы ничему не научитесь. я бы в вашем коде инициализировал бы максимум не так:

int maximum = num[0];

ведь на момент, когда ваша программа дойдет до этого места - у вас в массиве пока еще не будет никаких данных, поэтому на всех позициях будут значения по умолчанию - 0. и получается, что вы максимум инициализировали нулем, соответственно, если пользователь введет числа -1, -2, -3 - то получит в ответе максимум = 0, хотя числа 0 он вообще не вводил. поэтому при таком коде, я бы инициализировал максимум минимально возможным числом

int maximum = Integer.MIN_VALUE;

кроме того, я в упор не понимаю зачем вам этот блок кода и что в нем вообще происходит:

if (N == 1){
    maximum = Math.max(num[0], num[i]);
}

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

создал переменную N для соблюдения условия 1. вводит с консоли число N, которое должно быть больше 0. Создаю цикл повторяющийся до значения N. Инициализирую массив num и присваиваю порядковому числу массива значения вводимое с клавиатуры. Переменная maximum получает значение первого порядкового числа массива num[], После этого в цикле идет сравнение порядкового номера текущего цикла с предыдущим. по поводу этого if (N == 1){ maximum = Math.max(num[0], num[i]);} - согласен. Тут от бессилия задал условие для значения N == 1. То есть если мы ввели первое число 1, то количество вводимых цифр то же 1. В таком случае можно было записать maximum = num[0]. Надеюсь я понятно выразил ход мысли. если чем кого обидел - прошу прощения(все задачи до этого решены самостоятельно) поэтому нервничаю.

"Создаю цикл повторяющийся до значения N." это вы про этот код говорите?

do {
    N = Integer.parseInt(reader.readLine());
}
while (N <= 0);

тут если пользователь ввел отрицательное число или 0 - вы снова считываете данные, ожидая, что пользователь сам поймет свою ошибку, исправится, и введет правильное число. не уверен, что такое поведение программы - это то, что указано в условии. скорее всего, если пользователь ввел отрицательное число - то достаточно просто закончить работу программы. "Инициализирую массив num и присваиваю порядковому числу массива значения вводимое с клавиатуры." вы наверное хотели сказать, что создаете массив размерностью N. если речь идет об этой строке

int[] num = new int[N];

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

int maximum = num[0];

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

задачу можно решить двумя способами. в обоих способах сначала получаем N. потом: способ 1. создаем массив размерностью N элементов. делаем цикл и наполняем в цикле наш массив теми данными, что введет пользователь. после этого цикла инициализируем переменную maximum первым элементом из массива, и делаем еще один цикл со второго элемента массива и до конца, где простой иф проверяет если текущее число больше максимума - тогда положить текущее значение в максимум. после цикла вывести максимум на экран способ 2. инициализируем максимум самым маленьким возможным числом, потом делаем цикл, внутри которого считываем данные с клавиатуры и сразу проверяем не больше ли они максимума. если больше - сохраняем считанное значение в максимум выводим максимум на экран как видно, во втором способе массив вообще не нужен :) у вас же получилась какая-то смесь из этих двух способов, поэтому что-то работает, а что-то - нет

да, не уточнил про нижний код

спасибо за развернутый ответ. Действительно массив еще не был пройден, обравистые знания были получены из других источников, поэтому работа кода, как вы правильно сказали смесь бульдога с носорогом.

Вот так сработало, по вашему способу 1. Да, действительно упустил момент, что максимум не был инициализирован в 1 цикле. Спасибо, пазл вроде сошелся. Вторым способом изначально не хотелось решать, хотелось освоить будущий материал.

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

вы мне подтвердили, что задача неправильна-это же мне сказал валидатор. и соответственно это понимал и я. Как исправить, и в чем конкретно была ошибка, я из вашего ответа не понял. Этот курс для людей (если верить утверждениям разработчиков) для людей ранее не занимавшихся программированием. то есть база 0. Или точнее null.Если хотите помочь - это нужно учитывать. что касается нижеуказанного цикла, я так и не понял почему он ошибочный. Мне кажется он выполняет условие 1. вводит с консоли число N, которое должно быть больше 0. если условие не верно цикл будет длится до выполнения условия.

do {
    N = Integer.parseInt(reader.readLine());
}
while (N <= 0);

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

Этот курс для людей (если верить утверждениям разработчиков) для людей ранее не занимавшихся программированием. то есть база 0. Или точнее null.Если хотите помочь - это нужно учитывать. Я это как раз и учитываю. Курс учит быть программистом, а умение находить ошибки в своем коде это почти основная часть работы, т.к. редко когда можно написать идеальный код сразу. В работе у вас будут Junit тесты, они тоже не будут говорить где ошибка. Из них вы узнаете, что подается на вход и какой выход ожидается (то, что я дал вам в первом сообщении). Человеку с опытом, я бы сразу указал на ошибку. На вас я потратил больше времени. Сначала искал ошибку, потом придумывал пример, чтобы вам показать её и чтобы вы могли найти её сами.

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

спасибо за уделенное время, оказанную помощь. Буду работать над ошибками