坚持使用Java 8流

Question

我正在尝试学习如何在Java 8中使用Streams，但我不知道如何在这里进行操作。

我有一份课程清单。 我需要知道一个学期的所有课程是否都没有学生，如果是这样，那就做点什么。 我想出了下面的代码，但是只要任何课程被迭代而没有任何学生，这就会给出Null Pointer Exception。我需要知道如何纠正它：

List<Student> students = semester.getCourses().stream().flatMap(course -> course.getStudents().stream())
                .filter(Objects :: nonNull).collect(toList());
        if (CollectionUtils.isEmpty(students)){
            //cancel the semester or do something
        }

public class Semester{

 int semId;
 List<Course> courses;
}

public class Course{
 int courseId;
 List<Student> students;
}

Answer 1

boolean allCourseHaveEmptyStudens=semester.getCourses().stream()
.allMatch(c-> c.getStudents()==null || c.getStudents().size==0)
;

我认为这足以满足您的要求。

注意：它可能会出现编译错误，因为我不使用编辑器工具。

Answer 2

在实际的代码NullPointerException可能来自course是null或course.getStudents()是null 。
这个过滤filter(Objects :: nonNull)过滤filter(Objects :: nonNull)很无奈。 它不会过滤null Student ，这不是您的要求。
此代码应该是您正在寻找的：

List<Student> students = 
 semester.getCourses()
         .stream()
         .filter(Objects::nonNull) // filter out null Course objects
         .map(Course::getStudents)  
         .filter(Objects::nonNull) // filter out null Student List
         .flatMap(Collection::stream)                                        
         .collect(toList());

还要注意，在任何地方添加空检查都不好：它使“真实逻辑”的可读性降低。
您可以通过在声明中初始化它们来至少避免这些用于集合字段，例如：

public class Semester{
 int semId;
 List<Course> courses = new ArrayList<>();
}

public class Course{
 int courseId;
 List<Student> students = new ArrayList<>();
}