Java 8: Karte Lambda Ausdruck

Question

Ich habe ein Map<String, List<Object>> multiFieldMap und ich muss itereate über seinen Wert den Wert auf multiFieldsList wie unten

public List<Object> fetchMultiFieldsList() { 
    List<Object> multiFieldsList = new ArrayList<Object>(); 
    for (Entry<String, List<Object>> entry : multiFieldMap.entrySet()) { 
     String entityName = entry.getKey(); 
     List<Object> ids = entry.getValue(); 
     for (Object id : ids) { 
      Object entity = queryService.query(entityName, queryService.property("id").eq(id)); 
      multiFieldsList.add(entity); 
     } 
    } 
    return multiFieldsList; 
} 

Am fragen, kann dieses Verfahren noch weiter vereinfacht werden gesetzt und hinzufügen?

Answer 1

Sie die Streams API verwenden können:

List<Object> multiFieldsList = 
    multiFieldMap.entrySet() 
       .stream() 
       .flatMap(e -> e.getValue() 
           .stream() 
           .map(o -> queryService.query(e.getKey(), queryService.property("id").eq(o)))) 
       .collect(Collectors.toList()); 

Answer 2

Sie können tatsächlich einen Stream verwenden, um Ihre innere Schleife zu vereinfachen.

können Sie ersetzen:

List<Object> ids = entry.getValue(); 
for (Object id : ids) { 
    Object entity = queryService.query(entityName, queryService.property("id").eq(id)); 
    multiFieldsList.add(entity); 
} 

mit:

entry.getValue().map(
    id -> queryService.query(entityName, queryService.property("id").eq(id)) 
).forEach(multiFieldsList::add); 

Aber Sie gewinnen nicht wirklich viel, dass aus. Ihre Wahl ...

Siehe @ Erans Antwort für eine "Full-Stream" -Lösung.

Answer 3

Sie können es wie folgt vereinfachen:

public List<Object> fetchMultiFieldsList() { 
    List<Object> multiFieldsList = new ArrayList<>(); 
    multiFieldMap.forEach((entityName, ids) -> 
     ids.forEach(id -> multiFieldsList.add(
      queryService.query(entityName, queryService.property("id").eq(id))) 
     ) 
    ); 
    return multiFieldsList; 
} 

Sofern Sie die Stream API verwenden möchten, die Methode Map.forEach könnte der größte Gewinn sein in Bezug auf Code-Vereinfachung, da Sie nicht mit Map.Entry und seine generische Signatur mehr zu tun haben müssen ...