Java 获取:删除SonarQube中的此冗余跳转

Java 获取:删除SonarQube中的此冗余跳转,java,spring,sonarqube,return,void,Java,Spring,Sonarqube,Return,Void,我有一个方法,其中有一个if-else条件。if-else看起来与下面类似 public void agreementCoduct(String agreement) { if(agreement.equals("Pass")) { { //do someting return; // getting Remove this Redundant Jump } else if(aggrement.equals("NotPass

我有一个方法,其中有一个if-else条件。if-else看起来与下面类似

public void agreementCoduct(String agreement) {
       if(agreement.equals("Pass")) {
         { //do someting
           return; // getting Remove this Redundant Jump
         } else if(aggrement.equals("NotPass")) {
           // do something
           return; // getting Remove this Redundant Jump
        } else {
          // do something 
          return; // getting Remove this Redundant Jump
        }

}

我正在运行Sonarqube来检查代码质量。它显示消息“删除此冗余跳转”。我想知道如何才能消除代码的气味,并仍然实现我想要的。因为这是无效的方法,所以返回;应该工作正常吗?

这是多余的,因为在最后一次工作之后,你什么都没做

这是多余的,因为上一次之后你什么都没做

返回语句无效,因为在if-elseif-else块之后没有任何代码。如果删除return语句,代码将保留匹配的If块,并在方法末尾返回

仅执行计算结果为true的if-elseif-else块中的第一个


void方法最后有一个隐式的“return”语句。

返回语句无效,因为if-elseif-else块之后没有任何代码。如果删除return语句,代码将保留匹配的If块,并在方法末尾返回

仅执行计算结果为true的if-elseif-else块中的第一个


void方法最后有一个隐式的“return”语句。

这些
return
语句是多余的,因为您使用的是
if-else if-else
构造。这意味着这些块中的任何一个块中的代码都只会执行,而其他块永远不会执行,这就是您想要的

这些
return
语句是多余的,因为您正在使用
if-else-if-else
构造。这意味着这些块中的任何一个块中的代码都只会执行,而其他块永远不会执行,这就是您想要的

您使用的是
if-else
,这样您就可以安全地删除它们,因为一旦您进入一个区块,您就不会输入另一个单面注释
协议==“Pass”
可能无法工作,您使用的是
if-else
,这样您就可以安全地删除它们,因为一旦你进入一个区块,你就不会再输入另一个单面注释
协议==“Pass”
可能不起作用,