Почему не работает Remove(Цикл в цикле)?

Вопрос по задаче

Решен

Решение уже есть, если раскоментить строчку, а цикл закоментить)) ВОПРОС! Почему подобные конструкции не работают?

for (Iterator<Object> iterPets = pets.iterator(); iterPets.hasNext();) {
            Object elemPets = iterPets.next();
            for (Iterator<Cat> iterCats = cats.iterator(); iterCats.hasNext();){
                if (elemPets.equals(iterCats.next()))
                    pets.remove(elemPets);
            }
        }

ИЛИ

for (Iterator<Object> iterPets = pets.iterator(); iterPets.hasNext();) {
            for (Iterator<Cat> iterCats = cats.iterator(); iterCats.hasNext();){
                if (iterPets.next().equals(iterCats.next()))
                    pets.remove(iterPets.next());
            }
        }

1. Внутри класса Solution создать public static классы Cat, Dog без конструктора или с конструктором без параметров.
2. Реализовать метод createCats, который должен возвращать множество с 4 котами.
3. Реализовать метод createDogs, который должен возвращать множество с 3 собаками.
4. Реализовать метод join, который должен возвращать объединенное множество всех животных - всех котов и собак.
5. Реализовать метод removeCats, который должен удалять из множества pets всех котов, которые есть в множестве cats.
6. Реализовать метод printPets, который должен выводить на экран всех животных, которые в нем есть.
Каждое животное с новой строки.

Требования:

Программа должна выводить текст на экран.
Внутри класса Solution должны быть созданы public static классы Cat, Dog.
Метод createCats() класса Solution должен возвращать множество (Set) содержащее 4 кота.
Метод createDogs() класса Solution должен возвращать множество (Set) содержащее 3 собаки.
Метод join() класса Solution должен возвращать объединенное множество всех животных - всех котов и собак.
Метод removeCats() должен удалять из множества pets всех котов, которые есть в множестве cats.
Метод printPets() должен выводить на экран всех животных, которые в нем есть. Каждое животное с новой строки.

package com.javarush.task.task08.task0820; import java.util.HashSet; import java.util.Iterator; import java.util.Set; public class Solution { public static void main(String[] args) { Set<Cat> cats = createCats(); Set<Dog> dogs = createDogs(); Set<Object> pets = join(cats, dogs); printPets(pets); removeCats(pets, cats); printPets(pets); } public static Set<Cat> createCats() { Set<Cat> result = new HashSet<Cat>(); for (int i = 0; i < 4; i++) { result.add(new Cat()); } return result; } public static Set<Dog> createDogs() { Set <Dog> result = new HashSet<Dog>(); for (int i = 0; i < 3; i++) { result.add(new Dog()); } return result; } public static Set<Object> join(Set<Cat> cats, Set<Dog> dogs) { Set <Object> pets = new HashSet<Object>(); pets.addAll(cats); pets.addAll(dogs); return pets; } public static void removeCats(Set<Object> pets, Set<Cat> cats) { //pets.removeAll(cats); for (Iterator<Object> iterPets = pets.iterator(); iterPets.hasNext();) { Object elemPets = iterPets.next(); for (Iterator<Cat> iterCats = cats.iterator(); iterCats.hasNext();){ if (elemPets.equals(iterCats.next())) pets.remove(elemPets); } } } public static void printPets(Set<Object> pets) { for (Iterator <Object> iterPets = pets.iterator(); iterPets.hasNext();){ System.out.println(iterPets.next()); } } static class Cat{ } static class Dog{ } }

Комментарии (4)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Ksenia VolkovaJava Developer в DXC

Master

15 июля 2020, 19:37

решение

Потому что когда перебираешь элементы коллекции итератором, то и удалять надо методом remove итератора, а не коллекции.

Oleg Chilikin QA Automation Engineer

15 июля 2020, 20:26

Согласен, заработало, тогда не понятно, почему в предыдущем задании iterator.remove(); не срабатывал, только так:

public static void main(String[] args){
        Set<Cat> cats = createCats();
        Iterator <Cat> iterator = cats.iterator();
        cats.remove(iterator.next());
        printCats(cats);
    }

а выдавал:

Exception in thread "main" java.lang.IllegalStateException
  at java.base/java.util.HashMap$HashIterator.remove(HashMap.java:1505)
  at olegChilikin.practice.Solution.main(Solution.java:16)

Process finished with exit code 1

Говорилось, о том, что данный Exception вылетает из- за того, что мы пытаемся удалить не просто Set <String> или Set <Integer>, а Set <Cat>

import java.util.HashSet;
import java.util.Iterator;
import java.util.Set;

/*
Set из котов
*/

public class Solution {

    public static void main(String[] args){
        Set<Cat> cats = createCats();
        for (Iterator <Cat> iterator = cats.iterator(); iterator.hasNext();){
            iterator.remove();
            break;
        }
        printCats(cats);
    }

    public static Set<Cat> createCats() {
        Set <Cat> cats = new HashSet<>();
        cats.add(new Cat());
        cats.add(new Cat());
        cats.add(new Cat());
        return cats;
    }

    public static void printCats(Set<Cat> cats) {
        for (Iterator <Cat> iterator = cats.iterator(); iterator.hasNext();){
            System.out.println(iterator.next().name);
        }
    }

    public static class Cat{
        String name;
        public Cat(){
            this.name = "Barsik";
        }
    }
}

Ksenia VolkovaJava Developer в DXC

Master

15 июля 2020, 21:12

решение

Потому что ты не вызывал команду next(), поэтому у тебя итератор фактически элементы не перебирал.

for (Iterator <Cat> iterator = cats.iterator(); iterator.hasNext();){
    iterator.next();
    iterator.remove();
    break;
}

Oleg Chilikin QA Automation Engineer

16 июля 2020, 09:49

Теперь разобрался, Благодарю))