Question

我正在使用SonarQube，它显示以下错误：

Public methods should throw at most one checked exception.

// Noncompliant
public void delete() throws IOException, SQLException { /* ... */ }

// Compliant
public void delete() throws SomeApplicationLevelException { /* ... */ }

这是否意味着，SomeApplicationLevelException是一个父类，而IOException和SQALException是从它派生的？我们应该抛出父类异常？从而坚持只抛出1个检查异常的方法？

因为我有两个我已定义的例外，例如Exception1和Exception2延伸Exception。我的方法说，sampleMethod()抛出它们，即

public void sampleMethod() throws Exception1, Exception2 {
}

此处显示错误。那么我应该将一个类作为父类（比如MainException）并从中派生Exception1和Exception2并抛出父异常类？如下所示：

public void sampleMethod() throws MainException {
}

上述解决方案是否合适？

Answer 1

如果您的应用程序中有一个声明为throws SQLException, IOException的方法，您可能会将内部实现详细信息泄露给方法的用户。具体来说，你说：

您的方法是使用JDBC和文件I / O实现的。您的用户并不关心您的方法是如何实施的;他们只关心你的方法做什么。
您的方法包括它的任何未来版本，绝不会抛出任何其他已检查的异常。如果您将来更改方法以便它可能抛出另一个已检查的异常，则会破坏向后兼容性。

建议是创建自己的特定于应用程序的类（派生自Exception），并且只在您的方法中抛出它。如果您愿意，可以将特定于应用程序的异常中的SQLException或IOException（或任何其他异常）包装为 cause 。

但请注意，有一种思想流派可以说Java checked exceptions are a bad idea（其中一个原因是C＃，还有更多现代语言，如Kotlin，不会检查异常）。

更新：上述答案与问题的第一个版本（编辑＃1）有关。随后更新了这个问题，指出两个抛出的异常是应用程序定义的异常，因此上述大部分理由不再适用。 this post中解释了更新问题的答案。

Answer 2

IOexception和sqlexception都是被检查的异常s，彼此完全不同，现在如果我们从一个异常扩展并抛出父异常，这在java中不是强制性的，那将会是一种误导用户的API。

但是，如果你想在你的应用程序中这样做以避免sonarqube错误，你可以捕获所有特定的异常并抛出一个自定义异常，将原始异常信息包装在异常消息中。

例如

try{

      ///piece of code that throws IOException and SQLException
}catch(IOException | SQLException ex){
   throw new DataException(ex,"Any customized message you want");
}

此DataException将包含在具有此try catch的方法签名的throws子句中。

DataException扩展了Exception类，并通过在构造函数中传递ex，将原始异常包装在自定义异常中，并保留原始异常信息。

SonarQube错误：重构此方法最多抛出一个已检查的异常

2 个答案: