Ordnungsgemäße Verwendung von Optional.ifPresent ()

Ich versuche die ifPresent()Methode der OptionalAPI in Java 8 zu verstehen .

Ich habe eine einfache Logik:

Optional<User> user=...
user.ifPresent(doSomethingWithUser(user.get()));

Dies führt jedoch zu einem Kompilierungsfehler:

ifPresent(java.util.functionError:(186, 74) java: 'void' type not allowed here)

Natürlich kann ich so etwas machen:

if(user.isPresent())
{
  doSomethingWithUser(user.get());
}

Aber das ist genau wie ein überfüllter nullScheck.

Wenn ich den Code in diesen ändere:

 user.ifPresent(new Consumer<User>() {
            @Override public void accept(User user) {
                doSomethingWithUser(user.get());
            }
        });

Der Code wird schmutziger, was mich daran erinnert, zum alten nullScheck zurückzukehren.

Irgendwelche Ideen?

Optional<User>.ifPresent()nimmt ein Consumer<? super User>als Argument. Sie übergeben ihm einen Ausdruck, dessen Typ nichtig ist. Das wird also nicht kompiliert.

Ein Verbraucher soll als Lambda-Ausdruck implementiert werden:

Optional<User> user = ...
user.ifPresent(theUser -> doSomethingWithUser(theUser));

Oder noch einfacher mit einer Methodenreferenz:

Optional<User> user = ...
user.ifPresent(this::doSomethingWithUser);

Dies ist im Grunde das gleiche wie

Optional<User> user = ...
user.ifPresent(new Consumer<User>() {
    @Override
    public void accept(User theUser) {
        doSomethingWithUser(theUser);
    }
});

Die Idee ist, dass der doSomethingWithUser()Methodenaufruf nur ausgeführt wird, wenn der Benutzer anwesend ist. Ihr Code führt den Methodenaufruf direkt aus und versucht, sein ungültiges Ergebnis an zu übergeben ifPresent().

Zusätzlich zur Antwort von @ JBNizet besteht mein allgemeiner Anwendungsfall ifPresentdarin, zu kombinieren .isPresent()und .get():

Alter Weg:

Optional opt = getIntOptional();
if(opt.isPresent()) {
    Integer value = opt.get();
    // do something with value
}

Neuer Weg:

Optional opt = getIntOptional();
opt.ifPresent(value -> {
    // do something with value
})

Das ist für mich intuitiver.

Warum komplizierten Code schreiben, wenn Sie es einfach machen könnten?

In der Tat, wenn Sie die OptionalKlasse unbedingt verwenden wollen , ist der einfachste Code der , den Sie bereits geschrieben haben ...

if (user.isPresent())
{
    doSomethingWithUser(user.get());
}

Dieser Code hat die Vorteile des Seins

1. lesbar
2. einfach zu debuggen (Haltepunkt)
3. nicht schwierig

Nur weil Oracle die OptionalKlasse in Java 8 hinzugefügt hat, bedeutet dies nicht, dass diese Klasse in allen Situationen verwendet werden muss.

Verwenden Sie flatMap. Wenn ein Wert vorhanden ist, gibt flatMap einen sequentiellen Stream zurück, der nur diesen Wert enthält. Andernfalls wird ein leerer Stream zurückgegeben. Es besteht also keine Notwendigkeit zu verwenden ifPresent(). Beispiel:

list.stream().map(data -> data.getSomeValue).map(this::getOptinalValue).flatMap(Optional::stream).collect(Collectors.toList());

Sie können eine Methodenreferenz wie folgt verwenden:

user.ifPresent(ClassNameWhereMethodIs::doSomethingWithUser);

Methode ifPresent()get Consumerobject als Paremeter und (von JavaDoc ): "Wenn ein Wert vorhanden ist, rufen Sie den angegebenen Consumer mit dem Wert auf." Wert es ist Ihre Variable user.

Wenn sich diese Methode doSomethingWithUserin der UserKlasse befindet und nicht static, können Sie die Methodenreferenz wie folgt verwenden:

user.ifPresent(this::doSomethingWithUser);