в чем ошибка?

for (int i = 0; i<set.size();){ if (set.get(i) > 10){ set.remove(i); set.size()-1; i++ такой код тоже неправильный?

А у множества есть команда get(int index)? Оно же не структурировано. Давайте я вам так помогу. Вот наглядный вариант для тестирования:

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

public class Solution {
    public static Set<Integer> createSet() {
        Set<Integer> numbers = new HashSet<>();
        System.out.println("Добавлены числа:");
        for (int i = 0; i < 20; i++) {
            int randomNumber = (int) (Math.random() * 100);
            numbers.add(randomNumber);
            System.out.print(randomNumber + " ");
        }
        System.out.println();
        return numbers;
    }

    public static Set<Integer> removeAllNumbersGreaterThan10(Set<Integer> set) {
        for (int i = 0; i < set.size(); i++) {
            if (i > 10) {
                set.remove(i);
            }
        }
        return set;
    }

    public static void main(String[] args) {
        Set<Integer> set = removeAllNumbersGreaterThan10(createSet());
        System.out.println("Остались числа:");
        for (Integer i : set) {
            System.out.print(i + " ");
        }
    }
}

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

из Вашего примера, я понял свою ошибку, тогда вот так писать метод можно, он правильный? public static Set<Integer> removeAllNumbersGreaterThan10(Set<Integer> set) { Iterator<Integer> iterator = set.iterator(); while (iterator.hasNext()) { Integer numberfromset = iterator.next(); if (numberfromset > 10) { set.remove(numberfromset); } } return set; } или вот такой for (Integer numbers : set) { if (numbers >= 10) { set.remove(numbers); } } return set; } я прав или нет7

Вы близки к победе. Разберем нюансы. Логика подхода ясна. Но есть тонкость - когда вы в процессе перебираете коллекцию с помощью итератора и одновременно ее модифицируете - получается ConcurrentModificationException. Избежать этого можно если удалять с помощью метода итератора remove() или работать с копией коллекции, а удалять из оригинала.

public static Set<Integer> removeAllNumbersGreaterThan10(Set<Integer> set) {
    Iterator<Integer> iterator = set.iterator();
    while (iterator.hasNext()) {
        Integer numberfromset = iterator.next();
        if (numberfromset > 10) {
            //set.remove(numberfromset); //меняем эту строку
            iterator.remove();
        }
    }
    return set;
}

Эта же ошибка проявляется и во втором варианте. Дело в том, что цикл for-each() производит свои действия с неявным вызовом итератора. Но здесь у вас другая ошибка. В задании сказано удалять числа больше 10.

public static Set<Integer> removeAllNumbersGreaterThan10(Set<Integer> set) {
    for (Integer numbers : set) {
        if (numbers >= 10) {
            set.remove(numbers);
        }
    }
    return set;
}

Есть еще возможность избежать исключения ConcurrentModificationException, если работать с классом Concurrent - например, ConcurrentHashMap, но вам пока рано и, наверно, валидатор это не примет. Тестируйте свой код, у вас всё для этого есть. Не забывайте форматировать код, так его легче читать. И когда размещаете на сайте - оформляйте специальными тегами (кнопка code).

Спасибо! iterator.remove() помогло решить задачу. А вот как : работать с копией коллекции, а удалять из оригинала я не совсем понял...

Создаете копию коллекции. Обращаю внимание - копию, а не вторую переменную на один и тот же объект. Перебираете копию удобным вам способом. Находите "лишние" значения - а удаляете их у оригинала.

вроде все так легко и просто звучит, а на самом деле сложно)

Чтобы было честно и справедливо - вы показываете код, что вы сделали. Если надо - пишите словами, поясняя каждую строчку. А мы - я и все остальные участники комьюнити - смотрим и помогаем. Но работа с вашей стороны должна быть большая, чем с нашей. На работе работу работать всё же вам придется.

Set<Integer> copy = new HashSet<>(set); //создали копию Iterator<Integer> it = copy.iterator(); // итератору передаем копию while (it.hasNext()) { // пробегаемся по циклу if (it.next() > 10) { set.remove(); удаляем элемент, но тут выскакивает ошибка return set; }

Немного исправил и добавил промежуточный вывод. Посмотрите, что исправлено.

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

public class Solution {
    public static Set<Integer> createSet() {
        Set<Integer> numbers = new HashSet<>();
        System.out.println("Добавлены числа:");
        for (int i = 0; i < 20; i++) {
            int randomNumber = (int) (Math.random() * 100);
            numbers.add(randomNumber);
            System.out.print(randomNumber + " ");
        }
        System.out.println();
        return numbers;
    }

    public static Set<Integer> removeAllNumbersGreaterThan10(Set<Integer> set) {
        Set<Integer> copy = new HashSet<>(set); //создали копию
        Iterator<Integer> it = copy.iterator(); //итератору передаем копию
        while (it.hasNext()) { //пробегаемся по циклу
            int i = it.next();
            if (i > 10) {
                set.remove(i); // тут нужно указать, что удаляем
            }
        }
        System.out.println("\ncopy: " + copy);
        System.out.println("set: " + set + "\n");
        return set;
    }


    public static void main(String[] args) {
        Set<Integer> set = removeAllNumbersGreaterThan10(createSet());
        System.out.println("Остались числа:");
        for (Integer i : set) {
            System.out.print(i + " ");
        }
    }
}

Вы в чем работаете? В IDEA? Почему игнорируете очевидные ошибки (красные подчеркивания)? Публикуя код, оформляйте его тэгами.

Спасибо! то есть нужно было в скобки вставить set.remove(i); а можно было просто set.remove(it.next()); без переменной? работаю в идеи.

Нет, каждая команда it.next() берет новое число из множества. Значит в сравнении будет одно, а на удаление пойдет другое.