JavaRush/Помощь по Java задачам/Объясните неправильность кода

13 уровень

05.05.2020
388просмотров
16комментариев

Объясните неправильность кода

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

Решен

package com.javarush.task.task08.task0814;

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

/*
Больше 10? Вы нам не подходите
*/

public class Solution {
    public static Set<Integer> createSet() {
        // напишите тут ваш код
        Set<Integer> set = new HashSet<>();
        for (int i = 0; i < 20; i++) {
            set.add(i);
        }

        return set;
    }

    public static Set<Integer> removeAllNumbersGreaterThan10(Set<Integer> set) {
        // напишите тут ваш код
        int threshold = 10;
        for (int i : set) {
            if (i > threshold) set.remove(i);
        }
        return set;
    }
        public static void main (String[]args){

        }
    }

Почему программа не проходит последний пункт?

Создать множество чисел(Set<Integer>), занести туда 20 различных чисел.
Удалить из множества все числа больше 10.

Требования:

Программа не должна выводить текст на экран.
Программа не должна считывать значения с клавиатуры.
Класс Solution должен содержать только три метода.
Метод createSet() должен создавать и возвращать множество Set состоящих из 20 различных чисел.
Метод removeAllNumbersGreaterThan10() должен удалять из множества все числа больше 10.

package com.javarush.task.task08.task0814; import java.util.HashSet; import java.util.Set; /* Больше 10? Вы нам не подходите */ public class Solution { public static Set<Integer> createSet() { // напишите тут ваш код Set<Integer> set = new HashSet<>(); for (int i = 0; i < 20; i++) { set.add(i); } return set; } public static Set<Integer> removeAllNumbersGreaterThan10(Set<Integer> set) { // напишите тут ваш код int threshold = 10; for (int i : set) { if (i > threshold) set.remove(i); } return set; } public static void main (String[]args){ } }

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

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

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

Ksenia Volkova Java Developer в DXC

Master

5 мая 2020, 13:09

Потому что ты удаляешь элементы из коллекции, по которой в это время итерируешься, а это приводит к ошибке. Безопасно удалять во время перебора можно методом remove итератора.

Igor

Уровень 13

5 мая 2020, 13:15

import java.util.Arrays;
import java.util.HashSet;
import java.util.Set;

class Scratch {

        public static void main(String[] args) {
            Set<Integer> ints = new HashSet<>(Arrays.asList(1,2,3));

            ints.remove(1);

            System.out.println(ints);
        }

}

Тут же он не ругается, например, и все работает

Ksenia Volkova Java Developer в DXC

Master

5 мая 2020, 13:21

Потому что тут не перебора в цикле for each, как в коде задачи.

Igor

Уровень 13

5 мая 2020, 13:26

А можно для меня, глупого, объяснить в чем и как возникает ошибка перебора и модификации коллекции одновременно?

Ksenia Volkova Java Developer в DXC

Master

5 мая 2020, 13:31

решение

Когда ты перебираешь элементы коллекции циклом for each, внутри этого процесса используется итератор. У итератора есть курсор, который указывает на текущий элемент. Если при этом ты начнешь менять коллекцию "извне" (не методом итератора), то этот курсор собьется. Это примерно как забираться по лестнице и выдергивать из-под себя ступеньки.

Igor

Уровень 13

5 мая 2020, 13:34

Спасибо большое!

KIRA

Уровень 41

Expert

5 мая 2020, 12:59

В качестве проверки добавьте в метод main

Set<Integer> set = createSet();
        removeAllNumbersGreaterThan10(set);
        for (Integer i : set){
            System.out.println(i);
        }

Только не забудьте удалить перед сдачей задачи

KIRA

Уровень 41

Expert

5 мая 2020, 12:58

решение

Дело в том что нельзя перебирать коллекцию циклом for each и модифицировать ее одновременно. Если вы хотите использовать цикл for each то сделайте копию своего Set

Set<Integer> setcopy = new HashSet<>(set);

После можете перебирать setcopy, а удалять с основного set

Igor

Уровень 13

5 мая 2020, 13:13

import java.util.Arrays;
import java.util.HashSet;
import java.util.Set;

class Scratch {

        public static void main(String[] args) {
            Set<Integer> ints = new HashSet<>(Arrays.asList(1,2,3));

            ints.remove(1);

            System.out.println(ints);
        }

}

Это конкретно для валидатора такое решение? ведь выше код проходит одним for each и модифицирует коллекцию одновременно

KIRA

Уровень 41

Expert

5 мая 2020, 13:19

Тут вы не просматриваете коллекцию, а говорите удалить значение с коллекции. По аналогии в вашей задаче можете написать так

set.remove(10);

и он удалит без всяких проблем

KIRA

Уровень 41

Expert

5 мая 2020, 13:20

нет решение не для валидатора, а потому что нельзя просматривать циклом for each и удалять с этой коллекции. будет ошибка

Михаил

Уровень 17

5 мая 2020, 12:48

т.е. пока есть следующая строка, выполняется тело цикла,дальше берешь следующую строку и если она больше 10 - удаляешь

Михаил

Уровень 17

5 мая 2020, 12:45

Чтобы корректно удалять элементы во время перебора, нужен итератор. Делай через цикл while а не for each

Igor

Уровень 13

5 мая 2020, 12:51

компилятор не ругается, я прохожу по всем элементам данного множества, зачем делать проверку на следующий элемент?

Igor

Уровень 13

5 мая 2020, 13:07

и for each порождает свой итератор

Ksenia Volkova Java Developer в DXC

Master

5 мая 2020, 13:22

Да, но он "под капотом", ты не можешь обратиться к нему напрямую и использовать его методы.