考虑以下代码：
if (object.getAttribute("someAttr").equals("true")) { // ....
显然这行代码有潜在的bug，属性可能为null，我们会得到一个NullPointerException。所以我们需要将其重构为以下两种选择之一：
第一个选择：
if ("true".equals(object.getAttribute("someAttr"))) { // ....
第二个选项：
String attr = object.getAttribute("someAttr");
if (attr != null) {
if (attr.equals("true")) { // ....
第一种选项阅读起来比较困难，但更加简洁，而第二种选项意图清晰，但冗长。
就可读性而言，您更喜欢哪个选项？
我一直使用的是
if ("true".equals(object.getAttribute("someAttr"))) { // ....
因为尽管它阅读起来有点困难，但更加简洁，我认为它足够易读，所以您很容易习惯它
在第二种选择中，你可以利用短路运算符&&:
String attr = object.getAttribute("someAttr");
if (attr != null && attr.equals("true")) { // ....
有些情况下，简洁的方法一开始可能感觉不对，但实际上会变得习惯。这是其中之一；另一个情况是：
String line;
while ((line = bufferedReader.readLine()) != null) {
// Use line
}
在一个条件语句中出现副作用？这是不可想象的！但是，当你认识到特定的模式时，它实际上比其他选择更好。
这种模式很常见，尤其在Java中，我认为任何有经验的开发人员都应该能够识别它。结果非常简洁。（有趣的是，我有时会看到使用相同习惯用法的C#代码，但在C#中，等号运算符与字符串一起使用也很好用。）
最重要的是：使用第一个版本，并熟悉它。
我喜欢选项1，而且我认为它足够易读。
顺便说一下，选项3是引入一个带有默认值参数的getAttribute方法。
在功能等效的情况下，始终追求更短的代码。特别是在像这样不会牺牲可读性的情况下。
我有另一个答案;
List<Map<String, Object>> group = jjDatabase.separateRow(db.Select("SELECT * FROM access_user_group WHERE user_id=1 ;"));
for (int j = 1; j < 100; j++) {
String rulId="0";//defult value,to privent null pointer exeption in group_c
try {
rulId = group.get(0).get("group_c" + j)).toString();
} catch (Exception ex) {
ServerLog.Print( "Handeled error in database for " + "group_c" + (j < 10 ? "0" + j : j) +"This error handeled and mot efect in program");
rulId = "0";
}}
这是我的方法，需要一个PropertyUtil类，但只需编写一次：
/**
* Generic method to encapsulate type casting and preventing nullPointers.
*
* @param <T> The Type expected from the result value.
* @param o The object to cast.
* @param typedDefault The default value, should be of Type T.
*
* @return Type casted o, of default.
*/
public static <T> T getOrDefault (Object o, T typedDefault) {
if (null == o) {
return typedDefault;
}
return (T) o;
}
客户端代码可以这样做：
PropertyUtil.getOrDefault(obj.getAttribute("someAttr"), "").equals("true");
或者，对于一个列表：
PropertyUtil.getOrDefault(
genericObjectMap.get(MY_LIST_KEY), Collections.EMPTY_LIST
).contains(element);
或者对于 List 的使用者，它会拒绝 Object：
consumeOnlyList(
PropertyUtil.getOrDefault(
enericObjectMap.get(MY_LIST_KEY), Collections.EMPTY_LIST
)
)
默认情况下可能是空对象模式的实现https://en.wikipedia.org/wiki/Null_Object_pattern
Util.isEmpty(string) - 返回 string == null || string.trim().isEmpty()
Util.notNull(string) 如果 string == null，则返回 ""，否则返回 string。
Util.isNotEmpty(string) 返回 ! Util.isEmpty(string)
我们有一个约定，对于字符串而言，Util.isEmpty(string) 语义上表示为 true，Util.isNotEmpty(string) 语义上表示为 false。
if (object.getAttribute("someAttr") != null && object.getAttribute("someAttr").equals("true")) { // ....
（我将不再使用它）