was ist da nicht gut?

Frage zur Aufgabe

Gelöst

Das Programm funktioniert, die Zweitletzte Bedingung wird allerdings nicht erfüllt? warum denn das?

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.*; /* Aufgabe zu Algorithmen */ public class Solution { public static void main(String[] args) throws Exception { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); int max = 0; int amount = 0; int a = Integer.parseInt(reader.readLine()); if(a > 0){ while(amount < a){ int input = Integer.parseInt(reader.readLine()); amount++; if(input > max) max = input; if(amount == a && amount > 0){ System.out.println(max); break; } } } } }

Kommentare (10)

Beliebt
Neu
Alt

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

Shuy

Level 9 , Frankfurt, Germany

13 Mai 2020, 10:48

Also eine negative Zahl darf erstmal gar nicht eingegeben werden laut dem 1. Punkt, daher musst du eine Abfrage einsetzen die die Eingabe nicht annimmt, wenn sie kleiner als 0 ist. Ich würde deine 2. If Abfrage erstmal aus der while schleife packen, da die while schleife die Aufgabe hat die Eingaben aufzunehmen und den max value zu ändern. Sobald diese aus der While Schleife ist, brauchst du eigentlich gar nicht mehr eine If Abfrage für die Ausgabe der größten Zahl, da sobald du a>0 hast und die Eingaben reinkommen, muss ein max ausgegeben werden. amount == a wird dann auch obsolet, da diese Bedingung erfüllt ist, sobald man aus der While Schleife raus ist. amount > 0 ebenso, sonst würde man in der While Schleife stecken beleiben. if{ while{ if{ } //Für max value änderung } //Ende WhileSchleife Ausgabe } //Ende des If Blocks

Thomas

Level 41 , Bayreuth, Germany

13 Mai 2020, 11:43

lies dir die Bedingungen nochmal durch 1. liest eine Zahl N (muss größer sein als 0) von der Konsole ein 2. liest N Zahlen von der Konsole ein erste Zahl N größer 0 -> danach N zahlen einlesen, da steht nichts mehr von wegen >0. Das erste >0 ist auch logisch, ne negative Anzahl von Zahlen kann man nicht einlesen.

Shuy

Level 9 , Frankfurt, Germany

13 Mai 2020, 13:33

Mein erster Satz war auch auf das N bezogen, der Fehler an sich ist es ja auch max direkt eine 0 zuzuweisen. Ob die Überprüfung jetzt verlangt das der Fall ausgeschlossen wird, dass N nicht negativ sein darf, weiß ich nicht aber würde ich persönlich mit einer Abfrage absichern. Der Rest meines Textes weißt auch nur auf paar Unnötige Verschachtelungen hin.

Thomas

Level 41 , Bayreuth, Germany

13 Mai 2020, 13:44

;) dann sind wir uns ja einig, und er sollte eigentlich selbst herausfinden, daß max = 0 der Fehler ist. Deswegen mein Vorschlag, drei negative Zahlen auszuprobieren. Die erste Bedingung hat er übrigens erfüllt (Zeile 19). Und wie von dir erwähnt ist die zweite if Abfrage nicht notwendig. Funktionieren wird es trotzdem. Behält er diese Abfrage ist wiederum das break redundant (und auch das amount > 0). Schön ist das aber wirklich nicht. Bis bald und sorry, dass ich dich falsch verstanden hab

Thomas

Level 41 , Bayreuth, Germany

13 Mai 2020, 06:25

Die Zahlen, die zur Berechnung des Maximums herangezogen werden, dürfen auch negativ sein. Das haben die Bedingungen nicht ausgeschlossen. Wenn man bei deinem Code z.B. drei negative Zahlen eingibt, ist das Maximum 0.

hidden #10620349

Level 8

13 Mai 2020, 14:19

es wird ja kein maximum berechnet, lediglich die grösste zahl wiedergegeben. Under der letzte punkt sagt doch wenn N kleiner gleich 0 ist darf nichts angezeigt werden?

Thomas

Level 41 , Bayreuth, Germany

13 Mai 2020, 14:27

Under der letzte punkt sagt doch wenn N kleiner gleich 0 ist darf nichts angezeigt werden? Erste Eingabe: N -> darf nicht keiner als 0 sein. Jetzt N Zahlen eingeben -> die dürfen kleiner als 0 sein. es wird ja kein maximum berechnet, lediglich die grösste zahl wiedergegeben. Dann vergiss, dass ich berechnen gesagt habe und mache daraus: die eingegebenen Zahlen, aus denen die Größte angezeigt werden soll, dürfen auch negativ sein.

hidden #10620349

Level 8

13 Mai 2020, 15:09

ja ich hatte gerade ein kleines Chaos mit den Kommentaren. also ich bin zu dämlich um das zu lösen. wie soll max denn aussehen er muss ja irgend ein wert initialisiert bekommen

Thomas

Level 41 , Bayreuth, Germany

13 Mai 2020, 16:17

könntest ja als max einen (den ersten Wert) der N einzulesenden Zahlen nehmen. Muß ja mindestens einer sein. Was man sonst noch machen kann lernst du später ;)

hidden #10620349

Level 8

13 Mai 2020, 16:25

ich habs mittlerweile herausgefunden :) BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); int max = N; int amount = 0; int N = Integer.parseInt(reader.readLine()); if(N > 0){ while(amount < N){ int input = Integer.parseInt(reader.readLine()); amount++; if(input > max){ max = input; }else if(max == 0 && input < max){ max = input; } if(amount == N){ break; } } } if(N > 0 && max > 0 || max < 0){ System.out.println(max); } aesthetisch wahrscheinlich zum kotzen und überkompliziert, ABER es funktioniert :D danke für deine hilfe! ohne dich wäre ich noch in lektion 2