Last checkpoint fails

Frage zur Aufgabe

Gelöst

My results are correct, but I don't get the last checkpoint. I could rewrite everything in a different style, but first I would like to know why it doesn't work this way. Thanks in advance!

Schreib ein Programm, das:
1. liest eine Zahl N (muss größer sein als 0) von der Konsole ein
2. liest N Zahlen von der Konsole ein
3. Zeigt die größte der N eingegebenen Zahlen an.

Anforderungen:

Das Programm muss die Zahlen von der Tastatur einlesen.
Das Programm muss eine Zahl auf dem Bildschirm anzeigen.
Die Klasse muss die Methode public static void main besitzen.
Füge der Klasse Solution keine neuen Methoden hinzu.
Das Programm muss die größte der N eingegebenen Zahlen anzeigen.
Das Programm darf nichts anzeigen, wenn N kleiner oder gleich 0 ist.

package de.codegym.task.task05.task0532; import java.io.*; import java.util.ArrayList; /* Aufgabe zu Algorithmen */ public class Solution { public static void main (String[] args) throws Exception { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); int maximum; int n; ArrayList<Integer> l1 = new ArrayList(); while (true) { n = Integer.parseInt(reader.readLine()); if (n <= 0) ; //continue; or System.exit(0); not working either else break; } for (int count = 0; count < n; count++) { l1.add(Integer.parseInt(reader.readLine())); } maximum = l1.get(0); for (int a : l1) { maximum = maximum <= a ? a : maximum; } System.out.println(maximum); } }

Kommentare (2)

Beliebt
Neu
Alt

Du musst angemeldet sein, um einen Kommentar schreiben zu können

CodeMeUp

Level 12 , Germany

25 September 2020, 10:17

It does not like the while loop.

while (true) {
            n = Integer.parseInt(reader.readLine());
            if (n <= 0) ; //continue; or System.exit(0); not working either
            else break;
        }

I rewrote the code the following way:

if (n > 0) {
            for (int count = 0; count < n; count++) {
                l1.add(Integer.parseInt(reader.readLine()));
            }

            maximum = l1.get(0);

            for (int a : l1) {
                maximum = Math.max(a, maximum);
            }
            System.out.println(maximum);
        }

That way it all worked.

Nouser

Level 36 , Germany

25 September 2020, 09:29

Read one number that must not be < 0. Let's name it count. Then in a loop read count numbers (all numbers allowed -, +, 0) from the keyboard and determine the max from that read numbers.