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

[ignore]

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

[TmacV2]

为啥

[iRiven]

@TmacV2 url 为空的时候直接调用就会抛出 NullPointerException

[ignore]

真没想到能遇上这样的代码,以前看着别人的截图也就乐乐

[xuanbg]

这个 catch 是为可能是不合法的或者无法访问的 url 准备的

[misaka19000]

可能是为了方便在 catch 中处理空指针异常

[w4w]

遇到 NullPointerException，严禁使用 catch 来隐藏编码错误

[LGA1150]

@w4w #6
catch 的是 IOException

[pkupyx]

@xuanbg 这个 catch 应该是管网络 IO 不可用的异常

[phpfpm]

看完这个帖子，我久久不能平静

[LGA1150]

@ignore 能把完整代码贴出来么？显式抛 NPE 很可能是为了快速失败（ fail-fast ）
https://stackoverflow.com/questions/45632920/why-should-one-use-objects-requirenonnull/45632988

[ychost]

可能完全是为了终止当前语句块，但是又不想终止方法，可以用下面的骚操作实现：
do {
if (url == null) {
break;
}

} while (false)

[unicloud]

距离发帖过去 1 小时 20 分钟了，现在心情平静了吗？

[meteor957]

现在平静了吗

[altchen]

现在平静了吗
楼主你这么容易激动，对不起你这个 ID 啊。:doge

[tabris17]

背后的原因让人暖心

[xuanbg]

@pkupyx 对的，url 不为空不代表 url 就正确返回结果，请求出现异常是最正常不过的事情。这时候抛出异常显然不合适，所以要用 try/catch 吃下来 IOException，并通过日志输出异常。

[yeqizhang]

@LGA1150 #7 赞同这位老哥，这个他手动抛空指针了，后面代码不一定会抛... 所以看情况。这代码确实不太好。11 楼那个看起来更舒服点

[ipwx]

我觉得这个代码很好啊（前提是出 null 真是异常而不是预期中的错误）。谁知道我一个 null 放后面的库都会干些啥，到时候真有 null 了都不知道怎么调试。

[wolfie]

唯一槽点，扔到了 try 里面。没啥大问题。

[guyeu]

感觉没啥毛病啊。。入參不合法还不让抛异常了吗