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

看完这块代码,我久久不能平静

  •  4
     
  •   ignore · 11 天前 · 10135 次点击

    try { .... if(url == null) { throw new NullPointerException(); } .... } catch (IOException e) { e.fillInStackTrace(); }

    64 条回复    2021-04-09 16:33:25 +08:00
    TmacV2
        1
    TmacV2   11 天前 via iPhone
    为啥
    iRiven
        2
    iRiven   11 天前
    @TmacV2 url 为空的时候直接调用就会抛出 NullPointerException
    ignore
        3
    ignore   11 天前
    真没想到能遇上这样的代码,以前看着别人的截图也就乐乐
    xuanbg
        4
    xuanbg   11 天前
    这个 catch 是为可能是不合法的或者无法访问的 url 准备的
    misaka19000
        5
    misaka19000   11 天前
    可能是为了方便在 catch 中处理空指针异常
    w4w
        6
    w4w   11 天前
    遇到 NullPointerException,严禁使用 catch 来隐藏编码错误
    LGA1150
        7
    LGA1150   11 天前   ❤️ 1
    @w4w #6
    catch 的是 IOException
    pkupyx
        8
    pkupyx   11 天前
    @xuanbg 这个 catch 应该是管网络 IO 不可用的异常
    phpfpm
        9
    phpfpm   11 天前   ❤️ 9
    看完这个帖子,我久久不能平静
    LGA1150
        10
    LGA1150   11 天前   ❤️ 3
    @ignore 能把完整代码贴出来么?显式抛 NPE 很可能是为了快速失败( fail-fast )
    https://stackoverflow.com/questions/45632920/why-should-one-use-objects-requirenonnull/45632988
    ychost
        11
    ychost   11 天前   ❤️ 1
    可能完全是为了终止当前语句块,但是又不想终止方法,可以用下面的骚操作实现:
    do {
    if (url == null) {
    break;
    }

    } while (false)
    unicloud
        12
    unicloud   11 天前
    距离发帖过去 1 小时 20 分钟了,现在心情平静了吗?
    meteor957
        13
    meteor957   11 天前
    现在平静了吗
    altchen
        14
    altchen   11 天前
    现在平静了吗
    楼主你这么容易激动,对不起你这个 ID 啊。:doge
    tabris17
        15
    tabris17   11 天前   ❤️ 4
    背后的原因让人暖心
    xuanbg
        16
    xuanbg   11 天前
    @pkupyx 对的,url 不为空不代表 url 就正确返回结果,请求出现异常是最正常不过的事情。这时候抛出异常显然不合适,所以要用 try/catch 吃下来 IOException,并通过日志输出异常。
    yeqizhang
        17
    yeqizhang   11 天前
    @LGA1150 #7 赞同这位老哥,这个他手动抛空指针了,后面代码不一定会抛... 所以看情况。这代码确实不太好。11 楼那个看起来更舒服点
    ipwx
        18
    ipwx   11 天前
    我觉得这个代码很好啊(前提是出 null 真是异常而不是预期中的错误)。谁知道我一个 null 放后面的库都会干些啥,到时候真有 null 了都不知道怎么调试。
    wolfie
        19
    wolfie   11 天前
    唯一槽点,扔到了 try 里面。没啥大问题。
    guyeu
        20
    guyeu   11 天前
    感觉没啥毛病啊。。入參不合法还不让抛异常了吗
    AoEiuV020
        21
    AoEiuV020   11 天前   ❤️ 1
    没毛病,很可能是等后面自动抛 npe 就晚了,抛异常前已经执行的语句又不能回滚的,
    kincaider
        22
    kincaider   11 天前
    还不平静吗?
    mengkun
        23
    mengkun   11 天前   ❤️ 14
    Kasumi20
        24
    Kasumi20   11 天前
    保护代码在正常条件下执行而已,难道你从来不检查参数的吗
    iseki
        25
    iseki   11 天前 via Android
    我觉得很多时候这是正确的编码方式,Java 里毕竟没有 checkNotNull
    Leviathann
        26
    Leviathann   11 天前 via iPhone   ❤️ 1
    这个就是 guava 的 precondition.checknotnull 吧
    bigbyto
        27
    bigbyto   11 天前 via iPhone
    不是十分的优雅,不过没觉得有太大的问题。
    shm7
        28
    shm7   11 天前 via iPhone
    不晓得🤷‍♂️ 也许是为了把这种业务错误在这个代码块捕捉完也说不定
    TheWidowMaker
        29
    TheWidowMaker   11 天前 via Android
    @guyeu 抛 npe 就很奇怪了吧?
    raaaaaar
        30
    raaaaaar   11 天前
    @mengkun #23 emmmm
    Jaeden
        31
    Jaeden   11 天前
    throw NullPointerException,catch IOException
    LGA1150
        32
    LGA1150   11 天前   ❤️ 2
    @iseki #25
    Objects.requireNonNull
    iseki
        33
    iseki   11 天前
    @LGA1150 thx
    Blessing1
        34
    Blessing1   11 天前
    平常心对待!!!
    Still4
        35
    Still4   11 天前
    代码不全,估摸着 url 来自于其他地方,抛出错误是合理的,抛出 NPE 不合理
    gamexg
        36
    gamexg   11 天前
    非 java 开发
    如果函数参数不允许 null,那么我会在函数开头直接检查
    如果为参数 null,立刻报错并结束函数,防止程序执行一半后碰到空指针错误还要回滚操作。
    另外代码也能够清晰的表明,函数不允许 null 参数。

    不过一般是返回自定义错误,而不是返回 空指针异常
    hhjswf
        37
    hhjswf   11 天前
    俺寻思着,NPE 还用你抛啊
    cgb1021
        38
    cgb1021   11 天前
    是为了把所有错误都放在 catch 里处理
    wutiantong
        39
    wutiantong   11 天前
    搞不懂你们写 java 的点呢[狗头]
    sutra
        40
    sutra   11 天前
    打开 V2EX 我久久不能平静。
    angryfish
        41
    angryfish   11 天前 via iPhone
    看完代码,我也久久不能平静,写代码怎么了
    szq8014
        42
    szq8014   11 天前
    哈哈,楼主,现在轮到我们久久不能平静了
    CantSee
        43
    CantSee   11 天前
    实习生写的吧
    shutan
        44
    shutan   11 天前
    V 友交流之后应该就平静下来了
    shuqin2333
        45
    shuqin2333   11 天前
    没啥毛病啊
    beichenhpy
        46
    beichenhpy   11 天前
    没有前面的代码,不知道 try 是为了干什么
    unco020511
        47
    unco020511   11 天前
    非要解释的话也能解释的通,就是能快速把 npe 抛出来,不然还要到真正使用 url 的时候才能抛出 npe,然后 catch ioExcption 这个也没问题,因为 url != null,不代表就不会出现 io 问题
    passerbytiny
        48
    passerbytiny   11 天前 via Android
    passerbytiny
        49
    passerbytiny   11 天前 via Android
    手机操作容易误操作,蛋疼的很,等我再组织回复
    passerbytiny
        50
    passerbytiny   11 天前 via Android
    第一,不含任何信息的 NullPointerException,就算是方法开头的参数检验,看起来也是怪怪的,你哪怕弄个 NullPointException("参数:参数名") 也比这强;或者说,裸参数构造的 NullPointerException 包含的信息量太少,要是未预期的由 JVM 抛出还行,要是主动去抛出就显得很不负责任。

    第二,< try { .... } catch (IOException e) { e.fillInStackTrace(); >,这是明显的吃掉异常的行为,非常新手或者不负责任的行为。
    sdlzqjf
        51
    sdlzqjf   11 天前
    我不明白楼主为啥不能平静,不知道槽点在哪?从写法上来说这段代码值得商榷,但是从执行上来说是没问题的
    NullPointerException 是运行时异常,IOException 非运行时异常,catch IOException 并不会捕捉到 NullPointerException,仍然会抛给上层,执行起来逻辑是没问题的。
    当然,楼主这段代码没有业务逻辑,所以无法判断 throw NullPointerException 是不是合理
    guyeu
        52
    guyeu   11 天前
    @TheWidowMaker #29 java 自带的 Objects.requireNonNull,Guava 的 Preconditions.chechNotNull 都是抛出空指针异常,这俩 API 的设计者都觉得指针是空的抛个空指针就可以,有什么问题呢?
    lachesis
        54
    lachesis   11 天前
    脱离业务上下文单独看,槽点有,但还没到夸张的地步,能用
    gowk
        55
    gowk   11 天前
    @mengkun 你这个图可笑死我了
    Anarchy
        56
    Anarchy   11 天前 via Android
    槽点在哪?空指针问题提前抛出来没啥问题啊。
    hodur
        57
    hodur   11 天前
    不那么好,但又比用到 url 时自动抛出好一点
    SWYou
        58
    SWYou   10 天前 via iPhone
    看了这个帖子以后,我今天也试了试主动抛出 NPE,感觉可有逼格了。
    crclz
        59
    crclz   10 天前
    必须在函数开头检查引用是否为 null,这是最基础的编程常识。这样就可以区分是调用者出了问题,传入了 null,还是函数体有 bug,在中途访问了 null 。bug 就被局限在小范围内,而不是跨函数找 bug 。

    除此之外,没有代码上下文,唯一可能出问题的就是把 IOException 掩盖掉的问题。


    @guyeu C# visual studio 对传入参数的代码建议也包含 if(x == null) throw new NullReferenceException(nameof(x)),和 java 的思想一致。
    nowto
        60
    nowto   10 天前 via Android
    错误要早发现早暴露,能在编译器暴露的,就不要等到运行期。就算运行期也要能早暴露尽早暴露。所以一般会在 public 函数开始处做参数校验,抛出业务相关异常;而且应该尽量在 javadoc 注明某参数不接受 null,会抛什么样的异常。

    如果不做显式校验,而是使用到这个参数时依靠 jvm 自动报错:
    1 、会晚; 2 、是一个业务无关的异常; 3 、随着重构的进行,入参和参数使用的位置可能会很远,甚至不在同一个方法
    justforlook44444
        61
    justforlook44444   10 天前
    姑且不谈 try catch,
    github 上搜索 throw new NullPointerException,代码有 500 万行,其中很多都是这种写法;
    jdk 里面也有这种写法
    jdk/src/java.xml.crypto/share/classes/javax/xml/crypto/dsig/dom/DOMSignContext.java
    public DOMSignContext(Key signingKey, Node parent) {
    if (signingKey == null) {
    throw new NullPointerException("signingKey cannot be null");
    }
    if (parent == null) {
    throw new NullPointerException("parent cannot be null");
    }
    setKeySelector(KeySelector.singletonKeySelector(signingKey));
    this.parent = parent;
    }
    所以你不能平静的点在哪里?
    DavidDee
        62
    DavidDee   10 天前
    fail early?
    chinvo
        63
    chinvo   10 天前 via iPhone
    检查参数不应该是在方法开头

    if (url == null) throw new ArgumentNullException(nameof(url));

    privatetan
        64
    privatetan   10 天前
    很多源码不都有这样的判断么
    关于   ·   帮助文档   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   3597 人在线   最高记录 5497   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 23ms · UTC 03:32 · PVG 11:32 · LAX 20:32 · JFK 23:32
    ♥ Do have faith in what you're doing.