Jak iterować zagnieżdżone dla pętli odnoszące się do elementów nadrzędnych za pomocą Java 8 Streams i Lambdas?

Question

Mam poniższy fragment kodu.

public static ModuleKey getDeployableModuleFromModulesList(List<Module> modules) { 
     ModuleKey deployableModuleKey = null; 
     for(Module module : modules) { 
      List<Artifact> artifacts = module.getArtifacts(); 
      for(Artifact artifact : artifacts) { 
       if(artifact.getType().equals("ear")) { 
        return module.getKey(); 
       } else if(!artifact.getType().equals("ear")) { 
        if(artifact.getType().equals("war")) { 
         deployableModuleKey = module.getKey(); 
        } 
       } 
      } 
     } 
     return deployableModuleKey; 
    } 

Kod ten znajduje pierwsze '' z deployableModuleKey key = 'ucho' czy ostatni z key = 'wojna'. Chcę uzyskać tę samą funkcjonalność co powyżej, używając strumieni Java 8 i Lambdas.

tej pory co próbowałem to:

modules.stream().flatMap(e -> e.getArtifacts().stream()) 
    .filter(e -> e.getType().equals("ear")).findFirst() 
    .orElseGet(() -> modules.stream().flatMap(e -> e.getArtifacts().stream()) 
    .filter(e -> e.getType().equals("war")).reduce((a, b) -> b).orElse(null)); 

Powyższy fragment kodu zwróci obiekt typu Artifact zamiast Module. Chcę uzyskać ten moduł, w którym artefakt pasuje do warunków. Po znalezieniu modułu mogę zwrócić klucz, wykonując module.getKey(). Aby to zrobić, chciałbym wiedzieć, w jaki sposób odsyła się elementy pętli rodzica.

Nie jestem pewien, czy mój kod Java 8 jest całkowicie poprawny.
Czy ktoś może mi pomóc w tym zakresie?

Answer 1

Wymyśliłem rozwiązanie mojego problemu.

public static ModuleKey getDeployableModuleFromModulesList(List<Module> modules) { 
    Optional<ModuleKey> op = modules.stream().filter(module -> module.getArtifacts().stream().anyMatch(artifact -> artifact.getType().equals("ear"))).map(module -> module.getKey()).findFirst(); 
    if (!op.isPresent()) { 
     op = modules.stream().filter(module -> module.getArtifacts().stream().anyMatch(artifact -> artifact.getType().equals("war"))).map(module -> module.getKey()).reduce((a, b) -> b); 
    } 
    return op.orElse(null); 
} 

W każdym razie, inne rozwiązania również są mile widziane. W przypadku, gdy ktoś doda inne rozwiązanie, wypróbuję je wszystkie.

Answer 2

Rzeczą w przypadku strumieni jest, że po zmodyfikowaniu strumienia przy użyciu operacji pośredniej, takiej jak filter lub map, nie można odwołać się do wartości z poprzedniej operacji. Musisz więc znaleźć obejście, aby zawsze zachować (w twoim przypadku Module) wartość, do której chcesz się odnosić w swoim strumieniu. W poniższym rozwiązaniu otwieram dwa strumienie wtórne z operacjami anyMatch, które dają wartość dodatnią, jeśli moduł zawiera artefakt zawierający pożądany klucz.

To powinno załatwić sprawę:

ModuleKey key = Optional.ofNullable(modules.stream().filter(m -> m.getArtifacts().stream() 
    .anyMatch(a -> a.equals("ear"))).findFirst().orElse(modules.stream() 
    .filter(m -> m.getArtifacts().stream().anyMatch(a -> a.equals("war"))) 
    .reduce((a, b) -> b).orElse(null))).map(Module::getKey).orElse(null); 

Aby znaleźć pierwszy element, użyj findFirst, która zwraca Optional<T>. Łańcuch ten jest Optional z Optional#orElse, który rozpakowuje go, a jeśli jest pusty, zwraca inną wartość (w tym przypadku przy użyciu reduce((a, b) -> b), aby znaleźć ostatni element). W przypadku, gdy nic nie zostanie znalezione, nie chcemy, aby kod wyrzucał NullPointerException. Więc zanim zadzwonimy do metody getKey, zawijamy ją do Optional z Optional.ofNullable i ustawiamy domyślną wartość określoną w kodzie pod numerem orElse na null.

Na marginesie,

else if(!artifact.getType().equals("ear")) { 
    if(artifact.getType().equals("war")) { 
    } 
} 

może być zredukowana do

else if(artifact.getType.equals("war")) { 
} 

Answer 3

myślę, że są świadomi, że zbędny if sprawdzić w swoim starym stylu kodu Java.

Napisałem twój wielki pojedynczy strumień operacji, rozpadając się na mniejsze, łatwe w obsłudze funkcje, które każdy zrozumie.

public static ModuleKey getDeployableModuleFromModulesList(List<Module> modules) { 

    return findEARKey(modules.stream()) 
      .orElse(findWARKey(modules.stream()).orElse(null)); 

} 

public static Optional<ModuleKey> findEARKey(Stream<Module> moduleStream){ 

    return moduleStream.flatMap(e -> e.getArtifacts().stream()) 
      .filter(e -> e.getType().equals("ear")) 
      .map(Artifact::getKey).findFirst(); 
} 

public static Optional<ModuleKey> findWARKey(Stream<Module> moduleStream){ 

    return moduleStream.flatMap(e -> e.getArtifacts().stream()) 
      .filter(e -> e.getType().equals("war")) 
      .map(Artifact::getKey).findFirst(); 
}