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

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

已注册用户请登录

这是一个创建于 1772 天前的主题，其中的信息可能已经有所发展或是发生改变。

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 条回复 2021-02-08 15:24:06 +08:00

jadehare

2021-02-08 15:09:40 +08:00

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

stevenkang

2021-02-08 15:11:18 +08:00

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

jadehare

2021-02-08 15:16:43 +08:00

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

stevenkang

2021-02-08 15:17:53 +08:00

再补充一个

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

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

monsterxx03

2021-02-08 15:24:06 +08:00

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

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