I’ve waffled on it. Currently my opinion is to use whatever nullable annotation the project uses (if any, otherwise JetBrains’s). Essentially, I’m not sure if the official recommendation to avoid Optional for uses other than return types has reasoning I’m missing.
I do use it like this though and lament that we don’t have an Elvis operator (?:)
I’ve waffled on it. Currently my opinion is to use whatever nullable annotation the project uses (if any, otherwise JetBrains’s). Essentially, I’m not sure if the official recommendation to avoid
Optionalfor uses other than return types has reasoning I’m missing.I do use it like this though and lament that we don’t have an Elvis operator (
?:)Optional.ofNullable(thingThatMightBeNull).map(e -> e.someMethod()).orElse(null);Instead of wrapping it in an optional you can do
Objects.requireNonNullElse(value, defaultValue)Optional has more syntactic sugar for more complex scenarios / functional call chaining that prevents repetitive
ifchecksOptional.ofNullable(myObj) .map(MyClass::getProperty) .map(MyOtherClass::getAnotherProperty) .filter(obj -> somePredicate(obj)) .orElse(null)This is completely null safe, the function calls are only made if the object is not null