“抛出异常”是不好的做法吗？

Question

我正在查看一位同事的代码，但遇到一段类似于以下代码的代码：

public X Foo1(Y y) throws Exception {
    X result = new X(y);
    result.Foo2();
    return result;
}

我相信没有必要，throws Exception但我很难证明这一点。如果它更具体Exception（FileNotFound，NoMemory等等）可能很有意义，但是因为我认为这是不必要的。有人可以给我一些原因，这可能导致什么问题，以及为什么这是不好的做法？还是这个代码可以吗？

Answer 1

该throws声明是方法合同的一部分。定义合同时，您应始终尽可能 精确 。throws Exception因此，说是个坏主意。

出于同样的原因，这是不好的做法，因为不好的做法是说一个方法Object在保证返回a的情况下会返回一个String。

此外，该方法的调用者必须要捕获Exception（除非他想传播这种丑陋的东西），捕获Exception也是一个坏主意。请参阅以下问题的答案：捕获Throwable是否是错误的做法？