我最近看到一些代码,其中几乎每个构造函数抛出异常,并在返回int的方法中抛出下面代码之类的运行时异常:
if(condition){
return 1;
}
if(condition){
return 2;
}
if(condition){
return 3;
}
throw new RuntimeException("Unreachable code");
// method ends here
我不会在那里亲自抛出异常,因为我会使用if和else if语句构造它,在这种特殊情况下,你的代码根本不会因为它无法满足其中一个条件.
有很多地方你可以抛出运行时异常,如果你的代码工作正常就永远无法达到,有时似乎作者不信任代码工作,就上面的代码块而言.此外,如果它没有正确初始化,每个构造函数都可以抛出异常,但是你也可以构造它以使对象为null – 例如,你可以在main中检查它.
基本上,我要问的是什么时候值得抛出异常?
解决方法:
例外的重点是沟通特殊情况.
从这个意义上讲:如果你的例子中所有条件都是假的绝对意外,并且没有有效的返回值来表明这种情况,那么抛出RuntimeException是合理的做法;但我可能会将消息更改为:
throw new RuntimeException("All conditions failed: " + some data)
如上所述:它是关于沟通;在这种情况下,调试问题的人.因此,在此处包含所需的信息可能会有所帮助,这些信息可以理解为什么所有这些检查都是错误的.
关键是:该方法有合同;该合同应包括此类细节.含义:如果该方法是公共的,您应该添加带有清晰描述的@throws RuntimeException.
在这种情况下使用RuntimeException也是一种有效的做法;因为您不希望污染您的方法签名,并在整个地方使用已检查的例外.
编辑:当然,需要平衡.示例:我的类通常如下所示:
public class Whatever {
private final Foo theFoo;
public Whatever(Foo theFoo) {
Objects.requireNonNull(theFoo, "theFoo must not be null");
this.theFoo = theFoo;
所以,我的构造函数可能会抛出一个NPE;是.但是:只有那里.我的所有方法都可以依赖于所有字段都被初始化为非null的事实;它们是最终的,所以它们总是非空的.
意思是:一个人必须保持合理;并“发展”一种感觉:哪些问题是特殊但可能的;哪些是不可能的,你不会污染你的代码到处检查它们.
最后;只是为了说清楚 – 添加例外只是等式的一部分.什么东西抛出,那么你需要抓住一些东西!因此,如上所述:平衡进来.无论你在代码中做什么都必须“增加价值”.如果您的代码没有达到明确的目的,那么很可能是:您不需要它!
标签:java,oop,exception,exception-handling,throw
来源: https://codeday.me/bug/20190627/1307762.html