在字符串文字上调用equals

Question

我只是在整理我的代码而已，这是一段：

String saving = getValue();
if(saving != null && saving.equals("true")){
   // do something
}

然后我想到了另一种方法来摆脱检查null：

if("true".equals(saving)){
   // do something
}

绝对可以，但是 这样 做 安全 吗？我的意思是字符串文字存储在公共池中，而字符串对象create by
new在堆上。但是常量池中的字符串也是对象，对吗？

但是，尽管它使代码更短，但这似乎仍然不是正确的选择。

Answer 1

这是安全的-并且如您所见，这是避免空指针的一种好方法。

您提到了newfor字符串的使用。许多Java静态代码分析工具会建议始终使用文字new String("foo");。

编辑：

如果您愿意，甚至可以使用：

if (Boolean.valueOf(saving)) {
    ...
}

根据文档，通过null将返回false。