すでにやっているので、もっと簡潔にすることはできません。
あなたがしたくない.filter(Optional::isPresent) と 主張し、.map(Optional::get)。
これは、@ StuartMarksが記述するメソッドによって解決されましたが、結果として、これをにマップするOptional<T>ため、最後に.flatMap(this::streamopt)とを使用する必要がありますget()。
したがって、これはまだ2つのステートメントで構成されており、新しいメソッドで例外を取得できます！なぜなら、すべてのオプションが空の場合はどうなりますか？次に、findFirst()空のオプションが返され、get()失敗します！
だからあなたが持っているもの：
things.stream()
.map(this::resolve)
.filter(Optional::isPresent)
.map(Optional::get)
.findFirst();
で、実際にあなたが望むものを達成するための最良の方法、それはあなたに結果を保存したいですT、ではないとしてOptional<T>。
をCustomOptional<T>ラップOptional<T>し、追加のメソッドを提供するクラスを自由に作成しましたflatStream()。拡張できないことに注意してくださいOptional<T>：
class CustomOptional<T> {
private final Optional<T> optional;
private CustomOptional() {
this.optional = Optional.empty();
}
private CustomOptional(final T value) {
this.optional = Optional.of(value);
}
private CustomOptional(final Optional<T> optional) {
this.optional = optional;
}
public Optional<T> getOptional() {
return optional;
}
public static <T> CustomOptional<T> empty() {
return new CustomOptional<>();
}
public static <T> CustomOptional<T> of(final T value) {
return new CustomOptional<>(value);
}
public static <T> CustomOptional<T> ofNullable(final T value) {
return (value == null) ? empty() : of(value);
}
public T get() {
return optional.get();
}
public boolean isPresent() {
return optional.isPresent();
}
public void ifPresent(final Consumer<? super T> consumer) {
optional.ifPresent(consumer);
}
public CustomOptional<T> filter(final Predicate<? super T> predicate) {
return new CustomOptional<>(optional.filter(predicate));
}
public <U> CustomOptional<U> map(final Function<? super T, ? extends U> mapper) {
return new CustomOptional<>(optional.map(mapper));
}
public <U> CustomOptional<U> flatMap(final Function<? super T, ? extends CustomOptional<U>> mapper) {
return new CustomOptional<>(optional.flatMap(mapper.andThen(cu -> cu.getOptional())));
}
public T orElse(final T other) {
return optional.orElse(other);
}
public T orElseGet(final Supplier<? extends T> other) {
return optional.orElseGet(other);
}
public <X extends Throwable> T orElseThrow(final Supplier<? extends X> exceptionSuppier) throws X {
return optional.orElseThrow(exceptionSuppier);
}
public Stream<T> flatStream() {
if (!optional.isPresent()) {
return Stream.empty();
}
return Stream.of(get());
}
public T getTOrNull() {
if (!optional.isPresent()) {
return null;
}
return get();
}
@Override
public boolean equals(final Object obj) {
return optional.equals(obj);
}
@Override
public int hashCode() {
return optional.hashCode();
}
@Override
public String toString() {
return optional.toString();
}
}
次のように、私が追加したことがわかりflatStream()ます。
public Stream<T> flatStream() {
if (!optional.isPresent()) {
return Stream.empty();
}
return Stream.of(get());
}
使用されます：
String result = Stream.of("a", "b", "c", "de", "fg", "hij")
.map(this::resolve)
.flatMap(CustomOptional::flatStream)
.findFirst()
.get();
あなたはまだ返す必要がありますStream<T>あなたが返すことができないとして、ここではT理由ならば、!optional.isPresent()そして、T == nullあなたはそのような宣言が、その後、あなたの場合は.flatMap(CustomOptional::flatStream)追加しようとnullストリームにし、それが不可能です。
例として：
public T getTOrNull() {
if (!optional.isPresent()) {
return null;
}
return get();
}
使用されます：
String result = Stream.of("a", "b", "c", "de", "fg", "hij")
.map(this::resolve)
.map(CustomOptional::getTOrNull)
.findFirst()
.get();
NullPointerExceptionストリーム操作の内部をスローします。
結論
あなたが使用した方法は、実際には最良の方法です。
.flatMap(Optional::toStream)ました。