Wie vermeidet man die Verwendung von Optional.get und Optional.isPresent

Question

public ValueA map(ValueB valueB, Date date) { 
    Optional<ValueC> valueCOpt = find(valueB); 
    if (valueCOpt.isPresent()) { 
     ValueC valueC = valueCOpt.get(); 
     // call many getters on valueC and do a lot of logic with it. 
     return map(/*some parameters*/); 
    } 
    return null; 
} 

Dies scheint ziemlich hässlich. Der Vorteil von Optionals ist hier komplett verschwunden. Ich lese, dass man lieber map oder flatMap anstelle von get verwenden sollte. Aber ist es wirklich ein Vorteil, wenn ich jeden Getter wie

valueC.getFieldA() 

mit

valueCOpt.map(ValueC::getFieldA) 

ersetzen Sie wissen einige gemeinsame oder Best Practices hier?

Answer 1

können Sie verwenden

public ValueA map(ValueB valueB, Date date) { 
    return find(valueB) 
     .map(valueC -> { 
      // call many getters on valueC and do a lot of logic with it. 
      return map(/*some parameters*/); 
     }) 
     .orElse(null); 
} 

der entscheidende Punkt ist, dass die Abbildungsfunktion nur ausgewertet, wenn die optionale nicht leer ist, andernfalls das Ergebnis bleibt eine leere optional . orElse(null) wird null zurückgeben, wenn das optionale leer ist.

Answer 2

Was Sie brauchen, ist auf der Karte, dann ein OrElse() oder orElseThrow(), wenn Sie eine Ausnahme benötigen

ValueA valueA = valueCOpt.map(valueC -> mapToValue(valueC)) 
     .orElse(null); 

OrElse() wird verwendet, wenn Sie einen Standardwert benötigen, in diesem Fall seine null