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

This topic created in 1951 days ago, the information mentioned may be changed or developed.

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())

5 replies • 2021-02-08 15:24:06 +08:00

jadehare

Feb 8, 2021

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

stevenkang

Feb 8, 2021

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

jadehare

Feb 8, 2021

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

stevenkang

Feb 8, 2021

再补充一个

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

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

monsterxx03

Feb 8, 2021

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

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