Optional.ofNullable()
where always null or always not-null argument is passed.
There's no point in using Optional.ofNullable()
in this case: either Optional.empty()
or Optional.of()
should be used to explicitly state the intent of creating an always-empty
or always non-empty optional respectively. It's also possible that there's a mistake in
Optional.ofNullable()
argument, so it should be examined.
Example:
Optional<String> empty = Optional.ofNullable(null); // should be Optional.empty();
Optional<String> present = Optional.ofNullable("value"); // should be Optional.of("value");