Unfulfiled requests

Frage zur Aufgabe

In der Diskussion

I don't know why point 2,3 and 4 aren't fulfilled. I have tried many other options but always without results. Thanks for helping.

1. Erstelle eine Liste von Wörtern und fülle sie selbst aus.
2. Die korrigieren-Methode muss:
2.1. alle Wörter aus der Liste entfernen, die den Buchstaben „r“ enthalten.
2.2. alle Wörter duplizieren, die den Buchstaben „l“ enthalten.
2.3. wenn ein Wort sowohl „r“ als auch „l“ enthält, dann machen wir nichts.
2.4. alle anderen Wörter unangetastet lassen.

Zum Beispiel:
Rose Leier Liebe

Ausgabe:
Leier Liebe Liebe

Anforderungen:

Das Programm darf keine Daten von der Tastatur lesen.
Die korrigieren-Methode muss alle Wörter, die den Buchstaben "r" enthalten, aus der Liste entfernen. Es gibt eine Ausnahme: Wörter, die sowohl "r" als auch "l" enthalten, müssen unangetastet bleiben.
Die korrigieren-Methode muss Wörter mit dem Buchstaben "l" duplizieren (eine weitere Instanz dieses Wortes zur Liste hinzufügen). Es gibt eine Ausnahme: Wörter, die sowohl "l" als auch "r" enthalten, dürfen nicht dupliziert werden.
Die korrigieren-Methode muss Wörter, die sowohl "l" als auch "r" enthalten, unangetastet lassen.
Die korrigieren-Methode muss Wörter, die weder "l" noch "r" enthalten, unangetastet lassen.

package de.codegym.task.task07.task0716; import java.util.ArrayList; /* R oder L */ public class Solution { public static void main(String[] args) throws Exception { ArrayList<String> liste = new ArrayList<>(); liste.add("R"); // 0 liste.add("rl"); // 1 liste.add("rl"); // 2 liste = korrigieren(liste); for(String s: liste) { System.out.println(s); } } public static ArrayList<String> korrigieren(ArrayList<String> liste) { boolean r; boolean l; for(int i=0; i<liste.size();) { r=false; l=false; if(liste.get(i).contains("R")||liste.get(i).contains("r")) {r=true;} if(liste.get(i).contains("L")||liste.get(i).contains("l")) {l=true;} if(r==true&&l==false) {liste.remove(i);}else { if(r==false&&l==true) {liste.add(liste.get(i));i++;} else i++;} } return liste; } }

Kommentare (2)

Beliebt
Neu
Alt

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

Gudrun

Level 10 , Eberbach, Germany

Expert

4 Januar 2022, 15:37

There is no iteration number in the for-loop

Lisa

Level 41

4 Januar 2022, 16:02

That's done on purpose here. I fixed and commented the code and renamed some vars so that you can see what Pablo intended to do.

public static ArrayList<String> korrigieren(ArrayList<String> liste) {
    boolean containsR, // declare two booleans
            containsL;

    for (int i = 0; i < liste.size(); ) {
        containsR = false; // initialize the booleans inside the loop, that way we can reuse them
        containsL = false;

        // does the current list element contain r or R? Save the result in containsR
        if (liste.get(i).contains("R") || liste.get(i).contains("r"))
            containsR = true;
        // does the current list element contain l or L? Save the result in containsL
        if (liste.get(i).contains("L") || liste.get(i).contains("l"))
            containsL = true;

        // if the current element does contain an r and a l or it does not contain an r and a l
        if (containsL && containsR || !containsL && !containsR)  // do nothing, advance to next position in the list
            i++;
        else if (containsR) // contains R, remove current element, don't increment i, stay at this position
            liste.remove(i);
        else { // containsL is true here in any case, add the current element at the current position a 2nd time
                liste.add(i, liste.get(i));
                i++; // now we need to advance two positions as the next would be the current again,
                i++;
        }
    }
    return liste;
}