java - SonarQube，finally block 中的跳转语句(squid :S1143)

Question

我知道，不应该使用 finally block 中的跳转语句。在这个简单的示例中，“break”用于断开“switch”。 SonarQube (5.6.3) 与sonar-java 4.5.0.8398 报告一个问题:

“跳转语句不应出现在“finally” block 中(squid:S1143)”

public static void breakInFinallyIssue(){
    int a = 0;
    try{
        a = 1 / 0;
    }catch(Exception x){
        System.out.println("div by zero");
    }
    finally{
        switch (a) {
        case 0:
            //do something
            break;
        default:
            break;
        }
        //do something more
    }
}

这是已知的 FP/错误吗？

Answer 1

你是对的，这是一个误报。然而，这种复杂的逻辑不属于 finally block ，如果可能，应该提取到恰当命名的清理方法。这不仅会关闭警告，还会提高代码的可读性。