Question

我尝试创建两个列表 - 赔率和赔率如下：

public static void main(String[] args) {
    List<Integer> numbers = new ArrayList<>(Arrays.asList(1, 2, 3, 5, 8, 13, 21));
    List<Integer> odds = new ArrayList<>();
    List<Integer> evens = new ArrayList<>();
    numbers.stream().forEach(x -> x % 2 == 0 ? evens.add(x) : odds.add(x));
}

但它给了我不兼容的类型错误（lambda表达式中的错误返回类型缺少返回值）

将集合过滤到两个新集合的最佳方法是什么？

Answer 1

正如其他答案解释为什么它没有编译我会在你的情况下使用partitioningBy收集器并获取结果列表：

import static java.util.stream.Collectors.partitioningBy;

...

List<Integer> numbers = Arrays.asList(1, 2, 3, 5, 9, 13, 21);

Map<Boolean, List<Integer>> partition = 
    numbers.stream().collect(partitioningBy(x -> x % 2 == 0));

List<Integer> odds = partition.get(false);
List<Integer> evens = partition.get(true);

Answer 2

好吧，你可以通过一个微不足道的修改来编译你现有的代码：

 public static void main(String[] args) {         
     List<Integer> numbers = new ArrayList<>(Arrays.asList(1, 2, 3, 5, 8, 13, 21));
     List<Integer> odds = new ArrayList<>();
     List<Integer> evens = new ArrayList<>();
     numbers.stream().forEach(x -> (x % 2 == 0 ? evens : odds).add(x));
 }

条件?:运算符是表达式，它本身并不是一个有效的语句。我修改后的代码更改了条件运算符的使用，只是选择要添加的列表，然后在其上调用add - 并且该方法调用表达式是一个有效的语句。

另一种方法是collect使用Collectors.partitioningBy - 尽管在这种特殊情况下代码可能比您所拥有的代码更令人困惑。

Answer 3

三元运算符不是声明。如果您正在使用forEach块，则需要有效的Java语句或完整的块：

numbers.stream().forEach(x -> {
    if (x % 2 == 0 ) { 
        pairs.add(x); 
    } else { 
        ods.add(x);
    }
});

java streams - 如何将集合过滤到两个新集合

3 个答案: