如何以最有效的方式迭代列表并为其设置数据？

Question

我正在研究一个问题，其中我需要迭代一个N大小的数组列表，并与另一个列表进行比较，并且对于所有匹配的元素，我需要在其中设置一些数据。下面给出了一个示例和我到目前为止所做的尝试。

List<Emp> emp= new ArrayList<>(); 
List<Student> stud = new ArrayList<>();
 
class Emp -> int id, String status, String name; 
class Student -> int id, String studStatus, String name;

如果学生的id和员工的id匹配，则使用一些默认值更新emp状态。

到目前为止，我尝试了这段代码，但它并不能帮助我获得所需的结果。纠正我在这里做错了什么？

  stud.stream().filter(i -> emp.stream().filter(j ->
 j.getId().equals(i.getId()))
                                .map(i -> i.setStatus(Status.PASS)));

Answer 1

在您的代码中有几点需要注意：

1. .filter应返回一个布尔值，以便对记录进行筛选。但在你的情况下，这是不可能的。您应该对内部筛选器输出进行计数，看看它是否大于0。或者，简化此操作并使用anymatch
2. .map需要相同实体/类的响应。但是当您使用setter时，您返回的是void，因此您的setter并没有真正生效。使用其他自定义设置器返回更新的对象，或者使用代码块返回更新的对象。
3. .map不是终端操作。因此，整个表达式甚至没有被执行。您可能需要像收集这样的终端操作，或者甚至只是一个计数来使轮子运动起来
4. 您似乎使用了一个枚举来表示状态，但类似乎不是这样定义的...

如果你想让你的代码有修正..下面是你可以做的

    stud.stream()
        .filter(i -> emp.stream().filter(j -> j.getId() == i.getId()).count() > 0L) // ensure the outer filter is getting a boolean output
        .map(i -> {
            i.setStudStatus(Status.PASS);
            return i; // unless you return, the setter does not take effect
            })
        .count(); // terminal operation

    stud.stream()
        .filter(s -> {return emp.stream().anyMatch(e -> (e.id == s.id)); }) 
        .forEach(s -> s.setStatus(Status.PASS));