Как получить данные из коллекции опций? - PullRequest
Новая
       28

Как получить данные из коллекции опций?

0 голосов
/ 30 января 2019

У меня есть метод, возвращающий коллекцию товаров: 

Collection<Product> getProducts() { ... }

На каждый товар может быть гарантия.Но это не обязательно. 

interface Product {
    Optional<Guarantee> getGuarantee();
}

Теперь мне нужно просмотреть все продукты и проверить, не истек ли срок гарантии.Не просроченные должны быть собраны в список.

Вот что я делаю: 

List<Optional<Guarantee>> optionalGar = getProducts().stream()
      .map(f -> f.getGuarantee()).collect(Collectors.toList());

List<Guarantee> gar = optionalGar.stream()    
      .map(op -> op.orElse(null))             
      .filter(Objects::nonNull)
      .filter(g -> !g.hasExpired())
      .collect(Collectors.toList());

Есть ли способ избежать использования .orElse(null)?

(Замена на op.get() вызовет исключение в случае, если необязательный параметр пуст)

PS: я могу свободно выбирать между Java 8 и Java 9, поэтому оба решения(не уверен, что будет иначе) добро пожаловать

1 Ответ

0 голосов
/ 30 января 2019

Java 8 

List<Guarantee> expiredGuarantees = getProducts().stream()
                                                 .map(Product::getGuarantee)    
                                                 .filter(Optional::isPresent)
                                                 .map(Optional::get)
                                                 .filter(not(Guarantee::hasExpired))
                                                 .collect(toList());

Java 9

Java9 получил Optional::stream.Таким образом, вы можете заменить фильтрацию и отображение одним flatMap: 

List<Guarantee> expiredGuarantees = getProducts().stream()
                                                 .map(Product::getGuarantee)    
                                                 .flatMap(Optional::stream)
                                                 .filter(not(Guarantee::hasExpired))
                                                 .collect(toList());

Примечание

В Java 8 отсутствует метод Predicates.not.Он включен только с 11-й версии.

Добавив следующий метод в свой проект, вы сможете использовать его с указанными выше решениями. 

public static <T> Predicate<T> not(Predicate<T> predicate) { 
    return predicate.negate();
}

Обновление

Хотяэто не сообщество CodeReview , вот несколько примечаний к вашему коду:

  • Объединяя два конвейера в один, ваш код будет чище (в данном конкретном случае).
  • Предпочитайте method reference, а не lambda, когда это возможно
  • Дайте соответствующие имена вашим переменным, чтобы облегчить поддержку вашего кода
...