Ответ 1
Не должно выглядеть так:
boolean areAllSuccessful = entries.stream()
.map(entry -> entry.isSuccessful())
.reduce(Boolean.TRUE, Boolean::logicalAnd);
Java 8 Lambdas - Побитовые операции И
В настоящее время я занимаюсь проблемой преобразования циклов, которые используют побитовые операции с использованием Lambdas в Java 8.
Учитывая набор сложных записей, требуется, чтобы цикл пересекал ВСЕ записи и вызывал на них заданный метод (метод возвращает логическое значение). Затем верните результат.
Другими словами, мне нужно вызвать метод для всех записей и сохранить результат. Причина в том, что каждая запись независимо выполняет сложную операцию и должна выполняться. Конечный результат представляет собой комбинацию результатов.
Фрагмент кода:
boolean areAllSuccessful = true;
for (SomeEntry entry : entries) {
areAllSuccessful = areAllSuccessful & entry.doComplexAction(); // keep calling the method on other entries regardless of the result.
}
return areAllSuccessful;
Проблема в том, что функции лямбда в Java 8 обычно выполняют операции короткого замыкания (как только ложная запись обнаружена, "цикл" ломается, и возвращается ложный результат).
Мое лучшее решение до сих пор заключалось в использовании комбинации карт/фильтров/счетчиков:
return entries
.stream()
.map(entry -> entry.doComplexAction())
.filter(result -> result == false)
.count() > 0
Есть ли более умный/чистый способ сделать это?
Спасибо!
Ответы
Ответ 2
Самый простой и эффективный способ - использовать ссылку метода с allMatch()
return entries.stream().allMatch(SomeEntry::isSuccessful);
Если у вас есть 1000 элементов, используйте вместо этого parallelStream().
Это не обрабатывает каждый элемент (он возвращается с первой false), поэтому, если ваш метод isSuccessful() имеет побочные эффекты, это плохое имя, и вы должны переименовать его или реорганизовать код для выполнения побочных эффектов в process() (или аналогичном) методе и isSuccessful() возвращает результат, бросая IllegalStateException, если process() не был вызван.
Если вы не реорганизуете, какой-то разработчик (включая вас) вызовет isSuccessful(), не понимая, что он "делает материал", что может быть плохо.
Ответ 3
Вам не нужно map(), если вы собираетесь использовать count():
return !(entries
.stream()
.filter(entry -> !entry.isSuccessful())
.count() > 0);
Если метод isSuccessful() не имел побочных эффектов, и все, что вам нужно знать, было, если все записи были успешными, вы могли бы использовать allMatch():
return entries
.stream()
.allMatch(entry -> entry.isSuccessful());
Это действительно операция короткого замыкания, которая вернет false, как только найдет запись, где isSuccessful() - false, не потребляя весь поток, если это необходимо. Но вы уже прокомментировали, что isSuccesful() действительно означает "выполнить некоторые сложные действия, а затем сказать мне, если они были успешными", поэтому он не применяется.