Рефакторинг двух for в потоки java 8

У меня возникла небольшая проблема переписать мои два for в потоки java 8.

// This is a method parameter
Map<String, Collection<String>> preSelectedValues;

List<PersonModel> parameters = parameterSearchService.getParameterNames();

for(Iterator<Map.Entry<String, Collection<String>>> it = preSelectedValues.entrySet().iterator(); it.hasNext();) {
     Map.Entry<String, Collection<String>> entry = it.next();
     for(int i = 0; i < parameters.size(); i++) {
          if (entry.getKey().startsWith(parameters.get(i).getName())) {
               it.remove();
          }
     }
}

Я пробовал следующие потоки вести себя так же, как и раньше:

Map<String, Collection<String>> filteredParameters = preSelectedValues.keySet().stream()
    .filter(x -> isParameterValid(x, parameters))
    .collect(Collectors.toMap(k -> k, v -> preSelectedValues.get(v)));

isParameterValid метод:

private boolean isParameterValid(String parameterId, List<PersonModel> preSelectedValues) {
     return preSelectedValues.stream()
            .anyMatch(v -> !v.getName().startsWith(parameterId));
}

В основном я пытаюсь отфильтровать карту preSelectedValues, которая начинается с параметра parameterId. Но почему-то, когда я использую потоки, он либо фильтрует все, либо ничего.

Пользовательский скаляр GraphQL
Пользовательский скаляр GraphQL
Листовые узлы системы типов GraphQL называются скалярами. Достигнув скалярного типа, невозможно спуститься дальше по иерархии типов. Скалярный тип...
Как вычислять биты и понимать побитовые операторы в Java - объяснение с примерами
Как вычислять биты и понимать побитовые операторы в Java - объяснение с примерами
В компьютерном программировании биты играют важнейшую роль в представлении и манипулировании данными на двоичном уровне. Побитовые операции...
Поднятие тревоги для долго выполняющихся методов в Spring Boot
Поднятие тревоги для долго выполняющихся методов в Spring Boot
Приходилось ли вам сталкиваться с требованиями, в которых вас могли попросить поднять тревогу или выдать ошибку, когда метод Java занимает больше...
Полный курс Java для разработчиков веб-сайтов и приложений
Полный курс Java для разработчиков веб-сайтов и приложений
Получите сертификат Java Web и Application Developer, используя наш курс.
3
0
63
2
Перейти к ответу Данный вопрос помечен как решенный

Ответы 2

Ответ принят как подходящий

Ваш метод isParameterValid не имеет той же логики, что и исходные циклы, поскольку:

  1. Вы переключили экземпляр и аргумент в вызове на startsWith.
  2. Вызов anyMatch с помощью !v.getName().startsWith(parameterId) только говорит вам, не начинается ли хотя бы один элемент List<PersonModel> с parameterId. Исходным условием сохранения записи в Map было то, что все элементы List<PersonModel> не начинаются с parameterId (или, на самом деле, наоборот - parameterId не начинается ни с одного из имен элементов List<PersonModel>).

Поэтому я отказался от метода возврата условия удаления записи из Map:

private boolean isParameterInvalid(String parameterId, List<PersonModel> preSelectedValues) {
     return preSelectedValues.stream()
            .anyMatch(v -> parameterId.startsWith(v.getName()));
}

И тогда конвейер потока может выглядеть так:

Map<String, Collection<String>> filteredParameters = preSelectedValues.entrySet().stream()
    .filter(x -> !isParameterInvalid(x.getKey(), parameters))
    .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));

Кстати, поскольку ваши оригинальные петли мутируют исходный Map, вы можете добиться того же с removeIf.

Если вы пытаетесь изменить исходную карту, вы можете использовать removeIf:

preSelectedValues.keySet().removeIf(
    key -> parameters.stream()
            .map(PersonModel::getName)
            .anyMatch(key::startsWith)
);

Другие вопросы по теме