please..

Question about the task

Greater than 10? You're not a good fit for us

Under discussion

please give me ans ???

package com.codegym.task.task08.task0814; import java.util.HashSet; import java.util.Set; /* Greater than 10? You're not a good fit for us */ public class Solution { public static HashSet<Integer> createSet() { HashSet<Integer> set = new HashSet<Integer>(); for(int i=0;i<20;i++) { set.add(i+1); } return set; } public static HashSet<Integer> removeAllNumbersGreaterThan10(HashSet<Integer> set) { Iterator<Integer> itr = Set.Iterator(); while(itr.hasNext()) if(itr.next() > 10) itr.remove(); } public static void main(String[] args) { } }

Comments (7)

Popular
New
Old

You must be signed in to leave a comment

KARAN BHOITE

Level 15 , Nagpur, India

8 August 2019, 01:26

getting this error com/codegym/task/task08/task0814/Solution.java:23: error: cannot find symbol Iterator<Integer> itr = Set.Iterator(); ^ symbol: class Iterator location: class com.codegym.task.task08.task0814.Solution

Kotlet

Level 28 , Wroclaw, Poland

8 August 2019, 13:58

In line 23 try set.iterator(); instead of Set.Iterator(); Also remember to return something out of removeAllNumbersGreaterThan10 method (hint HashSet<Integer>). Now your method is missing return.

filchnodead

Level 15

8 August 2019, 17:46

Let me give you advice. Use "for each" loop for iteration. For example, this code:

Iterator<Integer> itr = Set.Iterator();
     while(itr.hasNext())
      if(itr.next() > 10)
          itr.remove();
    }

may be replaced by that:

for (int i : set) {
if (i > 10) {
set.remove(i);
}
}

That's more convenient, isn't it?)

filchnodead

Level 15

8 August 2019, 17:49

What about your problem. 1) removeAllNumbersGreaterThan10 should be "void" 2) In the 23 line you should use

set.Iterator()

instead of

Set.Iterator();

KARAN BHOITE

Level 15 , Nagpur, India

9 August 2019, 02:36

yes i did it but still getting error same error

KARAN BHOITE

Level 15 , Nagpur, India

9 August 2019, 02:54

i did this now but still last condition not met {Be sure that the removeAllNumbersGreaterThan10 method works with sets of any size.} public class Solution { public static HashSet<Integer> createSet() { HashSet<Integer> set = new HashSet<Integer>(); for(int i=0;i<20;i++) { set.add(i+1); } return set; } public static HashSet<Integer> removeAllNumbersGreaterThan10(HashSet<Integer> set) { for (int i:set) { if(i>10) {set.remove(i); } } return set; } public static void main(String[] args) { } }

filchnodead

Level 15

10 August 2019, 08:27

Your mistake is that you modify the original set, you shouldn't do that. You gotta create a new set and return it. There is an example:

public static HashSet<Integer> removeAllNumbersGreaterThan10(HashSet<Integer> set) {
        HashSet<Integer> result = new HashSet<>();
        for (int i : set) {
            if (i <= 10) {
                result.add(i);
            }
        }

        return result;
    }