Как уменьшить сложность if/else?

Question

У меня есть некоторые вложенные операторы if/else, но я хочу уменьшить их накладные расходы.

В этом примере я оцениваю, из какого выпадающего меню был нажат элемент li, и является ли этот элемент li первым (currentIndex === 0).

Код:

if (parentIndex === 1) {
  // But its own index is 0
  if (currentIndex === 0) {
    parentIndex = 2;
    updatedIndexPos = array[1];
    mainIndex =list_2.indexOf(updatedIndexPos);
    // If the previous line is -1:
    if (mainIndex === -1) {
      parentIndex = 1;
      updatedIndexPos = filterIndexPos;
      mainIndex = list_1.indexOf(updatedIndexPos);
    }
  } else {
    mainIndex = list_1.indexOf(mainIndexPos);
  }
} else if (parentIndex === 2) {
  // But its own index is 0
  if (currentIndex === 0) {
    parentIndex = 1;
    updatedIndexPos = array[0];
    mainIndex = list_1.indexOf(updatedIndexPos);
    // If the previous line is -1:
    if (mainIndex === -1) {
      parentIndex = 2;
      updatedIndexPos = filterIndexPos;
      mainIndex = list_2.indexOf(updatedIndexPos);
    }
  } else {
    mainIndex = list_2.indexOf(mainIndexPos);
  }
}

Глядя на это, много кода используется повторно, и eslint дает мне сложность 7 для него. Я попытался разбить его на более мелкие функции и передать аргументы, но все еще не смог решить сложность этого блока кода.

Answer 1

Попробуйте извлечь общие части, что-то вроде (не проверено):

if (parentIndex === 1) {
  potentialNewParent = 2;
  potentialUpdatedIndexPos = array[1];
  nullParentIndex = 1;
  mainList = list_1;
  otherList = list_2;
} else {
  // Do the same
}

if (currentIndex === 0) {
  parentIndex = potentialNewParent;
  updatedIndexPos = potentialUpdatedIndexPos;
  mainIndex = mainList.indexOf(updatedIndexPos);
  // If the previous line is -1:
  if (mainIndex === -1) {
    parentIndex = nullParentIndex;
    updatedIndexPos = filterIndexPos;
    mainIndex = otherList.indexOf(updatedIndexPos);
  }
} else {
  mainIndex = otherList.indexOf(mainIndexPos);
}

Answer 2

Часто полезно моделировать логику как конечный автомат. Итак, вы определили состояния и переходы между ними.

var action = 'foo';
var actionParams = {...};
var currentState = getState({parentIndex: parentIndex, ...});
if (currentState.canPerform(action)) {
  currentState.perform(action, actionParams);
}

Answer 3

Предполагая, что parentIndex всегда равен 1 или 2, его можно сильно упростить. Поскольку я понятия не имею, что он должен делать, я не проверял это, и имена переменных могут не подходить:

const mainList = parentIndex === 1 ? list_1 : list_2;
const secondaryList = parentIndex === 1 ? list_2 : list_1;

if (currentIndex === 0) {
  parentIndex = parentIndex === 1 ? 2 : 1;
  updateIndexPos = array[parentIndex - 1];
  mainIndex = secondaryList.indexOf(updatedIndexPos);
  if (mainIndex === -1) {
    parentIndex = parentIndex === 1 ? 2 : 1;
    updatedIndexPos = filterIndexPos;
    mainIndex = mainList.indexOf(updatedIndexPos);
  }
} else {
  mainIndex = mainList.indexOf(mainIndexPos);
}

Answer 4

Предполагая, что parentIndex может быть только 1 или 2, может ли это работать для вас?

var elements = [2, 1];
if ((parentIndex === 1 || parentIndex === 2) && currentIndex === 0) {

  oldParentIndex = parentIndex;
  parentIndex = elements[oldParentIndex-1];
  updatedIndexPos = array[oldParentIndex%2];
  mainIndex = this["list_"+parentIndex].indexOf(updatedIndexPos);

  if (mainIndex === -1) {
    parentIndex = oldParentIndex;
    updatedIndexPos = filterIndexPos;
    mainIndex = this["list_"+oldParentIndex].indexOf(updatedIndexPos);
  }
} else if ((parentIndex === 1 || parentIndex === 2) && currentIndex !== 0) {
  mainIndex = this["list_"+parentIndex].indexOf(updatedIndexPos);
}