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; } Esto parece bastante feo. La ventaja de los opcionales desaparece por completo aquí. Leí que uno debería usar map o flatMap en lugar de get . Pero, ¿realmente es un beneficio si reemplazo cada getter como
valueC.getFieldA()con
valueCOpt.map(ValueC::getFieldA)¿Conoces algunas prácticas comunes o mejores aquí?
Puedes usar
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); } el punto clave es que la función de mapeo solo se evalúa si el opcional no está vacío, de lo contrario, el resultado permanece como un opcional vacío. orElse(null) devolverá null si el opcional está vacío.
Lo que necesita es mapear, luego orElse() o orElseThrow() si necesita una excepción
ValueA valueA = valueCOpt.map(valueC -> mapToValue(valueC)) .orElse(null); orElse() se usa cuando necesita un valor predeterminado, en este caso es null