优化java 8流操作

Question

    public List<String> getProdutosMaisCompradosQuantidade(int filial, int X){
    Map<String, ProdutoFilial> aux;

    if(filial==0) {
        aux = new HashMap<>(ValoresFixos.CATALOGO_PRODUTOS_TAMANHO_INICIAL);
        filiais.stream()
               .forEach( (f) -> {
                    Map<String, ProdutoFilial> aux2 = f.getMapProdutosDadosFilialSemEncapsulamento();
                    aux2.forEach((k,t) -> {
                            if(t.getQuantidade()>0){
                                if(aux.containsKey(k)) aux.get(k).atualizarValores(t);
                                else aux.put(k,t);
                            }
                    });
                });
    }
    else aux = filiais.get(filial-1).getMapProdutosDadosFilialSemEncapsulamento();

    List<String> list = 
       aux
       .entrySet()
       .stream()
       .sorted(new ComparadorProdutoQuantidade())
       .map(e -> e.getKey()+"\n     |   o Quantidade: "+e.getValue().getQuantidade()+"; Comprado por "+e.getValue().getNumeroCompradores()+" Clientes Distintos")
       .collect(Collectors.toList());

    if(X>list.size()) X = list.size();
    list.subList(X, list.size()).clear();

    return list;

}

我在这里使用的所有方法几乎都是O(1)复杂度，比较器也不太费力，所以这不应该是问题，有什么我可能不知道的东西可以帮助我优化这个流操作吗？也许我用的入口集可以避免...？因为这可能是这里最贵的手术...

编辑1：也许我应该解释一下这个方法背后的想法。它的主要目的是对地图aux进行排序，并返回一个带有排序键的列表（键也被修改了，但这不是主要目的）

Answer 1

您可以尝试用map和collect替换foreach语句，如Java8中所述--转换列表的最佳方式：map还是foreach？

然后，您可以尝试使用并行流是否可以提高性能。

可以用流API替换创建aux映射的语句（使用collectors.tomap和/或collectors.groupingby）。这被认为比使用foreach更干净，后者使用有状态操作。