Dlaczego nie działa?

Pytania dotyczące zadania

R czy L

Składnia Java, Poziom 7, Lekcja 9

Dyskutowane

Czemu to nie działa? w Intellij wszystko śmiga, a jak wysyłam zadanie nagle wyskakuje mi błąd

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> listaDoNaprawy){ for (String s : listaDoNaprawy) { if (!s.contains("r") && s.contains("l")) { listaDoNaprawy.add(s); } if (s.contains("r") && !s.contains("l")) { listaDoNaprawy.remove(s); } } return listaDoNaprawy; } }

Komentarze (4)

Popularne
Najnowsze
Najstarsze

Musisz się zalogować, aby dodać komentarz

Przemo

Poziom 15 , Legnica, Poland

3 lutego, 12:57

Bo nie można jednocześnie iterować przez listę i jednocześnie coś w niej usuwać. Po to są te zadania, żebyś to zrozumiał. Każdy na początku robi ten błąd i o to chodzi. Robisz listę zapasową albo szukasz innego rozwiązania. Też przy tym zadaniu spędziłem kupę godzin.

Thomas

Poziom 41 , Bayreuth, Germany

5 lutego, 11:41

You can iterate over a collection and modify that collection. You just can't use an iterator to go through the collection and use other than the iterators methods to modify the collection. A for-each loop internally uses an iterator. So you would have to use the iterators remove method. But the for-each loop does not pass it through, doesn't make that remove method accessible.

List<String> source = new ArrayList<>(Arrays.asList("one", "two", "three"));
List<String> collection;

// ok
collection = new ArrayList<>(source);
System.out.println(collection);
for (int i = collection.size() - 1; i >= 0 ; i--) {
    collection.remove(i);
}
System.out.println(collection);

// ok, the same just using while and isEmpty
collection = new ArrayList<>(source);
System.out.println(collection);
while(!collection.isEmpty())
    collection.removeFirst();
System.out.println(collection);

// ok
collection = new ArrayList<>(source);
System.out.println(collection);
Iterator<String> it = collection.iterator();
while (it.hasNext()) {
    it.next();
    it.remove();
}
System.out.println(collection);

// nok  -> ConcurrentModificationException
collection = new ArrayList<>(source);
System.out.println(collection);
for (String s : collection) { // bang
    collection.remove(s);
}

So you of course can make a copy of the source collection, iterate over that copy and delete from the source. But it is not really necessary.

kacper DevOps w Capgemini

30 stycznia, 19:57

hej, robisz "for each" a to głównie wykorzystuje się do wyświetlenia lub sprawdzenia danych (nie modyfikujesz listy). Jak zrobisz kopię listy i zastąpisz w for(....) to powinno zadziałać.

Thomas

Poziom 41 , Bayreuth, Germany

29 stycznia, 09:30

Probably because this code may throw a ConcurrentModificationException