Quelle est la façon la plus élégante de combiner les options?

Voici ce que j'ai obtenu jusqu'à présent:

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

Cela me semble à la fois hideux et de gaspillage. Si firstChoice est présent, je suis inutilement de l'informatique secondChoice.

Il y a également une plus efficace version:

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

Ici je ne peux pas la chaîne certaine fonction de mappage à la fin, sans dupliquer le mappeur ou la déclaration d'une autre variable locale. Tout cela rend le code plus compliqué que le réel problème étant résolu.

Je préfère être écrit ceci:

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

Cependant Facultatif::sinon, évidemment, n'existe pas. Maintenant ce qui?

double possible de Obtenir la valeur d'une Option ou d'une autre
... lorsque l'on est confronté avec le problème de la valeur null, " maintenant, nous avons deux problèmes.
Btw, Paul Sandoz vient de publier un changeset pour l'examen qui introduit Optional.or() méthode, qui fait exactement ce que vous voulez: firstChoice().or(this::secondChoice). Plus d'infos sur JDK-8080418. En Java-9 fois nous allons être heureux!

InformationsquelleAutor Sebastian Oberhoff | 2015-09-25

41

Essayez ceci:
```
firstChoice().map(Optional::of)
             .orElseGet(this::secondChoice);
```
La méthode map vous donne un Optional<Optional<Foo>>. Ensuite, le orElseGet méthode aplatit ce retour à une Optional<Foo>. Le secondChoice méthode sera évaluée uniquement si firstChoice() retourne le vide en option.
- Remarque: Java 9 présente les Optional::or méthode. Cela pourrait donc être écrite firstChoice().or(this::secondChoice) en Java 9.
- Cela ne fonctionne qu'avec Java8 si this::secondChoice renvoie une Supplier plutôt que Optional - l'exemple de l'OP n'a pas.
- un Supplier. C'est une méthode de référence.
- Dans le cas des OP exemple, la méthode renvoie en Option, ce qui n'est pas un Fournisseur et j'ai été en soulignant que pour Java8, c'est différent.
- retourne un Optional. this::secondChoice est une référence à une méthode de type Supplier<Optional<Foo>>. Si vous appelez la méthode de référence, vous obtenez le Optional. L'exemple de la réponse qui fonctionne pour Java 8 et ainsi de suite.
- Argh, négligé ce détail. Mon mauvais.
InformationsquelleAutor marstran
5

Vous pouvez simplement remplacer qu'avec,
```
Optional<Foo> firstChoice = firstChoice();
return firstChoice.isPresent()? firstChoice : secondChoice();
```
Le code ci-dessus ne seront pas appel, à moins que firstChoice.isPresent() est faux.

Mais vous avez à préparer pour appeler à la fois des fonctions pour obtenir le résultat souhaité. Il n'y a pas d'autre moyen d'échapper à la vérification.
- Meilleur des cas, est le Premier choix de retourner true.
- Pire des cas sera le Premier choix de retourner faux, et par conséquent une autre méthode
  appel pour le deuxième choix.
InformationsquelleAutor Suresh Atta

Peut-être quelque chose comme ceci :

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

À partir de : Le chaînage d'Options en Java 8

InformationsquelleAutor Marthym

Voici la généralisation de @marstran solution pour un certain nombre d'options:

@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()));
}

De sortie:

foo called
bar called
Optional[bar]

InformationsquelleAutor Tagir Valeev

1

J'ai été assez frustré par le fait que ce n'était pas prise en charge par java 8, que je suis passé de retour à la goyave est options qui ont ou:
```
public abstract Optional<T> or(Optional<? extends T> secondChoice)
```
Retourne cette Option si elle a une valeur actuelle; secondChoice autrement.

InformationsquelleAutor Erik

Paresseux calculs et nombre arbitraire de Optional éléments

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

InformationsquelleAutor Pepe-Soft

-1

Ici est un moyen qui fonctionne pour un nombre arbitraire de Optional's dans un flux de données de l'API:
```
return Arrays.asList(firstChoice, secondChoice).stream()
  .filter(Optional::isPresent)
  .map(Optional::get)
  .findFirst().orElse(null);
```
Ce n'est pas le plus court. Mais plus clair et compréhensible.

Une autre façon est d'utiliser des firstNonNull() de Goyave de commons-lang si vous utilisez déjà l'un de ces bibliothèques:
```
firstNonNull(firstChoice.orElse(null), secondChoice.orElse(null));
```
- Je trouve cela beaucoup moins compréhensible que le if solution à la question, d'ailleurs.
- Eh bien, je suis d'accord si la seule chose que vous avez à faire est de 2 Optionals. C'est pourquoi j'ai dit que cette méthode fonctionne pour nombre arbitraire de Optionals.
- Le problème de votre solution, c'est qu'il n'est pas de court-circuit: vous devez évaluer tous les choix qui est exactement ce que l'OP cherche à l'éviter. OPs code firstChoice() et secondChoice() sont en fait des méthodes, et vous ne peut pas simplement modifier votre solution d'omettre de l'appel de la deuxième si le premier produit non-vide en option.
- Que voulez-vous dire? findFirst() défini comme un court-circuting opération dans la documentation. Vous pouvez facilement vérifier que Optional.get() et Optional.isPresent() méthodes appelé qu'une seule fois à l'aide du débogueur.
InformationsquelleAutor Denis Bazhenov

Vous devez vous connecter pour publier un commentaire.