Зачем использовать корпус переключателя в редукторе? - PullRequest
3 голосов
/ 22 апреля 2019

Интересно, каковы преимущества использования синтаксиса регистра переключателя в редукторе вместо, например, синтаксис сопоставления объектов? Я еще не встречал ни одного примера, в котором используется другое утверждение, кроме switch case, и я удивляюсь, почему для этого нет альтернатив. Пожалуйста, опишите ваши мысли о преимуществах и недостатках обоих способов (только если они оправданы).

const initialState = {
  visibilityFilter: 'SHOW_ALL',
  todos: []
};

// object mapping syntax
function reducer(state = initialState, action){
  const mapping = {
    SET_VISIBILITY_FILTER: (state, action) => Object.assign({}, state, {
      visibilityFilter: action.filter
    }),
    ADD_TODO: (state, action) => Object.assign({}, state, {
      todos: state.todos.concat({
        id: action.id,
        text: action.text,
        completed: false
      })
    }),
    TOGGLE_TODO: (state, action) => Object.assign({}, state, {
      todos: state.todos.map(todo => {
        if (todo.id !== action.id) {
          return todo
        }

        return Object.assign({}, todo, {
          completed: !todo.completed
        })
      })
    }),
    EDIT_TODO: (state, action) => Object.assign({}, state, {
      todos: state.todos.map(todo => {
        if (todo.id !== action.id) {
          return todo
        }

        return Object.assign({}, todo, {
          text: action.text
        })
      })
    })
  };
  return mapping[action.type] ? mapping[action.type](state, action) : state
}

// switch case syntax
function appReducer(state = initialState, action) {
  switch (action.type) {
    case 'SET_VISIBILITY_FILTER': {
      return Object.assign({}, state, {
        visibilityFilter: action.filter
      })
    }
    case 'ADD_TODO': {
      return Object.assign({}, state, {
        todos: state.todos.concat({
          id: action.id,
          text: action.text,
          completed: false
        })
      })
    }
    case 'TOGGLE_TODO': {
      return Object.assign({}, state, {
        todos: state.todos.map(todo => {
          if (todo.id !== action.id) {
            return todo
          }

          return Object.assign({}, todo, {
            completed: !todo.completed
          })
        })
      })
    }
    case 'EDIT_TODO': {
      return Object.assign({}, state, {
        todos: state.todos.map(todo => {
          if (todo.id !== action.id) {
            return todo
          }

          return Object.assign({}, todo, {
            text: action.text
          })
        })
      })
    }
    default:
      return state
  }
}

1 Ответ

2 голосов
/ 22 апреля 2019

Нет никаких преимуществ операторов switch в редукторах (о которых я знаю), кроме того, что они идиоматичны / стандартизированы и могут помочь другим людям понять ваш код.

Лично я перешел наПереключатель редукторов.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...