Насколько правильным считается брать store.getState() в создателях действий?

Question

[Андрей]

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

export function changePassWithCode(body = {}) {
    body.email = store.getState().authorizationReducer.emailForForgotPass;
    body.validationCode = store.getState().authorizationReducer.ForgotPassValidationCode;
    const
        url     = `${config.apiUrl}/user/myFuckingAPIOnPHP`,
        isRSAA  = true;

    return {type: POST_AUTHORIZATION_CHANGE_FORGOT_PASSWORD, url, isRSAA, body};
}

Answer 1

[eddy-lazar]

В целом такой подход не является анти-патерном, зависит от ситуации. Если в проекте используется redux-thunk, тогда удобней это делать напрямую из функции, вторым параметром thunk передает как раз метод getState

export const SOME_ACTION = 'SOME_ACTION';
export function someAction() {
  return (dispatch, getState) => {
    const {items} = getState().otherReducer;
    dispatch(anotherAction(items));
  }
}

И не нужно каждый раз глобально импортировать store.

https://habrahabr.ru/post/330692/ - хорошая статья на тему что хорошо в redux что плохо и так далее.

Comments

[Максим]

Андрей, eddy-lazar, Поддерваживаю , раз уж такую функцию вложили специально для удобства, значит есть необходимость. Лично я сам не раз к этому прибегал.

[Андрей]

eddy-lazar Максим ,а насколько корректно написан этот action, нормально ли то что он может вернуть два разных? один это загружать файл, второй добавить к загрузке

export function uploadDocument( files, isInitial, isReuploading = false) {
    cons tauthorizationReducer = store.getState().authorizationReducer;

    if (isInitial && authorizationReducer.uploadDocumentRequest) {
        return { type: ADD_ADDITIONAL_DOCUMENTS_FOR_UPLOAD, files }
    }

    return {
        type: UPLOAD_DOCUMENT, name, files, isInitial, isReuploading
    };
}