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

[ignore]

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

[AoEiuV020]

没毛病，很可能是等后面自动抛 npe 就晚了，抛异常前已经执行的语句又不能回滚的，

[kincaider]

还不平静吗？

[mengkun]

[Kasumi20]

保护代码在正常条件下执行而已，难道你从来不检查参数的吗

[iseki]

我觉得很多时候这是正确的编码方式，Java 里毕竟没有 checkNotNull

[Leviathann]

这个就是 guava 的 precondition.checknotnull 吧

[bigbyto]

不是十分的优雅，不过没觉得有太大的问题。

[shm7]

不晓得🤷‍♂️ 也许是为了把这种业务错误在这个代码块捕捉完也说不定

[TheWidowMaker]

@guyeu 抛 npe 就很奇怪了吧？

[raaaaaar]

@mengkun #23 emmmm

[Jaeden]

throw NullPointerException，catch IOException

[LGA1150]

@iseki #25
Objects.requireNonNull

[iseki]

@LGA1150 thx

[Blessing1]

平常心对待！！！

[Still4]

代码不全，估摸着 url 来自于其他地方，抛出错误是合理的，抛出 NPE 不合理

[gamexg]

非 java 开发
如果函数参数不允许 null，那么我会在函数开头直接检查
如果为参数 null，立刻报错并结束函数，防止程序执行一半后碰到空指针错误还要回滚操作。
另外代码也能够清晰的表明，函数不允许 null 参数。

不过一般是返回自定义错误，而不是返回 空指针异常

[hhjswf]

俺寻思着，NPE 还用你抛啊

[cgb1021]

是为了把所有错误都放在 catch 里处理

[wutiantong]

搞不懂你们写 java 的点呢[狗头]

[sutra]

打开 V2EX 我久久不能平静。