为什么当异常已更改为自定义异常时，SonarQube 会抱怨通用异常

Question

我的任务是为其他人长期以来制作的应用程序进行 JUnit 测试，我无法修改原始代码，所以当我遇到这样的事情时：

public String clean(String url) throws Exception {
    if (url.indexOf(invalidURL) {
        throw new Exception("Severe XSSS detected");
    }
}

我编写测试时使用：抛出异常，SonarQube 抱怨通用异常，如果我编写自定义异常，例如：public class MyException extends Exception，SonarQube 仍然不喜欢它，任何帮助将不胜感激！

Answer 1

我正在解释您的代码：如果参数 url 包含此无效 URL，您似乎在某处有一个无效的 URL，并且您不想做您的业务。所以我猜IllegalArgumentException 比通用的Exception 精确得多。

来自IllegalArgumentException的JavaDoc：

抛出表明一个方法被传递了一个非法或不适当的参数。

此外，我认为 SonarQube 在这一点上会接受IllegalArgumentException 而不会抱怨。

Answer 2

现有答案提供了有用的信息，但如果您在原始帖子中指定的限制是正确的，那么这些答案并没有真正的帮助。

如果您实际上无法修改被测代码，则无法修复 SonarQube 问题。

Answer 3

MyException 扩展了Exception，因此您也必须更改方法声明 - throws 语句：

public String clean(String url) throws MyException {
   if (url.indexOf(invalidURL) {
      throw new MyException("Severe XSSS detected");
   }
}

更改是安全的，因为代码与之前的选项兼容：

try {
    object.clean("invalidUrl")
} catch (Exception e) {
    // MyException is handled by this catch block
}