すでに行っているように、これ以上簡潔にすることはできません。
あなたは望んでいないと主張し.filter(Optional::isPresent) 、 .map(Optional::get) .
これは @StuartMarks が説明するメソッドによって解決されましたが、結果としてそれを にマップするようになったため、最後にandOptional<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();
を返すことはできないため、ここでaを返す必要があります。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ストリーム操作の中でスローするようになりました。
結論
あなたが使用した方法は、実際には最良の方法です。