V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
stevenkang
V2EX  ›  编程

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

  •  
  •   stevenkang · 2021-02-08 15:07:51 +08:00 · 1566 次点击
    这是一个创建于 1145 天前的主题,其中的信息可能已经有所发展或是发生改变。

    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
        1
    jadehare  
       2021-02-08 15:09:40 +08:00
    记得 java 有个说法是 null 放在 == 前面,少写了=会报错
    stevenkang
        2
    stevenkang  
    OP
       2021-02-08 15:11:18 +08:00
    @jadehare 这一行代码重点其实不在 null 是否在前,而是 判空 写法等于自杀
    jadehare
        3
    jadehare  
       2021-02-08 15:16:43 +08:00
    @stevenkang #2 理解错了,看你下面注释以为只是说 null 而不是这个 null 的判断
    stevenkang
        4
    stevenkang  
    OP
       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
        5
    monsterxx03  
       2021-02-08 15:24:06 +08:00
    写傻了, 这种用 sonarqube 扫一下都能直接检查出来的.

    话说 null 放前面是从 C 里带来的习惯, C 里 if (a=1) {} 能编译过的, 但逻辑是不对的.
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   2924 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 29ms · UTC 14:14 · PVG 22:14 · LAX 07:14 · JFK 10:14
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.