Ошибка цикломатической сложности в моем коде

Question

Я получаю 2 ошибки в моем файле grunt

line 61 col 25 This function's cyclomatic complexity is too high. (10)

line 101 col 22 This function's cyclomatic complexity is too high. (10)

Как я мог уменьшить цикломатическую сложность в этом случае? мои функции не так сложны

первая ошибка

    remove: function(line, row, type) {
      var spreadSelected = (row.spreadSelected && type === 'spread'),
        totalSelected = (row.totalSelected && type === 'total'),
        moneyLineSelected = (row.moneyLineSelected && type === 'moneyline'),
        lineValue;
      if (spreadSelected || totalSelected || moneyLineSelected) {
        switch (type) {
          case 'spread':
            lineValue = row.spread.line;
            break;
          case 'total':
            lineValue = row.total.line;
            break;
          case 'moneyline':
            lineValue = row.moneyLineId;
            break;
          default:
            break;
        }

        AuthFactory.getCustomer().then(function(credentials) {
          betSlipSelectionRequest('/betSlip/removeSelection', {
            customerId: credentials.customer,
            game: row.game,
            pair: row.pair,
            line: lineValue
          });
        });

        if (spreadSelected) {
          row.spreadSelected = false;
        }
        if (totalSelected) {
          row.totalSelected = false;
        }
        if (moneyLineSelected) {
          row.moneyLineSelected = false;
        }
      }
    }...

а затем вторая функция ошибки

    add: function(line, row, type) {
      var spreadSelected = (row.spreadSelected && type === 'spread'),
        totalSelected = (row.totalSelected && type === 'total'),
        moneyLineSelected = (row.moneyLineSelected && type === 'moneyline'),
        lineValue;
      if (!(spreadSelected || totalSelected || moneyLineSelected)) {
        switch (type) {
          case 'spread':
            lineValue = row.spread.line;
            break;
          case 'total':
            lineValue = row.total.line;
            break;
          case 'moneyline':
            lineValue = row.moneyLineId;
            break;
          default:
            break;
        }
        AuthFactory.getCustomer().then(function(credentials) {
          betSlipSelectionRequest('/betSlip/addSelection', {
            customerId: credentials.customer,
            game: row.game,
            pair: row.pair,
            line: lineValue
          });
        });
        switch (type) {
          case 'spread':
            row.spreadSelected = true;
            break;
          case 'total':
            row.totalSelected = true;
            break;
          case 'moneyline':
            row.moneyLineSelected = true;
            break;
        }
      }
    }

странная вещь здесь: эта ошибка только у меня, мой сотрудник открывает те же файлы и запускает grunt, и в их терминалах нет ошибок.

Answer 1

Способ уменьшить цикломатическую сложность функции состоит в том, чтобы разделить ее на несколько более мелких функций и распределить эту сложность на простые для понимания куски. Например, вы можете извлечь свой оператор switch-case, получив в результате что-то вроде этого:

remove: function(line, row, type) {
    var spreadSelected = (row.spreadSelected && type === 'spread'),
        totalSelected = (row.totalSelected && type === 'total'),
        moneyLineSelected = (row.moneyLineSelected && type === 'moneyline'),
        lineValue;
    if (!(spreadSelected || totalSelected || moneyLineSelected)) {
        lineValue = getLineValue(row, type);
    }
    // ... and so on, in reasonable chunks.
}

function getLineValue(row, type) {
    var lineValue;
    switch (type) {
        case 'spread':
            lineValue = row.spread.line;
            break;
        case 'total':
            lineValue = row.total.line;
            break;
        case 'moneyline':
            lineValue = row.moneyLineId;
            break;
        default:
            break;
    return lineValue;
}

Затем мы обнаружим, что вы можете использовать getLineValue функция в вашем втором блоке:

add: function(line, row, type) {
    var spreadSelected = (row.spreadSelected && type === 'spread'),
        totalSelected = (row.totalSelected && type === 'total'),
        moneyLineSelected = (row.moneyLineSelected && type === 'moneyline'),
        lineValue;

    if (!(spreadSelected || totalSelected || moneyLineSelected)) {
        lineValue = getLineValue(row, type);
    }
    // ... and so on
}

Таким образом, с этим одним изменением вы распределили сложность для другой функции, а также полностью устранили некоторую сложность, исключив дублирование.