如何通过单一方法调用改写代码以解决Sonar合规问题？

我完全理解你的困扰——Sonar有时候会对这种「条件分支后无条件执行同函数调用」的代码误判为冗余，但你的逻辑是明确的：当bool1为false时先执行func2(obj1, true)，之后无论如何都要执行func2(obj1, false)，而且你不想重复写func2的调用代码。

先回顾一下你的原始代码：

func1(obj1, bool1){
    if(!bool1){
        func2(obj1,true);
    }
    func2(obj1,false);  // Note this is called UNCONDITIONALLY
}

Sonar触发合规性问题的原因，大概率是它检测到你重复调用了func2，且仅第二个参数不同，误以为这是冗余代码或者逻辑错误（比如是不是应该用else分支？但你的需求显然不是）。下面提供两种不重复func2核心代码的优化方案：

方案1：用参数列表批量调用 #

把需要传递的flag参数收集到数组中，再通过循环调用func2，这样既保留了原有逻辑顺序，又避免了重复写完整的func2调用语句：

func1(obj1, bool1){
  const flags = [];
  // 根据条件添加需要调用的flag
  if(!bool1){
    flags.push(true);
  }
  // 无条件添加false的flag
  flags.push(false);
  // 批量调用func2
  flags.forEach(flag => func2(obj1, flag));
}

方案2：封装调用逻辑为辅助函数 #

把func2的调用逻辑封装成一个内部小函数，这样即使多次调用，也只是重复调用辅助函数，而非重复写func2的完整参数和调用语句，Sonar通常会认可这种写法：

func1(obj1, bool1){
  // 封装func2的调用，复用obj1参数
  const callFunc2 = (flag) => func2(obj1, flag);
  
  if(!bool1){
    callFunc2(true);
  }
  callFunc2(false);
}

这两种方案都完全保留了你原本的业务逻辑：当bool1为false时先执行func2(obj1, true)，之后必定执行func2(obj1, false)，同时解决了Sonar的合规性问题，也没有重复func2的代码。

内容的提问来源于stack exchange，提问作者gaurav b