从ArrayList中移除奇数元素

Question

public static List<Integer> removeOddNumbers(ArrayList<Integer> list) {
        if (list.isEmpty()) { throw new Error(); }
        List<Integer> toRemove = new ArrayList<>();

        for (int i : list) {
            if (i % 2 != 0) { toRemove.add(i); }
        }
        list.removeAll(toRemove);
        return list;
}

我试图从ArrayList中删除所有奇数元素，然后返回该ArrayList。

我在第一行发现一个指向列表整数的错误

测试：

ArrayList<Integer> arrayList = new ArrayList<Integer>();
Collections.addAll(arrayList, 3, 5, 6, 24, 7, 9, 1, 8, 28, 11);
ArrayList<Integer> result = removeOddNumbers(arrayList);
System.out.println(result);

结果：

[6, 24, 8, 28]

Answer 1

lambda方式：

public static List<Integer> filter(List<Integer> numberList) {
    return numberList.stream()
        .filter(number -> number % 2 != 0)
        .collect(Collectors.toList());
}

您应该使用以下命令调用此方法：

List<Integer> list = Arrays.asList(3, 5, 6, 24, 7, 9, 1, 8, 28, 11);
List<Integer> result = removeOddNumbers(numbers);
System.out.println(result);

问题是该方法的返回类型是< code>List

ArrayList<Integer> arrayList = new ArrayList<Integer>();
Collections.addAll(arrayList, 3, 5, 6, 24, 7, 9, 1, 8, 28, 11);
List<Integer> result = removeOddNumbers(arrayList);
System.out.println(result);

N. B@YCF_L发布的方法编辑你通过参数传递的列表，我的方法改为创建一个新列表，保持原始列表不变

Answer 2

如果您使用的是Java 8，则可以只使用Collection：：removeIf：

list.removeIf(i -> i % 2 != 0);

完整的方法应该如下所示：

public static List<Integer> removeOddNumbers(List<Integer> list) {
    list.removeIf(i -> i % 2 != 0);
    return list;
}

例

List<Integer> list = new ArrayList<>(Arrays.asList(1, 2, 3, 6, 5, 4));
list.removeIf(i -> i % 2 != 0);
System.out.println(removeOddNumbers(list));

产出

[2, 6, 4]

Answer 3

您应该使用迭代器从列表中删除项。这将节省存储奇数元素的临时列表的使用。

public static List<Integer> removeOddNumbers(List<Integer> arrayList) {
    Iterator<Integer> itr = arrayList .iterator();
    while (itr.hasNext())
    {
        int x = (Integer)itr.next();
        if (x % 2 != 0) {
            itr.remove();
        }
    } 
    return arrayList;
}

运筹学

你可以使用Java8获得更简单的代码。