Maneira mais limpa de indexar uma coleção por uma propriedade do item que é uma coleção

Question

eu tenho um List<Foo> e quero uma goiabada Multimap<String, Foo> onde agrupamos os Fooé por cada tag de seu Collection<String> getTags() função.

Estou usando Java 8, portanto, lambdas e referências de método são boas/encorajadas.

Por exemplo, se eu tiver:

foo1, tags=a,b,c
foo2, tags=c,d
foo3, tags=a,c,e

eu conseguiria um Multimap<String, Foo> com:

a -> foo1, foo3
b -> foo1
c -> foo1, foo2, foo3
d -> foo2
e -> foo3

Answer 1

Você pode usar o coletor personalizado para isso:

Multimap<String, Foo> map = list.stream().collect(
    ImmutableMultimap::builder,
    (builder, value) -> value.getTags().forEach(tag -> builder.put(tag, value)),
    (builder1, builder2) -> builder1.putAll(builder2.build())
).build();

Isto não causa efeitos colaterais extras (ver aqui sobre isso), é concorrente e mais idiomático.

Você também pode extrair esses lambdas ad-hoc em um coletor completo, algo assim:

public static <T, K> Collector<T, ?, Multimap<K, T>> toMultimapByKey(Function<? super T, ? extends Iterable<? extends K>> keysMapper) {
    return new MultimapCollector<>(keysMapper);
}

private static class MultimapCollector<T, K> implements Collector<T, ImmutableMultimap.Builder<K, T>, Multimap<K, T>> {
    private final Function<? super T, ? extends Iterable<? extends K>> keysMapper;

    private MultimapCollector(Function<? super T, ? extends Iterable<? extends K>> keysMapper) {
        this.keysMapper = keysMapper;
    }

    @Override
    public Supplier<ImmutableMultimap.Builder<K, T>> supplier() {
        return ImmutableMultimap::builder;
    }

    @Override
    public BiConsumer<ImmutableMultimap.Builder<K, T>, T> accumulator() {
        return (builder, value) -> keysMapper.apply(value).forEach(k -> builder.put(k, value));
    }

    @Override
    public BinaryOperator<ImmutableMultimap.Builder<K, T>> combiner() {
        return (b1, b2) -> b1.putAll(b2.build());
    }

    @Override
    public Function<ImmutableMultimap.Builder<K, T>, Multimap<K, T>> finisher() {
        return ImmutableMultimap.Builder<K, T>::build;
    }

    @Override
    public Set<Characteristics> characteristics() {
        return Collections.emptySet();
    }
}

Então a coleção ficaria assim:

Multimap<String, Foo> map = list.stream().collect(toMultimapByKey(Foo::getTags));

Você também pode retornar EnumSet.of(Characteristics.UNORDERED) de characteristics() método se o pedido não for importante para você.Isto pode fazer com que as máquinas de recolha interna funcionem de forma mais eficiente, especialmente em caso de redução paralela.

Answer 2

ImmutableMultimap.Builder<String, Foo> builder = ImmutableMultimap.builder();
list.forEach(foo -> foo.getTags().forEach(tag -> builder.put(tag, foo));
return builder.build();

Answer 3

Isso é um pouco mais idiomático para fluxos Java 8:

    Multimap<String, Foo> map = list.stream()
            //First build a stream of Pair<String, Foo>
            .flatMap(f -> f.tags.stream().map(s -> new AbstractMap.SimpleImmutableEntry<>(s, f)))
            //Then collect it up into a multimap.
            .collect(
                    Multimaps.toMultimap(
                            x -> x.getKey(),
                            x -> x.getValue(),
                            MultimapBuilder.hashKeys().arrayListValues()::build
                    )
            );

Eu sei que o uso de uma classe pair é feio, mas a principal coisa que eu queria mostrar é a Multimaps.toMultmap colecionador.Há também um Multmaps.flatteningToMultimap para outros casos de uso.