Как исправить код записи с API потока Java 8?

Если код работает, зачем его менять? Тем не менее, вот решение, использующее потоковый API:

public Set<Product> getProductsByFilter(Map<String, List<String>> filterParams) {

    Set<Product> productsByBrand = filterParams.containsKey("brand") ?
                filterParams.get("brand")
                         .stream()
                         .flatMap(brandName -> listOfProducts.stream()
                                            .filter(product -> brandName.equalsIgnoreCase(product.getManufacturer())))
                         .collect(Collectors.toCollection(HashSet::new)) : new HashSet<>();

   Set<Product> productsByCategory =
            filterParams.containsKey("category") ?
                  filterParams.get("category")
                               .stream()
                               .flatMap(category -> this.getProductsByCategory(category).stream())
                               .collect(Collectors.toCollection(HashSet::new)) : new HashSet<>();

        productsByCategory.retainAll(productsByBrand);
        return productsByCategory;
}

или, как предлагает @shmosel, вы можете избежать тернарного оператора с помощью getOrDefault:

Set<Product> productsByBrand = 
       filterParams.getOrDefault("brand", Collections.emptyList())
                   .stream()
                   .flatMap(brandName -> listOfProducts.stream()
                              .filter(product -> brandName.equalsIgnoreCase(product.getManufacturer())))
                   .collect(Collectors.toCollection(HashSet::new));

Set<Product> productsByCategory =
      filterParams.getOrDefault("category", Collections.emptyList())
                  .stream()
                  .flatMap(category -> this.getProductsByCategory(category).stream())
                  .collect(Collectors.toCollection(HashSet::new));

Попытка сделать его максимально основанным на лямбда-потоке.

Сначала текущий код вопроса вернет пустой набор, если какой-либо критерий (бренд или категория) не указан. Если это не ошибка, а предполагаемое поведение, тогда имеет смысл проверить, так ли это, и быстро вернуть пустой набор. Итак, код может начинаться так:

final List<String> filterBrands = filterParams.getOrDefault("brand", Collections.emptyList()));
final List<String> filterCategories = filterParams.getOrDefault("categories", Collections.emptyList()));
if (filterCategories.isEmpty() || filterCategories.isEmpty()) {
  return Collections.emptySet();
}

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

На этом этапе вы можете использовать другой подход к ответу, состоящий в создании набора продуктов с двумя критериями, а затем использовать пересечение. Однако, поскольку вы можете получить доступ к бренду, также известному как производство, непосредственно из Продукта, существует альтернатива, которая не требует явного создания экземпляров таких наборов, мы могли бы улучшить производительность в зависимости от соотношения размера прибыли по сравнению с каждым набором критериев. Итак, если результатом будет всего несколько продуктов ... скажем, 5, но каждый критерий или один из критериев может оказаться несколько тысяч, вы можете сэкономить память или перезарядить процессор, выполнив следующие действия:

final Set<String> manufacturers = filterBrands.stream()
        .map(String::toLowerCase)
        .collect(Collectors.toSet());

return filterCategories.stream()
    .flatMap(this::getProductsByCategory)
    .distinct() // optional but might be better performance 
                // if categories share products.
    .filter(product -> manufacturers.contains(product.getManufacturer().toLowerCase()))
    .collect(Collectors.toSet());