das ist doch richtig?

Frage zur Aufgabe

In der Diskussion

was ist denn hier der Fehler? der letzte Punkt will nicht

1. Lies 10 Wörter von der Tastatur und füge sie zu einer Liste von Zeichenketten hinzu.
2. Bestimme, ob die Liste nach aufsteigender Zeichenkettenlänge geordnet ist.
3. Wenn dies nicht der Fall ist, zeige den Index des ersten Elements an, das diese Reihenfolge verletzt.

Anforderungen:

Deklariere eine Zeichenkettenlistenvariable und initialisiere sie sofort.
Lies 10 Zeilen von der Tastatur und füge sie zur Liste hinzu.
Wenn die Liste nach aufsteigender Zeichenkettenlänge geordnet ist, musst du nichts anzeigen.
Wenn die Liste nicht nach aufsteigender Zeichenkettenlänge geordnet ist, zeige den Index des ersten Elements an, das diese Reihenfolge verletzt.

package de.codegym.task.task07.task0718; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; import java.util.ArrayList; /* Reihenfolge überprüfen */ public class Solution { public static void main(String[] args) throws IOException { //schreib hier deinen Code BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); ArrayList<String> liste = new ArrayList<String>(); int min = 0; for(int i = 0; i < 10; i++){ liste.add(reader.readLine()); } for(int i = 0; i < 10; i++){ int a = liste.get(i).length(); if(a > min){ min = a; }else{ System.out.println(liste.get(i)); break; } } } }

Kommentare (7)

Beliebt
Neu
Alt

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

Gellert Varga

Level 23 , Szekesfehervar, Hungary

17 Mai 2020, 20:04

1) Line 26: if(a > min) // Are You sure? What if the length of StringList-elements is this: 2, 3, 4, 4, 5, 6, 7, 7, 8, 10, The order is right, doesn't break the right order, but your program do this output: 4 2) Line 29: System.out.println(liste.get(i)); You have to display the index of the element, not the element itself.

hidden #10620349

Level 8

18 Mai 2020, 06:28

right okay, how about that: int min = 0; int counter = 0; for(int i = 0; i < 10; i++){ int a = liste.get(i).length(); counter++; if(a >= min){ min = a; }else{ System.out.println(counter); break; } } now we have the index counter, and the a >= min doesnt change the end result of not accepting the last point, same result with a > min, only the output is diffrent....? why? i dont get it

Gellert Varga

Level 23 , Szekesfehervar, Hungary

18 Mai 2020, 23:15

The point what I wanted to indicate: This condition "if (a> min)" was wrong. You corrected it very well: "if (a>= min)" - it's good! Ascending Order: Only counts as a breakpoint if the next item is SMALLER. If it's the same, it could still be good. The newer next item may be again bigger...

hidden #10620349

Level 8

19 Mai 2020, 07:22

okay but it still doesnt want to accept the last quest.. but why, whats still wrong? can you send me your code?

Gellert Varga

Level 23 , Szekesfehervar, Hungary

19 Mai 2020, 18:47

The 'counter ++' variable was in the wrong place. We increased its value by 1 - but sooner than it should have. Therefore, the output always indicates a greater value than the correct value. I'm sending you a picture here:

Please let me know if this latest version works.

hidden #10620349

Level 8

19 Mai 2020, 18:58

yea, wow, what should i say, its working, thanks! i spent so much time thinking about that

Gellert Varga

Level 23 , Szekesfehervar, Hungary

19 Mai 2020, 19:05

OK, all right.:)