API потока Java: вход в поток ⇐ JAVA

[Anonymous]

Мне очень нравится, как можно программировать потоки Java как способ перехода от входной переменной к выходной с помощью серии четко определенных стандартизированных шагов, таких как отображение и фильтрация. Поэтому иногда жалко покидать эту структуру. Иногда кажется, что проще и читабельнее оставить эту структуру, но это может быть связано с отсутствием доступной библиотечной функциональности, лучших практик или моих знаний о них.
Давайте посмотрим на этот код «старой школы» (ну, в нем есть необязательный и var, так что не такой уж старый)

Код: Выделить всё

public Optional getValidSupplierAddress(Supplier supplier) {
var supplier = context.getAllSuppliers();
if (!supplier.hadRecentProjects) {
log.debug("supplier didn't have any recent projects");
return Optional.empty();
}
if (!supplier.validatedByBoardOfCommerce()) {
log.debug("supplier is not validated by board of commerce");
return Optional.empty();
}
var address = supplier.getAddress();
if (address == null) {
log.debug("no address specified for supplier");
return Optional.empty();
}
if (address.isPobox()) {
log.debug("address is not valid, no pobox allowed");
return Optional.empty();
}
return Optional.of(address);
}

Вы можете записать это примерно так:

Код: Выделить всё

public Optional getValidSupplierAddress(Supplier supplier) {
return Optional.of(supplier)
.filter(logOnFalse(Supplier::hadRecentProjects,
"supplier didn't have any recent projects"))
.filter(logOnFalse(Supplier::validatedByBoardOfCommerce,
"supplier is not validated by board of commerce"))
.map(supplier -> Optional.of(supplier.getAddress())
.filter(logOnFalse(Optional::isPresent,
"no address specified for supplier"))
.map(Optional::get)
.filter(logOnFalse(not(Address::isPobox),
"address is not valid, no pobox allowed"));
}

private  Predicate logOnFalse(Predicate condition, String logStrOnFalse) {
return condition.or(x1 -> { log.debug(logStrOnFalse); return false; });
}

Сделал ли я здесь более читабельный код или нет? Пока не совсем уверен. Но я думаю, что когда появятся какие-либо библиотечные функции, поддерживающие подобный подход, он может стать (даже?) более читабельным. Может ли кто-нибудь улучшить этот код, желательно используя существующие основные библиотеки?

Подробнее здесь: https://stackoverflow.com/questions/798 ... e-a-stream