Gfrktyl

I have a question regarding Java 8's Optional, the purpose of which is to tackle NullPointerException exceptions.

The question is, what is the reason for having both types to let us choose:

Optional.of(T value)     <-----non-null value, null value will throw NPE

Optional.ofNullable(T value)   <----- nullable value

Because what I expect is, when I use:

Optional.of(nullValue);

It won't throw a NullPointerException.

Expanded my question after some replies:

Why would people opt for Optional instead of normal if-else for null checking?

The javadoc of Optional.of reads that explicitly :

@throws NullPointerException if value is null

and that is where the requirement of handling the cases as expected by you comes into picture with the use of Optional.ofNullable which is a small block of code as :

public static <T> Optional<T> ofNullable(T value) {

    return value == null ? empty() : of(value); // 'Optional.of'

}

That said, the decision of choosing one over the other would still reside with the application design as if your value could possibly be null or not.

On your expectation part, that was not what the Optional was actually intended for. The API note clarifies this further (formatting mine):

Optional is primarily intended for use as a method return type where
there is a clear need to represent "no result," and where using
null is likely to cause error. A variable whose type is Optional should never itself be null; it should always point to an Optional instance.

purpose of Optional is to tackle NullPointerException exception.

Aside: Just to call it out clearly, that the choice would of course implicitly let you define if an NPE should be thrown at runtime or not. It's not determined at the compile time though.

the purpose of Optional is to tackle NullPointerException exception

Yes, it is, but at usage time not at creation.

So when you receive an Optional from a method then you can avoid NPE by using Optional.ifPresent, Optional.orElse,Optional.orElseGet and Optional.orElseThrow methods.

But this is not the case when you're creating an Optional. Since it's your own method you have to know whether the object is nullable or not.

The main point of Optional is to provide a means for a function returning a value to indicate the absence of a return value. See this discussion. This allows the caller to continue a chain of fluent method calls.

Stuart Marks

Please read this post for more detailed explanation.

I think it's quite simple and clear with Javadoc:

Optional.of(T value) is used when you are sure that there is never a null value and incase null value occurs than program throws NullPointerException and consider as bug.

Optional.ofNullable(T value) is used when you know that there can be a null value and in case of it your program should behave normally.

Why would people opt for Optional instead of normal if-else method for null checking?

Why would people opt for Optional instead of normal if-else method for null checking?

The main point of Optional<T> class is to provide a mean for performing null safe mapping operations.

employeeOptional.map(Employee::getName).map(String::toUpperCase).ifPresent(upperCasedNameConsumer)

The expression above can replace a cascade of if-else statements in a single readable expression.

Optional.of provides an assertion for the given argument to be a null-null value, otherwise, you can opt for Optional.ofNullable if you are not sure about the input.

I strongly recommend you to read the javadoc for Optional<T> class for more optional chaining methods that you can use for your advantage.

After reading some answers and comments I think this explanation is missing. Consider a method like

public Optional<String> name(Customer c) {

    return c.isNew() ? Optional.ofNullable(getName(c)) : Optional.of(getName(c));

}

Here you want to throw a NullPointerException if the customer isn't new and is supposed to have a name; your code is inconsistent if that's ever null. Yet the name may not yet exist if the customer is new, hence ofNullable in that case and the method returns Optional.