Java Stream: List of objects to HashMap without duplicates

Question

I'm trying to convert a List to Map without duplicates using a stream but I can't achieve it.

I can do it using a simple loop like this:

List<PropertyOwnerCommunityAddress> propertyOwnerCommunityAddresses = getPropertyOwnerAsList();

Map<Community, List<Address>> hashMap = new LinkedHashMap<>();

for (PropertyOwnerCommunityAddress poco : propertyOwnerCommunityAddresses) {

    if (!hashMap.containsKey(poco.getCommunity())) {
        List<Address> list = new ArrayList<>();
        list.add(poco.getAddress());
        hashMap.put(poco.getCommunity(), list);
    } else {
        hashMap.get(poco.getCommunity()).add(poco.getAddress());
    }
}

but when I try to use a stream, my mind crash.

I have to say the PropertyOwnerCommunityAddress contains two object more: Community and Address and the goal of all of this is for each community save the addresses in a key:value pair without duplicate the Community object.

Anyone can help me? Thank you!

Your entire loop body can be replaced with hashMap.computeIfAbsent(poco.getCommunity(), key -> new ArrayList<>()).add(poco.getAddress()); — Holger
– Holger, Commented Jun 10, 2020 at 8:31

Lino · Accepted Answer · 2020-06-10 08:42:55Z

7

As you can have multiple Addresses per Community you can't use the toMap() collector, but you need to use groupingBy():

Map<Community, List<Address>> map = propertyOwnerCommunityAddresses.stream()
    .collect(Collectors.groupingBy(
        PropertyOwnerCommunityAddress::getCommunity,
        Collectors.mapping(
            PropertyOwnerCommunityAddress::getAddress, 
            Collectors.toList())
        )
    );

Depending on your personal preference, this can look messy and maybe more complicated than the simple for-loop, which can also be optimized:

for(PropertyOwnerCommunityAddress poco : propertyOwnerCommunityAddresses) {
    hashMap.computeIfAbsent(poco.getCommunity(), c -> new ArrayList<>()).add(poco.getAddress());
}

Depending if you only want to have unique addresses you may want to use Set, so change the Collectors.toList() to Collectors.toSet() or when you stay with your for-loop change the definition of hashMap to Map<Community, Set<Address>> and in the loop exchange new ArrayList<>() with new HashSet<>()

edited Jun 10, 2020 at 8:42

answered Jun 10, 2020 at 8:30

Lino

20k6 gold badges55 silver badges73 bronze badges

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

1 Comment

Joarte Over a year ago

Thank you! The first stream is exaclty I was looking for!

azro · Accepted Answer · 2020-06-10 08:30:31Z

3

You have to use

groupingBy to get the Community as key
mapping to get the Address as list

Map<Community, List<Address>>  hashMap = propertyOwnerCommunityAddresses.stream()
            .collect(Collectors.groupingBy(PropertyOwnerCommunityAddress::getCommunity,
                    Collectors.mapping(PropertyOwnerCommunityAddress::getAddress, Collectors.toList())));

answered Jun 10, 2020 at 8:30

azro

54.2k9 gold badges39 silver badges75 bronze badges

1 Comment

Joarte Over a year ago

Thank you! Your solution is as valid as @Lino. Thank you for you post suggestions as well

Collectives™ on Stack Overflow

Java Stream: List of objects to HashMap without duplicates

2 Answers 2

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related