Комментарии / Профиль comerc / Хабр

Про многословность я наверно преувеличил. Ниже пример практически построчного переноса сайд-эффекта из под redux-thunk на redux-saga. Посмотрите, пожалуйста, что тут можно улучшить?

redux-thunk

const save = () => (dispatch, getState) => {
  const clearPost = ({ searchHub, errors, isSubmitting, ...result }) => result
  const state = getState()
  const errors = {}
  Object.keys(validators).forEach(key => {
    const validate = validators[key]
    const error = validate(state.postForm[key], state)
    if (error) {
      errors[key] = error
    }
  })
  if (!isEmpty(errors)) {
    dispatch(setErrors(errors))
    dispatch(appActions.setMainError('Исправьте ошибки в форме'))
    return
  }
  if (state.app.mainError) {
    dispatch(appActions.setMainError())
  }
  dispatch(setSubmitting(true))
  axios
    .post('/post/', clearPost(state.postForm))
    .then(response => {
      const post = response.data
      dispatch(postsActions.setPost(post))
      dispatch(push(`/post/${post.id}/`))
    })
    .catch(error => {
      dispatch(appActions.setMainError(error.toString()))
    })
    .then(() => {
      dispatch(setSubmitting(false))
    })
}

redux-saga

function* saveSaga(action) {
  const clearPost = ({ searchHub, errors, isSubmitting, ...result }) => result
  const state = yield select()
  const errors = {}
  Object.keys(validators).forEach(key => {
    const validate = validators[key]
    const error = validate(state.postForm[key], state)
    if (error) {
      errors[key] = error
    }
  })
  if (!isEmpty(errors)) {
    yield put(setErrors(errors))
    yield put(appActions.setMainError('Исправьте ошибки в форме'))
    return
  }
  if (state.app.mainError) {
    yield put(appActions.setMainError())
  }
  yield put(setSubmitting(true))
  try {
    const response = yield call(axios.post, '/post/', clearPost(state.postForm))
    const post = response.data
    yield put(postsActions.setPost(post))
    yield put(push(`/post/${post.id}/`))
  } catch (error) {
    yield put(appActions.setMainError(error.toString()))
  } finally {
    yield put(setSubmitting(false))
  }
}

В поисках дофамина в разработке или избавляем себя от рутины

comerc 22 мая 2017 в 19:51

Удобство разработки — странная штука. Развесистый IDE позволяет чрезмерно усложнять структуру проекта. Как я плевался от WebStorm после Atom. И как я ныне счастлив на VSCode после WebStorm.

-1

Почему программисты не могут программировать

comerc 20 мая 2017 в 15:29

Зато на собеседованиях любят эти глупости :)

Что взять за основу React приложения

comerc 19 мая 2017 в 19:57

По генераторам штудирую: раз, два.

Кстати, зачем в коде *, я такого раньше не видел:

var ids = {
  *[Symbol.iterator]: function () {
    var index = 0;

    return {
      next: function () {
        return { value: 'id-' + index++, done: false };
      }
    };
  }
};

По сагам что нашёл интересного: раз, два, три, четыре, пять.

Посмотрел на официальный пример и загрустил — как-то слишком многословно. Может есть какие-то улучшайзеры?

Почему программисты не могут программировать

comerc 19 мая 2017 в 05:58

Для меня необ'яснимая загадка, чем не нравится coffescript подавляющему большинству

Почему программисты не могут программировать

comerc 18 мая 2017 в 11:16

Добавил вариант с рекурсией — намного хуже, чем перебор.

1 2 ...

17 18

20 21 ...

43 44