相关疑难解决方法(0)

考虑这一行:

if (object.getAttribute("someAttr").equals("true")) { // ....

显然这条线是一个潜在的错误,属性可能是null,我们将得到一个NullPointerException.所以我们需要将它重构为以下两种选择之一:

第一种选择:

if ("true".equals(object.getAttribute("someAttr"))) { // ....

第二种选择:

String attr = object.getAttribute("someAttr");
if (attr != null) {
    if (attr.equals("true")) { // ....

第一个选项很难读,但更简洁,而第二个选项在意图上是明确的,但是很冗长.

在可读性方面,您更喜欢哪个选项？