Конвертировать в 2 функции в одну функцию - PullRequest
Новая
       5

Конвертировать в 2 функции в одну функцию

0 голосов
/ 26 марта 2020

Ниже приведены 2 функции для выбора параметров в раскрывающемся списке. Я хочу сделать это с одной функцией, а не с двумя разными функциями. Есть несколько вариантов, различающихся в зависимости от fromtype / totype . Но больше всего похоже на case A, I, W et c, общие для обеих функций. Как превратить следующие две функции в одну функцию.

Я думал о функции 

const getOptionKey = (type, metaType) => {
  switch (type) {
    case 'A':
      type = ['A'];
      break;
    case 'W1':
    case 'W':
      type = ['W'];
      break;
    case 'I':
      type = ['I'];
      break;
    case 'E':
      switch (metaType) {
        case 'A':
          type = ['A'];
          break;
        case 'W2':
          fromType = ['W'];
          break;
          default:
          fromType = [
            'A',
            'W',
            'C',
            'I',
            'CK'
          ];
      }
      break;
    default:
      type = [];
  }
}

, но тогда я не могу найти лучшее решение, как классифицировать коммутатор случаи, которые не являются общими. 

const getFromTypeOptionKey = (fromType, fromMetaType) => {
  switch (fromType) {
    case 'A':
      fromType = ['A'];
      break;
    case 'I':
      fromType = ['I'];
      break;
    case 'W1':
    case 'W':
      fromType = ['W'];
      break;
    case 'R':
      fromType = ['R'];
      break;
    case 'E':
      switch (fromMetaType) {
        case 'A1':
          fromType = ['A'];
          break;
        case 'W2':
          fromType = ['W'];
          break;
        case 'I1':
        case 'L':
        case 'CH':
        case 'C1':
        case 'AT':
          fromType = ['C'];
          break;
        default:
          fromType = [
            'A',
            'W',
            'C',
            'I',
            'CK'
          ];
      }
      break;
    default:
      fromType = [];
  }
  return fromType;
};

const getToTypeOptionKey = (toType, toMetaType) => {
  switch (toType) {
    case 'A':
      toType = ['A'];
      break;
    case 'CK':
      toType = ['CK'];
      break;
    case 'I':
      toType = ['I'];
      break;
    case 'W1':
    case 'W':
      toType = ['W'];
      break;
    case 'E':
      switch (toMetaType) {
        case 'A1':
          toType = ['A'];
          break;
        case 'W2':
          toType = ['W'];
          break;
        case 'I1':
        case 'CW':
        case 'C1':
        case 'PD':
        case 'PU':
        case 'AT':
          toType = ['C'];
          break;
        default:
          toType = [
            'A',
            'W',
            'C',
            'I',
            'CK'
          ];
      }
      break;
    default:
      toType = [];
  }
  return toType;
};

1 Ответ

2 голосов
/ 26 марта 2020

switch случай здесь делает его немного громоздким. Как насчет использования объекта и бита оператора if, например: 

// Easier to read here since made a tree
let mapping = {
  A: ['A'],
  I: ['I'],
  W: ['W'],
  W1: ['W'],
  R: ['R'],
  default: [],
  E: {
    A1: ['A'],
    W2: ['W'],
    I1: ['C'],
    L: ['C'],
    CH: ['C'],
    C1: ['C'],
    AT: ['C'],
    default: ['A', 'W', 'C', 'I', 'CK'],
  }
}

function get(fromType, fromMetaType) {
  return (fromType === 'E')
    ? mapping['E'][fromMetaType] || mapping['E'].default
    : mapping[fromType] || mapping.default;
}

console.log(get('A')); // [ 'A' ]
console.log(get('INVALID')); // []
console.log(get('E', 'INVALID')); // [ 'A', 'W', 'C', 'I', 'CK' ]
console.log(get('E', 'C1')); // [ 'C' ]
console.log(get('R', 'C1')); // [ 'R' ] note that the `C1` is discarded
...