使用lambda优化if else语句，以防止null检查？

Question

我有下面这个例子，感觉可读性不是很好。由于getStudents（）可能为null，getTests（）也可能为null，所以我必须进行大量检查。我必须执行isStudentsAvailable（）和isStestAvailable（）。

我如何将这个例子转移到不需要那些检查的lambda表达式中呢？会很乐意得到一些暗示。谢谢

      if(school.isStudentsAvailable()){
        sendMailtoStudents(school.getStudents());
        
        if(school.getStudents().isTestAvailable()) {
        printTests(school.getStudents().getTests());
         }
      }

Answer 1

由于getStudents（）可能为null，getTests（）也可能为null，所以我必须进行大量检查

啊，你有代码味。

有很多方法可以处理NUI的概念（未初始化、未知、不相关）。null是一种方法，Optional之类的包装类型是另一种方法，annotations是第三种方法，sentinel值是第四种方法。

但听起来这不是NUI的案子！听起来getStudents()返回null作为一种表达方式：有零个学生。

换句话说，它传达的是空的，而不是nui。

用null表示empty是错误的，因为与NUI不同，empty具有适合您的类型的约定的语义含义。school.getStudents返回一个列表（或set、collection、map或array--某种表示集合的类型），所有这些类型都能够很好地表示空类型。

换句话说，修复方法是转到getStudents方法的代码，并将returnull替换为returnlist.of()；。

那么您的调用代码就简单多了。而不是：

List<Student> students = getStudents();
if (students != null) {
    for (Student student : students) {
        // code here
    }
}

它简化为：

for (Student student : getStudents()) {
    // code here
}

哪个眼球会立刻喜欢，毫无疑问。读起来容易多了。

但是，假设您不能更改getStudents，因为它是提供的。

然后，当API设计得如此糟糕以至于濒临崩溃时，您应该做的就是编写一个包装器：

public static List<Student> fixedGetStudents(School school) {
    List<Student> students = school.getStudents();
    return students == null ? List.of() : students;
}

那就叫。每次使用它时都尝试‘修复它’意味着每次要使用getStudents时都在编写代码来解决潜在的无效性，这是一个样式错误，因为它违反了DRY（不要重复自己）。