Question

尝试通过一个简单的示例来了解Java 8流语法。看看关于这个主题的其他类似问题，但找不到任何与我的例子相符的解决方案，并且对我有用。基本上我试图使用两个嵌套循环重构以下代码段以使用新的流API：

List<Car> filteredCars = new ArrayList<>();

for (Car car : cars) {
    for (Wheel wheel : wheels) {
        if (car.getColor() == wheel.getColor() &&
                wheel.isWorking() == true ) {
            filteredCars.add(car);
            break;
        }
    }
}

return filteredCars;

管理得出这个返回void：

return cars.stream().forEach(
    car -> wheels.stream()
        .filter(wheel -> wheel.getColor() == car.getColor() &&
                    wheel.isWorking() == true)
        .collect(Collectors.toList()));

上面的流语法有什么问题，我错过了什么？

Answer 1

您无法在同一forEach上执行两个终端操作 - collect和Stream。

相反，您需要通过检查每辆车是否有匹配的工作轮来过滤汽车列表：

List<Car> filteredCars =
    cars.stream()
        .filter (
            car -> wheels.stream()
                         .anyMatch(wheel -> wheel.getColor() == car.getColor() &&      
                                            wheel.isWorking()))
        .collect(Collectors.toList());

Answer 2

问题是，您在List和forEach返回forEach内创建void（s）。这将等同于以下for循环：

for (Car car : cars) {
    List<Car> filteredCars = new ArrayList<>();
    for (Wheel wheel : wheels) {

        if (car.getColor() == wheel.getColor() &&
                wheel.isWorking() == true ) {

            filteredCars.add(car);
            break;
        }
    }
}

return filteredCars; // whoops cannot be accessed (scope) !!!

您可以在filter流上使用cars并在过滤后的流上收集使用collect以获得所需的结果：

Predicate<Car> carCheck = car -> wheels.stream().anyMatch(wheel -> car.getColor() == wheel.getColor() && wheel.isWorking());

List<Car> filteredCars = cars.stream().filter(carCheck).collect(Collectors.toList());

Java 8嵌套循环以流

2 个答案: