Was ist der eleganteste Weg, um Optionen kombinieren?

Hier ist, was ich habe, so weit:

Optional<Foo> firstChoice = firstChoice();
Optional<Foo> secondChoice = secondChoice();
return Optional.ofNullable(firstChoice.orElse(secondChoice.orElse(null)));

Diese Streiks mich als abscheulichen und verschwenderischen. Wenn firstChoice ist vorhanden, ich bin unnötig computing secondChoice.

Es gibt auch eine effizientere version:

Optional<Foo> firstChoice = firstChoice();
if(firstChoice.isPresent()) {
 return firstChoice;
} else {
 return secondChoice();
}

Hier kann ich nicht Kette einige mapping-Funktion, um das Ende, ohne duplizieren der mapper oder deklarieren eine lokale variable. All dies macht den code komplizierter als das eigentliche problem gelöst wird.

Ich würde viel lieber schreiben diese:

return firstChoice().alternatively(secondChoice());

Jedoch Optional::alternativ natürlich nicht vorhanden. Was nun?

mögliche Duplikate von Get Wert aus einem Optionalen oder anderen
... wenn konfrontiert mit dem problem der null ... jetzt haben wir zwei Probleme.
Btw, Paul Sandoz nur veröffentlicht, changeset für das review, das stellt Optional.or() Methode, die genau das tut, was Sie wollen: firstChoice().or(this::secondChoice). Mehr Infos unter JDK-8080418. In Java-9-mal werden wir glücklich sein!

InformationsquelleAutor Sebastian Oberhoff | 2015-09-25

41

Versuchen Sie dies:
```
firstChoice().map(Optional::of)
             .orElseGet(this::secondChoice);
```
Die map-Methode gibt Ihnen einen Optional<Optional<Foo>>. Dann, die orElseGet Methode flacht diese wieder zu einem Optional<Foo>. Die secondChoice Methode wird nur ausgewertet werden, wenn firstChoice() gibt die leeren optional.
- Hinweis: Java-9 stellt die Optional::or Methode. Dies könnte also geschrieben werden firstChoice().or(this::secondChoice) in Java 9.
- Dies funktioniert nur mit Java8, wenn this::secondChoice gibt eine Supplier eher als Optional - das Beispiel aus dem OP nicht.
- ein Supplier. Es ist eine Methode verweisen.
- In OP ' s Beispiel, dass die Methode gibt Optional, das ist nicht ein Anbieter - und ich war wies darauf hin, dass für Java8 ist es anders.
- gibt ein Optional. this::secondChoice ist eine Methode, Referenz vom Typ Supplier<Optional<Foo>>. Wenn Sie die Methode aufrufen, Verweis erhalten Sie die Optional. Das Beispiel in der Antwort funktioniert für Java 8 und höher.
- Argh, übersehen, dass detail. Mein schlechtes.
InformationsquelleAutor marstran
5

Können Sie einfach ersetzen,
```
Optional<Foo> firstChoice = firstChoice();
return firstChoice.isPresent()? firstChoice : secondChoice();
```
Den oben genannten code nicht nennen, es sei denn firstChoice.isPresent() ist falsch.

Aber Sie müssen darauf vorbereitet sein zu nennen, die beide Funktionen, um die gewünschte Ausgabe erhalten. Es gibt keinen anderen Weg, um zu entkommen die Prüfung.
- Besten Fall Erste Wahl "true" zurückgeben.
- Schlimmsten Fall Erste Wahl false zurückgeben, damit andere Methode
  Aufruf zur zweiten Wahl.
InformationsquelleAutor Suresh Atta

Vielleicht so etwas wie dieses :

Optional<String> finalChoice = Optional.ofNullable(firstChoice()
    .orElseGet(() -> secondChoice()
    .orElseGet(() -> null)));

Aus : Verkettung Optionen in Java 8

InformationsquelleAutor Marthym

Hier ist die Verallgemeinerung von @marstran Lösung für eine beliebige Anzahl von Optionen:

@SafeVarargs
public static <T> Optional<T> selectOptional(Supplier<Optional<T>>... optionals) {
    return Arrays.stream(optionals)
            .reduce((s1, s2) -> () -> s1.get().map(Optional::of).orElseGet(s2))
            .orElse(Optional::empty).get();
}

Test:

public static Optional<String> first() {
    System.out.println("foo called");
    return Optional.empty();
}

public static Optional<String> second() {
    System.out.println("bar called");
    return Optional.of("bar");
}

public static Optional<String> third() {
    System.out.println("baz called");
    return Optional.of("baz");
}

public static void main(String[] args) {
    System.out.println(selectOptional(() -> first(), () -> second(), () -> third()));
}

Ausgabe:

foo called
bar called
Optional[bar]

InformationsquelleAutor Tagir Valeev

1

Ich war frustriert genug, durch die Tatsache, dass diese nicht unterstützt in java 8, die ich wieder eingeschaltet, um Guave ist Optionen die oder:
```
public abstract Optional<T> or(Optional<? extends T> secondChoice)
```
Gibt diese Optional, wenn ein Wert vorhanden; secondChoice sonst.

InformationsquelleAutor Erik

Faul Berechnungen und beliebige Anzahl von Optional Elemente

Stream.<Supplier<Optional<Foo>>>of(
        this::firstChoice,
        this::secondChoice
).map(
        Supplier::get
).filter(
        Optional::isPresent
).findFirst(
).orElseGet(
    Optional::empty
);

InformationsquelleAutor Pepe-Soft

-1

Hier ist ein Weg, der funktioniert für eine beliebige Anzahl von Optional's in einem stream API:
```
return Arrays.asList(firstChoice, secondChoice).stream()
  .filter(Optional::isPresent)
  .map(Optional::get)
  .findFirst().orElse(null);
```
Es ist nicht der kürzeste. Aber mehr schlicht und verständlich.

Andere Möglichkeit ist die Verwendung firstNonNull() aus Guave von commons-lang, wenn Sie bereits mit einer dieser Bibliotheken:
```
firstNonNull(firstChoice.orElse(null), secondChoice.orElse(null));
```
- Ich finde das eine sehr viel weniger verständlich, als die if Lösung in der Frage, btw.
- Gut, ich akzeptiere, wenn das einzige, was Sie haben, ist 2 Optionals. Dies ist der Grund, warum ich sagte, dass dieser Weg funktioniert für beliebig viele Optionals.
- Das problem deiner Lösung ist, dass es nicht Kurzschluss: Sie haben zu bewerten, wie die Entscheidungen, das ist genau das, was der OP versucht, zu vermeiden. Im OPs-code firstChoice() und secondChoice() sind eigentlich Methoden, und man kann nicht einfach ändern Sie Ihre Lösung weglassen beim Aufruf der zweiten, wenn die erste, die nicht leer sind optional.
- Was meinst du? findFirst() definiert als kurz-circuting Betrieb in der Dokumentation. Sie können leicht überprüfen, dass Optional.get() und Optional.isPresent() Methoden nur einmal aufgerufen werden, mit debugger.
InformationsquelleAutor Denis Bazhenov

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.