java8 简化到映射的转换

Question

我有这样的类结构：public class A {    private List<B> bs;...//getters}public class C {    private Long id;...//getters}public class B {    private Long idOfC;...//more stuff}B：：getIdOfC matchs C：：getId在更好的设计中，B将只包含对C的引用，而不是它的id（我无法更改它），所以这就是为什么现在我需要创建一个映射，所以我的方法签名看起来像这样public Map<A, List<C>> convert(Collection<A> collection)在这个转换方法中，有一个List<C> getCsByIds(List<Long> id) 后来用于将其与B.idOfC匹配，但是应该只有一次对此方法的调用，因为它非常昂贵。所以如果我这样去： List<B> bs = Arrays.asList(new B(10L), new B(11L)); //10L and 11L are the values of idOfC   List<A> as = Arrays.asList(bs);   //And assuming getCsByIds returns Arrays.asList(new C(10L), new C(11L), new C(12L));然后    Map<A, List<C>> map = convert(as);    map.values().get(0) 返回类似的东西Arrays.asList(new C(10L), new C(11L))在我看来，这样做的方法非常庞大：    public Map<A, List<C>> convert(Collection<A> as) {    List<Long> cIds = as.stream()            .flatMap(a -> a.getBs().stream())            .map(B::getCId)            .collect(Collectors.toList());    //single call to gsCsByIds    Map<Long, C> csMap = getCsByIds(cIds)            .stream()            .collect(Collectors.toMap(C::getId, Function.identity()));    //a whole new map is created by iterating over the list called "as"    Map<A, List<C>> csByAs = new HashMap<>();    if (!csMap.isEmpty()) {        for (A a : as) {            Set<C> cs = getCsFromMap(csMap, a.getBs());            if (!cs.isEmpty()) {                csByAs.put(a, new ArrayList<>(cs));            }        }    }    return csByAs;}private Set<B> getCsFromMap(Map<Long, C> cMap, List<B> bs) {    return bs.stream()            .map(b -> cMap.get(b.getIdOfc()))            .collect(Collectors.toSet());}有没有办法让它更简单???

Answer 1

如果调用成本很高，那么您最初的想法可以自行执行。它可以进一步缩短为：getCsByIds

public Map<A, List<C>> convert(Collection<A> as) {

    List<Long> cIds = as.stream()

            .flatMap(a -> a.getBs().stream())

            .map(B::getIdOfC)

            .collect(Collectors.toList());

    Map<Long, C> csMap = getCsByIds(cIds).stream()

            .collect(Collectors.toMap(C::getId, Function.identity()));

    return as.stream()

            .collect(Collectors.toMap(Function.identity(),

                    a -> a.getBs().stream().map(b -> csMap.get(b.getIdOfC()))

                            .collect(Collectors.toList()), (a, b) -> b));

}

您可以在其中相应地选择合并功能。(a,b) -> b

Answer 2

也许只是直接迭代 As？（手头没有编译器，所以也许代码段还没有编译就绪）

public Map<A, List<C>> convert(Collection<A> as) {

  Map<A, List<C>> result = new HashMap<>();

  for(A a: as){

     List<Long> cIds = a.getBs().stream()

                         .map(B::getIdOfC)

                         .collect(Collectors.toList());

     result.put(a, getCsByIds(cIds));

  }

  return result;

}

Answer 3

这样的事情难道行不通吗？我没有编译器，所以我无法真正测试它

public Map<A, List<C>> convert(Collection<A> as) {

    return as.stream()

             .collect(Collectors.toMap(Function::identity,

                                       a -> a.getBs().stream()

                                                     .map(B::getIdOfC)

                                                     .flatMap(id -> getCsByIds(asList(id))

                                                                   .values()

                                                                   .stream())

                                                     .collect(Collectors.toList())

                                      )

                     );

}