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

[Anonymous]

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

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

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

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

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

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

private  Predicate testOrElse(Predicate condition, Runnable executeOnFalse) {
return condition.or(x1 -> { executeOnFalse.run(); return false; });
}

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

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