两层if嵌套该怎么优化

Question

pushData(flag) {
      const { $refs } = this;
      if (flag === REMAINING_SHAREHOLDERS) {
        if ($refs[flag].getData().length <= DYNAMIC_NUM) {
          $refs[flag].pushData({});
        }
      } else {
        $refs[flag].pushData({});
      }
    },

请问以上代码该怎么优化,主要针对内部的两层if嵌套

Answer 1

可以考虑使用更简洁的逻辑条件表达式（ternary operator）代替嵌套的if语句。

例如，可以将原来的代码改写成：

pushData(flag) {
  const { $refs } = this;
  const dataLength = $refs[flag].getData().length;
  if (flag === REMAINING_SHAREHOLDERS && dataLength <= DYNAMIC_NUM) {
    $refs[flag].pushData({});
  } else if (flag !== REMAINING_SHAREHOLDERS) {
    $refs[flag].pushData({});
  }
}

这样可以将嵌套的if语句简化为两个逻辑条件表达式，使代码更加简洁易懂。