java 8 stream API filter out empty value from map

Question

i have a map, need to operate on each entry's value, and return the modified map. I managed to get it working, but the resulted map contains entries with empty value, and I want to remove those entries but cannot with Java 8 stream API.

here is my original code:

Map<String, List<Test>> filtered = Maps.newHashMap();
for (String userId : userTests.keySet()) {
    List<Test> tests = userTests.get(userId);
    List<Test> filteredTests = filterByType(tests, supportedTypes);

    if (!CollectionUtils.isEmpty(filteredTests)) {
        filtered.put(userId, filteredTests);
    }
}
return filtered;

and here is my Java 8 stream API version:

userTests.entrySet().stream()
         .forEach(entry -> entry.setValue(filterByType(entry.getValue(), supportedTypes)));

userTests.entrySet().stream().filter(entry -> !entry.getValue().isEmpty());
        return userTests;

how can i remove entries with empty/null value from the map?
is there better way to write the code in stream API, so far I don't see it's better than my original code

Where is the definition of the filterByType method ?

Dimitri
– Dimitri

2017-11-17 16:37:16 +00:00
Commented Nov 17, 2017 at 16:37 — Dimitri
– Dimitri, Commented Nov 17, 2017 at 16:37

Brian Agnew · Accepted Answer · 2017-11-17 16:40:16Z

4

You need to collect into a new Map (say)

e.g.

 new HashMap<String, List<String>>().
                entrySet().
                stream().
                filter(entry -> !entry.getValue().isEmpty()).
                collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));

As it currently stands you're simply returning a stream with the intermediate (filtering) operations. The terminal operation will execute this and give you the desired collection.

answered Nov 17, 2017 at 16:40

Brian Agnew

273k38 gold badges342 silver badges443 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Anton Balaniuc · Accepted Answer · 2017-11-17 16:37:46Z

2

userTests.entrySet().stream().filter(entry -> !entry.getValue().isEmpty()); this has no effect. filter is not a terminal operation.

You need to collect the stream result into a new map:

HashMap<String, String> map = new HashMap<>();
map.put("s","");
map.put("not empty", "not empty");

Map<String, String> notEmtpy = map.entrySet().stream()
     .filter(e -> !e.getValue().isEmpty())
     .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));

answered Nov 17, 2017 at 16:37

Anton Balaniuc

11.8k2 gold badges40 silver badges53 bronze badges

Comments

Zach · Accepted Answer · 2017-11-17 16:44:46Z

0

Try inserting the filter in-line:

userTests.entrySet().stream().filter(entry -> !entry.getValue().isEmpty())
.forEach(entry -> entry.setValue(filterByType(entry.getValue(), supportedTypes)));

answered Nov 17, 2017 at 16:44

Zach

112 bronze badges

Collectives™ on Stack Overflow

java 8 stream API filter out empty value from map

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related