оптимизация java 8 потоков операций

Question

Итак, я старался максимально оптимизировать свой код. Следующий код, используемый для запуска на 5 секунд, однако мне удалось уменьшить его до примерно 1,4 секунды, однако его все еще недостаточно. Что я могу сделать, чтобы оптимизировать этот код еще больше? (Возможно, я должен упомянуть, что время, о котором я говорил, происходит, когда aux Map заканчивается 170080 ключами).

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) сложность и компаратор не слишком налогообложения либо так, что не должно быть проблемой, есть то, что я не знаю, что может помочь мне оптимизировать этот поток операций? Возможно, можно использовать элемент entrySet, который я использую ...? Потому что это, вероятно, самая дорогая операция здесь ...

EDIT1: Может быть, я должен объяснить идею этого метода. Это главная цель состоит в том, чтобы заказать карту всп и возвращает список с ключами упорядоченными (ключи также изменены, но это не главная цель)

Answer 1

Вы могли бы попытаться заменить forEach заявления с map и collect, как описано в Java 8 - Best way to transform a list: map or foreach?

Тогда вы можете попробовать, если использование параллельного потока улучшает вашу производительность.

Возможно, вы можете заменить свою инструкцию, которая создает карту aux с помощью Stream API (с использованием Collectors.toMap и/или Collectors.groupingBy). Это считается более чистым, чем использование forEach, которое использует операции с сохранением состояния.

Есть уже достаточно много вопросов о том, как сделать это https://stackoverflow.com/search?q=groupingBy+[java-stream] или https://stackoverflow.com/search?q=toMap+[java-stream]

Если вам нужно быстрее решение (с меньшим количеством изменений), вы могли бы попытаться заменить Map с ConcurrentHashMap и использовать параллельно поток. Вы можете использовать its merge function, чтобы сделать ваши вычисления параллелизуемыми.