Separa un optional solo se è presente

Question

Così Attualmente ho

String uri = website.getUri(); 
Optional<PageDetail> pageDetail = webClient.getDetailOfUri(uri); 
String displayName; 
String description; 
if (pageDetail.isPresent()) { 
    displayName = pageDetail.get().getName(); 
    description = pageDetail.get().getDescription(); 
} else { 
    displayName = uri; 
    description = ""; 
} 

sto chiamando il metodo getDetailOfUri(uri), che restituisce un Optional<PageDetail>, e vorrei per impostare le corde displayName e description ai valori della PageDetail campi dell'oggetto, se presente. Altrimenti, vorrei impostarlo su alcuni valori predefiniti.

La mia domanda è, c'è un modo migliore per riscrivere questo? Il mio codice attuale sembra un po 'lungo e noioso, e mi piacerebbe sapere se c'è un modo più conciso per farlo.

Answer 1

si potrebbe scrivere:

String uri = website.getUri(); 
Optional<PageDetail> pageDetail = webClient.getDetailOfUri(uri); 
String displayName = pageDetail.map(PageDetail::getName).orElse(uri); 
String description = pageDetail.map(PageDetail::getDescription).orElse(""); 

Se il Optional non è impostato, map tornerà lo stesso non impostato Optional. In caso contrario, verrà mappato a un Optional contenente il risultato di getName(). Quindi possiamo usare orElse per restituire un valore predefinito quando Optional non è impostato.

Answer 2

Uso Optional#orElseGet che prende un Supplier:

// Reference to the constructor, but you could use a Factory, etc. 
// All you need is a method that returns a PageDetail 
// See the Javadoc and http://www.byteslounge.com/tutorials/java-8-consumer-and-supplier 
Supplier<PageDetail> emptySupplier = PageDetail::new; 

pageDetail = pageDetail.orElseGet(emptySupplier); 
// works the same 
//pageDetail = pageDetail.orElseGet(() -> new PageDetail()); 

String displayname = pageDetail.getName(); 
String uri = pageDetail.getUri(); 

orElseGet creerà un PageDetail vuota solo se l'opzionale ha un valore nullo. Ciò mantiene efficiente il tuo codice di risorse.

campione compilabile/modificabile: https://ideone.com/9h1Ntg

Edit: Grazie a tutti per il feedback! In realtà stavo aggiungendo orElseGet che trovo meglio. Ho anche corretto il codice per scartare lo Optional in modo che pageDetail termini come un'istanza effettiva PageDetail.

Modifica 2: Aggiunto un esempio di sintassi diverso ed esempio modificabile/compilabile.