Java 获取:删除SonarQube中的此冗余跳转
我有一个方法,其中有一个if-else条件。if-else看起来与下面类似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
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”
可能不起作用,