Рефакторинг 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

Вот одно решение, которое, вероятно, работает так же хорошо, как ваш кодне намного меньше и труднее читать

Answer 2

Одна из ваших возможностей - использовать троичный оператор в вашей функции для определения значения, которое вы хотите передать:

    var returnValue;

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

    callback(returnValue);

Answer 3

Вы все еще можете использовать условия, если не указано иное, но вам нужно объединить некоторые коды, такие как

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

и

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

, чтобы сделать ваш код короче.