梳理陈年老系统遇到的一些神奇写法，欢迎品鉴

[stevenkang]

1 、REVERSE_INULL

log.info("xxRequest, param: {}", req.getOrderId())
if (req == null) {
    // xxx
}

2 、FORWARD_NULL

if (null == map && map.size() < 1) {
    // xxx
}

// null 写在前面更有可读性？

3 、这种不知道叫啥

dto.setAbc(ThisIsAVeryLongObjectNameBecauseItIsVeryLongItIsEasyToMisreadTheWordEnums.getEnumByCode(item.getAbc()) == null ? null : ThisIsAVeryLongObjectNameItShouldBeShorterOrAReasonableLineBreakToAvoidErrorsEnums.getEnumByCode(item.getAbc()).getDesc())

[jadehare]

记得 java 有个说法是 null 放在 == 前面，少写了=会报错

[stevenkang]

@jadehare 这一行代码重点其实不在 null 是否在前，而是 判空 写法等于自杀

[jadehare]

@stevenkang #2 理解错了，看你下面注释以为只是说 null 而不是这个 null 的判断

[stevenkang]

再补充一个

4 、IDENTICAL_BRANCHES
```code
if (result != null && result.isSuccessful() && result.getData() != null) {
response.put("data", result.getData());
} else {
response.put("data", result.getData());
}
```

上面的例子都是反面教材，请仔细甄别。

[monsterxx03]

写傻了, 这种用 sonarqube 扫一下都能直接检查出来的.

话说 null 放前面是从 C 里带来的习惯, C 里 if (a=1) {} 能编译过的, 但逻辑是不对的.