Niestety nie działa... Może ktoś próbował w podobny do mojego sposobu bez contains i iteratora.

Pytania dotyczące zadania

Dyskutowane

package pl.codegym.task.task07.task0716;

import java.util.ArrayList;

/*
R czy L
*/

public class Solution {
    public static void main(String[] args) throws Exception {
        ArrayList<String> lista = new ArrayList<String>();
        lista.add("rosa"); // 0
        lista.add("luz"); // 1
        lista.add("lira"); // 2
        lista = napraw(lista);

        for (String s : lista) {
            System.out.println(s);
        }
    }

    public static ArrayList<String> napraw(ArrayList<String> lista) {
        // tutaj wpisz swój kod
        for (int a = 0; a < lista.size(); a++){
            boolean jestR = false;
            boolean jestL = false;
            for (int b = 0; b < lista.get(b).length(); b++){

                if (lista.get(b).substring(b, b + 1).equals("r")){
                    jestR = true;
                }
                else if (lista.get(b).substring(b, b + 1).equals("l")){
                    jestL = true;
                }
            }
            if (jestR == true && jestL == false){
                lista.remove(a);
            }
            else if (jestR = false && jestL == true){
                lista.add(a, (lista.get(a)));
            }
        }
        return lista;
    }
}

1. Utwórz listę wyrazów i sam ją zapełnij.
2. Metoda napraw powinna:
2.1. usuwać z listy wszystkie wyrazy zawierające literę „r”.
2.2. duplikować wszystkie wyrazy zawierające literę "l".
2.3. pozostawiać niezmienione wszystkie wyrazy zawierające zarówno literę "r, jak i "l".
2.4. nic nie robić z pozostałymi wyrazami.

Na przykład:
rosa lira luz

Wyjście:
lira luz luz

Wymagania:

Program nie może wczytywać danych z klawiatury.
Metoda napraw powinna usuwać z listy wszystkie wyrazy zawierające literę „r”. Jest jednak wyjątek: wyrazy zawierające zarówno literę „r”, jak i „l” powinny pozostać niezmienione.
Metoda napraw musi duplikować wyrazy zawierające literę „l” (dodaj inną instancję tego słowa do listy). Jest jednak wyjątek: wyrazy zawierające zarówno literę „r”, jak i „l” nie powinny być duplikowane.
Metoda napraw nie powinna nic robić z wyrazami zawierającymi zarówno „l”, jak i „r”.
Metoda napraw nie powinna nic robić z wyrazami, które nie zawierają ani „l”, ani „r”.

package pl.codegym.task.task07.task0716; import java.util.ArrayList; /* R czy L */ public class Solution { public static void main(String[] args) throws Exception { ArrayList<String> lista = new ArrayList<String>(); lista.add("rosa"); // 0 lista.add("luz"); // 1 lista.add("lira"); // 2 lista = napraw(lista); for (String s : lista) { System.out.println(s); } } public static ArrayList<String> napraw(ArrayList<String> lista) { // tutaj wpisz swój kod for (int a = 0; a < lista.size(); a++){ boolean jestR = false; boolean jestL = false; for (int b = 0; b < lista.get(b).length(); b++){ if (lista.get(b).substring(b, b + 1).equals("r")){ jestR = true; } else if (lista.get(b).substring(b, b + 1).equals("l")){ jestL = true; } } if (jestR == true && jestL == false){ lista.remove(a); } else if (jestR = false && jestL == true){ lista.add(a, (lista.get(a))); } } return lista; } }

Komentarze (2)

Popularne
Najnowsze
Najstarsze

Musisz się zalogować, aby dodać komentarz

Guadalupe Gagnon

Poziom 37 , Tampa, United States

24 sierpnia 2020, 15:37

przydatny

#1 The 'b' for loop: This is supposed to check each letter of each word for "r" and "l". The current word is determined by the "a" loop by the lista.get(a), not lista.get(b). Fix this at all points between lines 27-35 #2 What happens when you remove a word from a list? The answer is that all the remaining words shift down 1 index. So if you had a list such as (these could be any words): [0:"word1", 1:"word2", 2:"word3", 3:"word4"] and you removed "word2", then "word3" would get moved down to index 1 and "word4" would get moved down to index 2. Why is this a problem? Lets say your code is running and it gets to "word2", this would mean that the loop counter "a" would equal 1, and "word2" meets the requirements to be removed. Now "word3" would be in index 1 and the loop would conclude and the code would increment "a", making "a" equal to 2. Now the code would pull the word at index 2, which is now "word4", and test it. If you didn't notice, "word3" was skipped. This is a big problem if "word3" is a word that needs to be duplicated or removed. #3 Along the same lines as point 2: Adding words to the middle of a list will bump all following words up 1 index. Take for the above example and instead imagine where you add "word2" instead of removing it. So "a" equals 1, you add "word2" to the index 1, now the list would look like this: [0:"word1", 1:"word2", 2:"word2", 3:"word3", 4:"word4"] The loop would conclude its current cycle, a would be incremented to 2, the word at index 2 is now "word2", and because it met the requirements to be added it would meet these same requirements again, get added, move everything up again, loop would continue, lo and behold "word2" is now in index 3.... and so on and so on. This is the creation of an infinite loop.

Guadalupe Gagnon

Poziom 37 , Tampa, United States

24 sierpnia 2020, 15:42

przydatny

Some tips: The String class has a method called .contains(), where you can pass it a string and it will return true/false if the String contains this. Example:

String test = "zebra";
System.out.println(test.contains("z")); // would output true
System.out.println(test.contains("x")); // would output false
System.out.println(test.contains("ebr")); // would output true

This could help with this task. The method napraw has a return value that is an ArrayList<String>. This means that it must return an ArrayList<String>, but not just the ArrayList that is passed into the arguments. Maybe it would be easier to create a new ArrayList and just add in anything that contains both "r" and "l", contains neither "l" or "r", and add anything twice that contains only "l". This way you don't need to worry about infinite loops and skipping anything in the original list.