我正在审查一个同事的代码,我遇到一段与此类似的代码:
public X Foo1(Y y) throws Exception { X result = new X(y); result.Foo2(); return result; }
我相信没有必要抛出异常部分,但我有困难证明这一点.如果是更具体的异常(FileNotFound,NoMemory等),那可能是有道理的,但是我认为这是不必要的.有人可以给我一些原因,这可能导致什么问题,为什么这是坏的做法?还是这个代码好吗?
解决方法
抛出声明是方法合同的一部分.定义合同时,您应始终尽可能精确.说抛出异常是一个坏主意.
这是坏的同样的原因,这是不好的做法,说一个方法返回一个对象,当它保证返回一个字符串.
此外,方法的调用者必然要抓住Exception(除非他想传播这个丑陋),而且捕获异常也是一个坏主意.看到这个问题的答案:Is it a bad practice to catch Throwable?