I seem to be making the same errors

Question about the task

Under discussion

I have the suspicion that I am making the same error that is preventing me from passing several tasks. Can someone help me out? If I can understand why this last task won't validate, then I may be able to complete other tasks as well.

Create a Map<String, Integer> and add ten entries that represent (last name, salary) pairs.
Remove from the map all people whose salary is below 500.

Requirements:

The program should not display text on the screen.
The program should not read values from the keyboard.
The createMap() method must create and return a HashMap that has (String, Integer) elements and contains 10 entries that represent (last name, salary) pairs.
The removeItemFromMap() method must remove from the map all people whose salary is below 500.

package com.codegym.task.task08.task0818; import java.util.HashMap; import java.util.Map; /* Only for the rich */ public class Solution { public static HashMap<String, Integer> createMap() { HashMap<String, Integer> map = new HashMap<>(); for (int i=0; i<10; i++){ map.put("A"+i, i+100); } return map;//write your code here } public static void removeItemFromMap(HashMap<String, Integer> map) { for (Map.Entry<String, Integer> pair : map.entrySet()){ if (pair.getValue()<500) map.remove(pair); removeItemFromMap(map); break; }//write your code here } public static void main(String[] args) { } }

Comments (1)

Popular
New
Old

You must be signed in to leave a comment

Thomas

Level 41 , Bayreuth, Germany

12 June 2020, 01:13

Two problems... first, you forgot brackets (line 23 -25). Without these you'd recursivly call removeItemFromMap() even if there is no value < 500 -> stackoverflow second, the key to remove a map entry is not pair but pair.getKey()