java取消引用可能的空指针

Question

在我的代码中，我收到了上述警告。这是我得到它的代码部分，

try {
        fileFile = new File(Main.class.getProtectionDomain().getCodeSource().getLocation().toURI());
    } catch (URISyntaxException | NullPointerException e) {
    }
    finally {
        if (fileFile.getPath()!= null){
            strPathName = fileFile.getPath();
        }
        if (fileFile.getName() != null){
            strFileName = fileFile.getName();
        }
    }  

if (fileFile.getPath()!= null){ 行是带有警告的行。 此代码不是 Main 类的一部分。它在同一个包的另一个类文件中的另一个类中。

我对编程不是很有经验，但我相信我几乎做了所有事情来防止或捕获空指针异常。为什么我仍然得到它，我能做些什么来摆脱它？感谢您的帮助。

在阅读了您的所有提示后，我解决了它。完整代码如下：

public static ArrayList<String> getCurrentPath() {

    File fileFile;
    String strPathName, strFileName;
    ArrayList<String> arrPathFileName;

    strFileName = null;
    strPathName = null;

    try {
        fileFile = new File(Main.class.getProtectionDomain().getCodeSource().getLocation().toURI());
        if (fileFile.getPath()!= null){
            strPathName = fileFile.getPath();
        }
        if (fileFile.getName() != null){
            strFileName = fileFile.getName();
        }
    } catch (URISyntaxException use) {
    }     
    arrPathFileName = new ArrayList<>(); 
    arrPathFileName.add(strPathName);
    arrPathFileName.add(strFileName);

    return arrPathFileName;
}

如前所述，我只是将 if 语句放入 try 块并删除 finally 块。

顺便说一句，还尝试将两个 if 块合并为一种方式：

if (fileFile != null){
            strPathName = fileFile.getPath();
            strFileName = fileFile.getName();
        }

但这会产生一个警告，即 fileFile 永远不会变为 null。 （从一开始我的观点是什么，所以警告“取消引用可能的空指针”真的让我很困惑。）

Answer 1

因此，如果您在第一行抛出异常，您的变量将不会被分配给File，并且会保留它的先前值（null，如果之前没有分配）。您的异常被捕获，然后您继续使用该未分配的变量。因此发出警告。请参阅下面的注释代码。

try {
        fileFile = // exception thrown. Variable not assigned
} catch (URISyntaxException | NullPointerException e) {
        // exception caught
    }
    finally {
       // unassigned variable used here...
        if (fileFile.getPath()!= null){
            strPathName = fileFile.getPath();
        }
        if (fileFile.getName() != null){
            strFileName = fileFile.getName();
        }
    }  

如果可行的话，我宁愿在 try 块中限定和使用变量。在您的 finally 块中，您需要尽可能小心，因为您可以从 try 块中的大多数地方来到它。

顺便说一句：

Main.class.getProtectionDomain().getCodeSource().getLocation().toURI();

如果您确实获得了 NPE，将会给您带来巨大的问题。以上哪个解决了 null ？我可能会更明确一点，这样您就可以检查每次调用的空值，并明确地确定哪个调用给了您空值。累吗？不幸的是。

Answer 2

“空指针取消引用”是计算机用空值调用方法的说法。稍微复杂一点，但你说你是新手，所以我想保持简单。

我们来看一个例子：

String s = null;
s = s.toUpperCase();

这是一个关于什么是空指针取消引用的简单示例。 s 是一个空引用（它的值为空），当我们取消引用时（获取它的值）我们有null，当我们在null 上调用toUpperCase() 时，出现了可怕的错误，因为null根本没有任何方法！ Java 会抛出一个 NullPointerException 来具体说明。

---

现在，回到你的代码，因为 fileFile 是在 try 块中分配的，我假设它在它之前被设置为 null 以避免 Java 对未初始化的变量大喊大叫。 （这一切都很好且正确。）在这个 try-block 中，如果您的 catch-block 发生任何异常，它将停止 try-block（意味着 fileFile 不会获得新值，这意味着它仍然是空）。

现在您会注意到警告是 可能 空指针取消引用。这意味着它不一定是null，但可能是！ （在我上面的示例中，它始终是一个用于比较的空指针取消引用。）具体来说，如果catch 捕获一个异常，它将为空。

需要明确的是，问题是这样的： fileFile.getPath()。这就像说它可能是null.getPath()，恶心。看起来你试图避免空指针问题，你应该做的是if (fileFile != null) {。然后在 if 里面做你想做的事。

---

另外，因为您似乎包含它以避免此警告，所以我会认真从 catch 块中删除 NullPointerException。这并不能帮助您避免警告。如果你想让我解释为什么它不好，你可以发表评论，我会的，否则就相信我的话，它对你没有帮助。