If-else重构

Question

我正在尝试重构if else语句的这一块。这段代码的最佳优化版本是什么？

`const handlePhoneNumberFormat = (rules, value, callback) => {
    if (value && value.match(phoneNumberRegex)) {
      if (value.includes('-')) {
        if (value.indexOf('-') === 2) {
          if (value.length !== 10) {
            callback('Please enter 7 digits after "-".');
          } else {
            callback();
          }
        } else if (value.indexOf('-') === 3) {
          if (value.length !== 14) {
            callback('Please enter 10 digits after "-".');
          } else {
            callback();
          }
        } else {
          callback();
        }
      } else {
        callback();
      }
    } else {
      callback('');
    }
  };`

Answer 1

这是一种可能与您的代码一样有效的解决方案

const handlePhoneNumberFormat = (rules, value, callback) => {
    if (value && value.match(phoneNumberRegex)) {
        const pos = value.indexOf('-');
        const length = value.length;
        if (pos === 2 && length !== 10) {
            callback('Please enter 7 digits after "-".');
        } else if (pos === 3 && length !== 14) {
            callback('Please enter 10 digits after "-".');
        } else {
            callback():
        }
    } else {
      callback('');
    }
};

没有重复，每个回调都是唯一的...尽管如果您需要较小的代码

const handlePhoneNumberFormat = (rules, value, callback) => {
    const t = { "2": 10, "3": 14 };
    if (value && value.match(phoneNumberRegex)) {
        const pos = value.indexOf('-');
        const a = t[pos] || 0;
        if (a && a !== value.length) {
            callback(`Please enter ${a - pos - 1} digits after "-".')`);
        } else {
            callback():
        }
    } else {
      callback('');
    }
};

尽管很小，但也很难阅读

Answer 2

如果其他条件仍然可以使用，但是必须结合一些代码，例如

 value.indexOf('-') and value.includes('-') value && value.match(phoneNumberRegex)

和

value.indexOf('-') === 3 and value.length !== 14

使代码更短。

Answer 3

您拥有的一个选择是在函数中使用三元运算符来定义要传递的值：

    var returnValue;

    returnValue = (value && value.match(phoneNumberRegex) && value.includes('-')  && value.indexOf('-') === 2) ? 'Please enter 7 digits after "-".' : returnValue ;

    callback(returnValue);