Jak sprawdzić, czy kolekcja nie jest pusta przy użyciu strumienia Javy

Question

Jestem nowy w Java 8. Nie jestem w stanie zrozumieć, co jest nie tak w poniższym kodzie. Pomysł polega na wysłaniu Collection<User>, jeśli nie jest pusta. Ale jeśli kolekcja jest pusta, niż wysłano odpowiedź jednostki Entity HttpStatus.NOT_FOUND.

@RequestMapping(value = "/find/pks", 
       method = RequestMethod.GET, 
       produces = MediaType.APPLICATION_JSON_VALUE) 
public ResponseEntity<Collection<User>> getUsers(@RequestBody final Collection<String> pks) 
{ 
    return StreamSupport.stream(userRepository.findAll(pks).spliterator(), false) 
     .map(list -> new ResponseEntity<>(list , HttpStatus.OK)) 
     .orElse(new ResponseEntity<>(HttpStatus.NOT_FOUND)); 
} 

Eclipse pokazuje mi błąd w następnym punkcie .orElse

Sposób orElse(new ResponseEntity<>(HttpStatus.NOT_FOUND)) jest nieokreślony dla rodzaju Stream<ResponseEntity<User>>

Moja metoda interfejs baza wygląda jak po

Iterable<T> findAll(Iterable<PK> pks); 

Answer 1

Jesteś mieszania dwóch rzeczy. Pierwszym zadaniem jest przekształcenie Iterable do Collection które można rzeczywiście rozwiązać stosując Stream API:

Collection<User> list= 
    StreamSupport.stream(userRepository.findAll(pks).spliterator(), false) 
    .collect(Collectors.toList()); 

Zauważ, że ten strumień jest strumień User ów, a nie strumień list. Dlatego nie można zmapować pliku list na coś innego z tym strumieniem. Operacja map zmapuje każdy element strumienia do nowego elementu.

Następnie można użyć tej listy, aby utworzyć ResponseEntity

return list.isEmpty()? new ResponseEntity<>(HttpStatus.NOT_FOUND): 
         new ResponseEntity<>(list, HttpStatus.OK); 

Można łączyć kroki tworząc Collector wykonywania tej czynności, choć nie daje żadnych korzyści, to tylko kwestia stylu:

ResponseEntity<User> responseEntity= 
    StreamSupport.stream(userRepository.findAll(pks).spliterator(), false) 
    .collect(Collectors.collectingAndThen(Collectors.toList(), 
     list -> list.isEmpty()? new ResponseEntity<>(HttpStatus.NOT_FOUND): 
           new ResponseEntity<>(list, HttpStatus.OK))); 

Answer 2

To zależy od twojego końcowego działania strumienia, pamiętaj, że strumień może zostać zużyty tylko jeden raz.

• Czy jest to operacja grupowania według/statystyki? Wtedy dostaniesz 0 lub pustą mapę grup.
• Jeśli zbierze się do listy, to jest to pusta lista.
• Jeśli jest to jedna z metod zwracających wartość Optional (na przykład findAny), można użyć opcjonalnych metod sprawdzania wartości null.

Answer 3

Nie jest konieczne, a często pomyłka, wrzucanie wszystkiego do jednej linii. W takim przypadku nie możesz - nie ma takiego API dla twojej intencji.

Keep it simple:

Collection<User> list = <your stream code that gets a list>; 
if (list.isEmpty()) 
    return new ResponseEntity<>(HttpStatus.NOT_FOUND); 
return new ResponseEntity<>(list, HttpStatus.OK); 

ale jeśli absolutnie koniecznością:

return <your code>.map(list -> new ResponseEntity<>(list, list.isEmpty() ? HttpStatus.NOT_FOUND : HttpStatus.OK));