在同一个 if 语句中检查 null 并检查对象属性是一个好习惯吗

Question

在同一个 if 语句中检查 null 并检查对象属性是一个好习惯吗？

考虑以下代码：

if (jamesBlunt != null && jamesBlunt.isReallyBad()) {
   // Don't buy his records
}

这个特定的代码是用java编写的，我知道表达式是从左到右计算的，所以从技术上讲它不会抛出NullPointerException，但总的来说，这在任何语言中都是好的做法吗？

Answer 1

当您使用 OR 语句时，如果 jamesBlunt 为 null，则会出现 NullPointerException。您应该使用 and，因为如果左侧语句为 false，则整个表达式将为 false。

使用：

if (jamesBlunt != null && jamesBlunt.isReallyBad()) {
   // Don't buy his records
}

当 Java 7 发布时，您可以使用快捷方式，

if(jamesBlunt?.isReallyBad() {
       // Don't buy his records
}

但在此之前，显式检查 null 将是最佳实践。 （事实上​​，最好的做法是不使用 jamesBlunt 对象...）

Answer 2

我假设 || 是一个拼写错误，而您的意思是 && :)

回答您的问题：这取决于。

jamesBlunt 为 null 是否有意义？如果没有，那么最好的做法是这样：

void buyARecord(Artist jamesBlunt) {
    if (jamesBlunt == null) {
        throw new IllegalArgumentException("James should never be null!");
    }
}

如果 jamesBlunt 为 null 确实有意义，那么你的方法就很好，假设 null 和 isReallyBad 在语义上意味着同样的事情。如果它们在语义上意味着不同的事物，那么您可能不应该将它们组合在一行上。

使用其他语言时您确实需要小心。许多（Java、C++、C# 等）的行为方式相同，但有些可能从右到左求值或延迟求值。请特别注意 Lisp 和 Scheme 等函数式语言，因为它们的行为往往与 Java 和 C# 等面向对象语言不同。

Answer 3

你想使用 &&而不是“或”。

是的，在同一个 if 语句中检查 null 是一个很好的做法，替代方案（嵌套两个 if）很丑陋，因为会增加更多缩进。

但之前检查也可以：特别是如果你想做一些错误检查。

Answer 4

就我个人而言，我会将这些分开。空检查是在函数开始时执行的一种操作，通常用于验证函数参数，通常称为“保护子句”。这些检查通常会引发异常或返回函数定义的错误代码。

这些可以使下面的代码更清晰，避免多层嵌套。如果您希望在相关对象上测试其他内容，它还可以避免重复空检查的需要。