JavaRush/Помощь по Java задачам/Прошу помочь, почему не срабатывает фильтр на стримы

14 уровень

12.01.2023
454просмотров
12комментариев

Прошу помочь, почему не срабатывает фильтр на стримы

Вопрос по задаче

Обсуждается

Предыстория. Предположим, я заполнил массив сам с помощью нехитрого цикла, в результате получилось [10, 20, 30, 40, 50, 60, 70, 80, 90, 100, 110, 120, 130, 140, 150] , в итоге четных должно быть 640, нечетных 560 Стандартное решение с циклами вполне успешно проходит проверку, но я хотел сделать с помощью стримов. И вот незадача. По моей логике при фильтре на индекс должны получаться те же 640 и 560, а выходит... 1200 и 0. 1200 - как раз сумма, т.е. фильтр работает у меня не так как нужно. Гуглил, нашел примерно такую же штуку. 2 варианта: (такой не работает совсем - получается 1200 и 0)

long numberOfPeopleInEvenHouses = 0;
long numberOfPeopleInOddHouses = 0;
System.out.println("Arrays.toString(houses) BEFORE FILTER " + Arrays.toString(houses));
numberOfPeopleInEvenHouses = Arrays.stream(houses).filter(i -> i % 2 == 0).sum();
System.out.println(numberOfPeopleInEvenHouses);
numberOfPeopleInOddHouses = Arrays.stream(houses).filter(i -> i % 2 != 0).sum();
System.out.println(numberOfPeopleInOddHouses);

И такой:

IntStream.range(0, houses.length).filter(i -> i % 2 == 0).forEach(System.out::println);

как результат выводит индексы 0, 2, 4... 14 а если добавить .sum() вместо вывода на экран, то 56. Сумму индексов. А нужно сумму значений элементов по этим индексам. Тоже не совсем, что хочется. Чувствую, что близко... но не могу добить. Прошу помочь, кто "шарит" в стримах или пытался решить задачу таким образом. Спасибо

Комментарии (12)

популярные
новые
старые

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

Стас ПасинковSoftware Developer в Zipy

Master

14 января 2023, 00:18

в вас ніде нема "фільтру на індекс" фільтр працює з елементами стріма, а не з індексами якогось зовсім іншого масиву. стрім у вас із чисел, я так розумію (не знаю шо таке те houses) - от він і сумує ті числа. а назвете ви ту змінну i, чи x, чи indexINDEXindex - туди не будуть попадати індекси, бо це все-одно стрім з чисел, а отже в тій змінній будуть самі числа, значення. тому вам і пропонують всякі ренджі, мапи і все таке "сложне") я б ще редьюс запропонував, але вам мабуть і без того важко))

hidden #598481

Уровень 27

13 января 2023, 12:04

Можно собрать массив в мапу, где ключом будет чётность элемента, а значением сумма соответствующих элементов массива:

Map<Boolean, Integer> collect = IntStream.range(0, houses.length)
		.boxed()
		.collect(Collectors.groupingBy(i -> i % 2 == 0, Collectors.summingInt(i -> houses[i])));

А можно пойти ещё дальше и получить через следующий стрим ключ от наибольшего значения

boolean isEvenGreater = IntStream.range(0, houses.length)
		.boxed()
		.collect(Collectors.groupingBy(i -> i % 2 == 0, Collectors.summingInt(i -> houses[i])))
		.entrySet().stream()
		.max(Comparator.comparingInt(Map.Entry::getValue)).orElseThrow()
		.getKey();

Timur Khasmamedov

Уровень 14

Expert

13 января 2023, 12:42

а чем тогда мой вариант или ребят, что предложили раньше, плох ? он вроде... полегче (особенно второй), чем это :) .map(index -> houses[index]) можете объяснить что делает в деталях ? я так понял собирает из индексов массив новых домов по критериям (здесь это четный индекс) раньше. Ну или как-то так.

IntStream intStream = IntStream.range(0, houses.length).filter(i -> i % 2 == 0)
                .map(index -> houses[index]);

        System.out.println("smth");
        intStream.forEach(System.out::println);

потому что вывод всего этого (пока надеюсь не шибко понятного) безобразия: smth 10 30 50 70 90 110 130 150

hidden #598481

Уровень 27

13 января 2023, 13:28

.map() и разнообразные вариации типа .mapToInt(), .mapToObj() и пр. - это преобразование, или маппинг, одного объекта в другой. В скобках описывается правило, по которому из объекта того типа, который был в стриме до этого, получается новый объект. В данном случае у тебя был стрим чисел, а ты сказал: вместо каждого числа беру соответствующее ему значение из массива houses. Получил из одного стрим другой стрим. Конкретно в данном случае из стрима чисел 0-15 получил стрим чисел значений массива. Вариант ниже неплохой, но он просто делает меньше. Смотри, вот Кира предложила.

IntStream.range(0, houses.length)
         .filter(i -> i % 2 == 0)
         .map(index -> houses[index])
         .sum();

Пройтись по стриму чисел 0-15, отобрать из них только чётные, получить соответствующие им значения из массива и сложить. Получили сумму значений чётных элементов массива, при этом все нечётные мы из стрима просто выкинули. Что надо сделать дальше? Повторить то же самое для нечётных чисел, потом сравнить, потом вывести в соответствующую строку. Мой вариант делает это одновременно и для чётных и для нечётных домов:

Map<Boolean, Integer> sumsMap = IntStream.range(0, houses.length)
		.boxed()
		.collect(Collectors.groupingBy(i -> i % 2 == 0, Collectors.summingInt(i -> houses[i])));

Взять стрим чисел, разделить на две группы чётных и нечётных (первый аргумент в groupingBy), просуммировать для каждой группы значения из массива (второй аргумент в groupingBy) и записать всё это в мапу.

hidden #598481

Уровень 27

13 января 2023, 13:28

В мапе оказываются всего две пары вхождений (false, XXX) и (true, YYY), где XXX - это сумма нечётных значений массива, а YYY - сумма чётных значений массива. Уже всё посчитано, осталось сравнить. Сравнить можно было бы постаринке, например

if (sumsMap.get(true) > sumsMap.get(false)) {
     // сумма чётных значений больше
}

Но ты сам хотел дорешать через стримы. Поэтому я из этих двух вхождений мапы делаю стрим, нахожу наибольшее количество, сравнивая значения, и возвращаю соответствующий ключ. Если вернулся ключ true, то значит больше тех, кто в groupingBy возвращал true при классификации, т.е. чётных.

Timur Khasmamedov

Уровень 14

Expert

13 января 2023, 14:59

Спасибо всем! (и в особенности vk). Пока попытался понять, что такое

IntStream.range(0, houses.length).filter(i -> i % 2 == 0).map(index -> houses[index]).sum()

разложив код на составляющие. Вышло так, что мы "пересобираем" стрим из индексов (это где фильтр) в новый ~~массив~~стрим (см. ниже), но уже... значений (или как-то так), который и суммируем. Дальше тяжело. Хотя да, я тоже пораскладывал и почитал документацию. В целом понятно, но еще не до конца, прямо говоря. Осталось еще раз повторить (особенно) лямбды и стримы, потом вернуться и добить сабж. Топик не закрываю пока что.

hidden #598481

Уровень 27

13 января 2023, 15:08

Да, только не "в новый массив", а "в новый стрим". Советую, несмотря на мнимую компактность записи, при длинных цепочках переносить каждый следующий вызов на новую строку. Так во-первых будет быстрее читаться, а во-вторых IDE будет подсказывать тебе, какой тип данных будет получаться после каждого следующего вызова, см. скриншот ниже.

Timur Khasmamedov

Уровень 14

Expert

13 января 2023, 11:30

Stream<Integer> integerStream = IntStream.range(0, houses.length).filter(i -> i % 2 == 0)
                .mapToObj(index -> houses[index]);
        numberOfPeopleInEvenHouses = integerStream.mapToInt(Integer::valueOf).sum();

        numberOfPeopleInEvenHouses = IntStream.range(0, houses.length).filter(i -> i % 2 == 0)
                .map(index -> houses[index]).sum()

пока разбираюсь, но выходит, что равнозначные результаты просто в первом случае я создаю промежуточный стрим (интежеров), верно ?

Gennady Shevchenko

Уровень 2

13 января 2023, 13:06

Я вас запутал. Я думал у вас есть еще какой-то массив в котором хранятся объекты House. Если вы работаете с интами, то вам метод mapToObj и не нужен, вам достаточно метод map. Промежуточный стрим делать не стоит, это плохой стиль кода.

Timur Khasmamedov

Уровень 14

Expert

13 января 2023, 13:08

Да, тут пока только массивы.

Gennady Shevchenko

Уровень 2

13 января 2023, 13:21

По поводу метода map - метод применяет заданную функцию в каждому элементу стрима. В вашем случае функция заключается в том, чтобы преобразовать индекс в элемент массива, который хранится по этому индексу.

KIRAРазработчик в prosv.ru

Expert

13 января 2023, 08:57

numberOfPeopleInEvenHouses = IntStream
.range(0, houses.length)
.filter(i -> i % 2 == 0)
.map(index -> houses[index])
.sum();