React / Redux-将this.props.history传递给thunk是反模式吗?

时间:2018-10-25 03:40:02

标签: reactjs react-router-dom

我在一个简单的应用程序中有一个表单。提交表单时,将引发重击。

handleSubmit(e) {
    e.preventDefault(); 
    this.props.dispatch(loginUser({
        username: this.state.username, 
        password: this.state.password, 
        history: this.props.history
    }));
}

如您所见,我将react-router-dom的历史传递给了thunk。这是重击:

export const loginUser = input => dispatch => {
    return fetch('xxxxxxxxxx', {
        method: 'POST', 
        body: JSON.stringify({
            username: input.username, 
            password: input.password
        }), 
        headers: {
            'Content-Type': 'application/json'
        }
    })
    .then(res => {
        if(!res.ok) {
            return Promise.reject(res.statusText)
        }
        return res.json(); 
    })
    .then(token => {
        localStorage.setItem('token', token.authToken); 
        input.history.push('/'); 
    })
    .catch(err => console.error(`error: ${err}`)); 
}

如您所见,如果获取成功,那么我将推入历史记录。

这是反模式吗?还是可以吗它有效*,但我想知道它是否超级奇怪/不推荐

1 个答案:

答案 0 :(得分:0)

根据此页面:https://reacttraining.com/react-router/web/guides/redux-integration您正在做正确的事情,包括在thunk输入中的历史记录。

报价: “这里的解决方案只是将历史对象(提供给所有路由组件)包括在操作的有效负载中,并且您的异步处理程序可以在适当的时候使用它来进行导航。”