番石榴checkNotNull有什么意义

我对Guava还是很陌生（说实话，我不是“很新”，我是这个主题的新手），所以我决定阅读一些文档，并在阅读本文时感到非常惊讶：

com.google.common.base.Preconditions.checkNotNull(...)

我不明白这种方法的意义。这意味着不要这样做：

myObject.getAnything();

（NullPointerException如果myObject为null ，则可能会导致）

我应该用

checkNotNull(myObject).getAnything();

这将抛出NullPointerException如果myObject是空和返回myObject，如果它不为空。

我很困惑，这可能是有史以来最愚蠢的问题，但是...

这有什么意义呢？在我可以想到的任何情况下，这两行所做的事情与结果完全一样。

我什至不认为后者更具可读性。

所以我一定想念一些东西。它是什么？

这个想法是要快速失败。例如，考虑这个愚蠢的类：

public class Foo {
    private final String s;

    public Foo(String s) {
        this.s = s;
    }

    public int getStringLength() {
        return s.length();
    }
}

假设您不想允许使用null值s。（否则getStringLength将抛出NPE）。照原样讲课，到您赶上时，null已经太晚了-很难找到谁把它放在那里。罪魁祸首很可能是完全不同的一类，而这种Foo情况本来可以在很久以前就建成。现在，您必须对代码库进行梳理，以找出谁可能在其中输入了null值。

相反，想象一下这个构造函数：

public Foo(String s) {
    this.s = checkNotNull(s);
}

现在，如果有人把一个null在那里，你会发现马上-你就会有堆栈跟踪指向正是你走错了电话。

另一个有用的时间是，如果要在执行可以修改状态的操作之前检查参数。例如，请考虑此类，该类将计算所获得的所有字符串长度的平均值：

public class StringLengthAverager {
    private int stringsSeen;
    private int totalLengthSeen;

    public void accept(String s) {
        stringsSeen++;
        totalLengthSeen += s.length();
    }

    public double getAverageLength() {
        return ((double)totalLengthSeen) / stringsSeen;
    }
}

调用accept(null)将引发NPE抛出-但在stringsSeen增加之前不会发生。这可能不是您想要的；作为类的用户，我可能希望，如果它不接受空值，那么如果您传递空值，则其状态应该保持不变（换句话说：调用应该失败，但不应使对象无效）。显然，在此示例中，您也可以通过s.length()在递增之前获取来解决此问题stringsSeen，但是您可以看到对于更长且涉及更多的方法，首先检查所有参数是否有效，然后再修改状态可能是有用的：

    public void accept(String s) {
        checkNotNull(s); // that is, s != null is a precondition of the method

        stringsSeen++;
        totalLengthSeen += s.length();
    }

myObject.getAnything(); （如果myObject为null，则可能导致NullPointerException）

不，只要有，它就会抛出NPE myObject == null。在Java中，没有机会使用null接收方调用方法（理论上是静态方法，但是可以并且应该始终在没有任何对象的情况下调用它们）。

我应该用 checkNotNull(myObject).getAnything();

不，你不应该。这将是相当多余的（Update）。

您应该使用checkNotNull以便快速失败。没有它，您可能会将非法传递null给另一种方法，进而将其传递给其他方法，依此类推，直到最终失败。然后，您可能需要一些运气才能发现实际上第一个方法应该被拒绝null。

yshavit的答案提到了一个重要的观点：传递非法值是不好的，但是存储它并在以后传递它更糟。

更新资料

其实，

 checkNotNull(myObject).getAnything()

也很有意义，因为您明确表达了不接受任何null的意图。没有它，有人可能会认为您忘记了支票并将其转换为类似

 myObject != null ? myObject.getAnything() : somethingElse

太太，我认为支票不值得冗长。用一种更好的语言，类型系统会考虑可为空性，并给我们一些语义糖，例如

 myObject!!.getAnything()                    // checkNotNull
 myObject?.getAnything()                     // safe call else null
 myObject?.getAnything() ?: somethingElse    // safe call else somethingElse

为nullable myObject，而仅当myObject已知非null时才允许使用标准点语法。

几分钟前，我已经阅读了整个主题。但是，我很困惑为什么要使用checkNotNull。然后查看Guava的Precondition类文档，我得到了我所期望的。过量使用checkNotNull肯定会降低性能。

我的想法是checkNotNull，数据验证需要有价值的方法，它直接来自用户，也可能来自最终的API与用户交互。不应在内部API的每种方法中使用它，因为使用它您不能停止异常，而是更正内部API以避免Exception。

根据DOC： 链接

使用checkNotNull：

public static double sqrt(double value) {
     Preconditions.checkArgument(value >= 0.0, "negative value: %s", value);
     // calculate the square root
}

关于性能的警告

此类的目的是提高代码的可读性，但是在某些情况下，这可能会付出巨大的性能代价。请记住，用于消息构造的参数值必须全部经过认真的计算，并且即使先决条件检查成功后，自动装箱和varargs数组的创建也可能发生（这几乎总是在生产中进行）。在某些情况下，这些浪费的CPU周期和分配可能构成一个真正的问题。性能敏感的前提条件检查始终可以转换为惯用格式：

if (value < 0.0) {
     throw new IllegalArgumentException("negative value: " + value);
}