Как правильно работать с данными List, используя Stream API?

Question

[dc65k]

Всем привет. Подскажите пожалуйста, как правильно исправить код, чтобы работать с данными, в данном случае с List. Я написал три, два из которых закомментированы, но я так понимаю, что проблема везде одна.

package stream;

import java.util.ArrayList;
import java.util.List;

public class Test5 {
    public static void main(String[] args) {
        List list = new ArrayList<>();
        list.add(3);
        list.add(8);
        list.add(1);
        list.add(5);
        list.add(9);

        List result = filterArray(list);

        System.out.println(result);
    }

    public static List filterArray(List list) {
        System.out.println(list);

        List result;

//        list.stream()
//            .forEach(element -> {
//                System.out.println(element);
//                result.add(element);
//            });

//        list.stream()
//            .map(element -> {
//                return element * 2;
//            })
//            .forEach(element -> {
//                result.add(element);
//            });

        list.stream()
            .filter(element -> {
                return element % 2 == 0;
            })
            .forEach(element -> {
                result.add(element);
            });

        return result;
    }
}

Добавлю, что вопрос именно, если я изначально работаю с List.
С вариантом, если инициализировать значения в stream, я разобрался.

public class Test6 {
    public static void main(String[] args) {
        Stream<Integer> stream1 = Stream.of(1, 2, 3, 4, 5, 1, 2);

        List result;

        result = stream1
                .filter(el -> {
                    System.out.println("!!!");
                    return el % 2 == 0;
                })
                .map(element -> {
                    return element * 2;
                })
                .collect(Collectors.toList());

        System.out.println(result);
    }
}

Comments

[Сергей Горностаев]

Никогда не изменяйте в стриме состояние.

[dc65k]

Сергей Горностаев, Как же правильно сделать?

[Сергей Горностаев]

dc65k, вам написали правильный ответ.

Answer 1

[Денис Загаевский]

return list.stream()
    .filter(....)
    .collect(Collectors.toList());

Comments

[dc65k]

Поправьте, пожалуйста, что не так.

package stream;

import java.util.ArrayList;
import java.util.List;
import java.util.stream.Collectors;

public class Test6 {
    public static void main(String[] args) {
        List list = new ArrayList<>();
        list.add(3);
        list.add(8);
        list.add(1);
        list.add(5);
        list.add(9);

//        List result = filterArray(list);

        List result = list.stream()
                .filter(el -> {
                    return el % 2 = 0;
                })
                .collect(Collectors.toList());

        System.out.println(result);

    }

//    public static List filterArray(List list) {
//        return list.stream()
//                .filter(el -> {
//                    return el % 2 = 0;
//                })
//                .collect(Collectors.toList());
//    }
}

[Денис Загаевский]

dc65k, эм, наверное то, что интерфейс List дженериковый, и тебе как минимум надо писать везде List<Integer>.
Формулируй вопросы более четко.

[dc65k]

Денис Загаевский, Похоже, я что-то ещё упускаю.

package stream;

import java.util.ArrayList;
import java.util.List;
import java.util.stream.Collectors;

public class Test6 {
    public static void main(String[] args) {
        List<Integer> list = new ArrayList<>();
        list.add(3);
        list.add(8);
        list.add(1);
        list.add(5);
        list.add(9);

//        List<Integer> result = filterArray(list);

        List<Integer> result;
        result = list.stream()
                .filter(el -> {
                    return el % 2 = 0;
                })
                .collect(Collectors.toList());
//
        System.out.println(result);

    }

//    public static List<Integer> filterArray(List<Integer> list) {
//        return list.stream()
//                    .filter(el -> el % 2 = 0)
//                    .collect(Collectors.toList());
//    }

}

[dc65k]

Bavashi, Спасибо.

[Денис Загаевский]

dc65k, ты напиши толком, что не так-то?

[dc65k]

Денис Загаевский, Спасибо, ты всё правильно подсказал. Я не заметил ошибку в этой строке: return el % 2 = 0;