开发者问题收集

React Router 中的 Props 错误

2018-07-09
202

我正在学习使用 Redux、React 和 React Router。 我遇到了这样的问题:

当我使用 Link 从“/”重定向到“/details/{id}”之类的 URL 时 - 我发现调用了错误的动作创建者。来自“/”组件的动作创建器被调用,而不是“/details/{id}”中组件的动作创建器。

但是如果我刷新页面,一切都会好起来。调用正确的操作。

带链接的路由: <Link to={ /details/${this.props.movi​​e.id> }>

其他错误: 如果我从此页面按“返回”返回“/”,我将收到错误,即我的 props 数据未定义。 例如,props 为空,并且未从 componentDidMount() 调用 http 请求的动作创建器(?)。

但是如果我再次刷新页面,一切都会好起来。

路由有什么问题?或者redux?

ReactDOM.render(
    <Provider store={store}>
      <BrowserRouter>
        <Routes />
      </BrowserRouter>
    </Provider>, 
    document.getElementById('root'));


const Routes = () => {
    return <div>
        <Route exact path="/" component={Main} />
        <Route path="/details/:id" component={MovieDetailsContainer} />
    </div>;
}

"/":

    class MoviesDiscover extends Component {
    componentDidMount() {
        this.props.dicoverMovies();
    }

    render() {
      ...
    }
}

const mapStateToProps = (state) => {
    return {
        items: state.movieItems,
        hasErrored: state.movieHasErrored,
        isLoading: state.movieIsLoading,
    };
};

const mapDispatchToProps = (dispatch) => {
    return {
        dicoverMovies: (url) => dispatch(dicoverMovies(url))
    };
};

export default connect(mapStateToProps, mapDispatchToProps)(MoviesDiscover);

"details/{id}":

class MovieDetailsContainer extends Component {
    componentDidMount() {
        var id = this.props.match.params.id;
        this.props.getMovie(id); // called from MoviesDiscover instead of this, after routing
    }

    render() {
        var id = this.props.match.params.id;
        ...
    }
}

const mapStateToProps = (state) => {
    return {
        item: state.movieItems,
        hasErrored: state.movieHasErrored,
        isLoading: state.movieIsLoading,
    };
};

const mapDispatchToProps = (dispatch) => {
    return {
        getMovie: (url) => dispatch(getMovieDetails(url))
    };
};


export default connect(mapStateToProps, mapDispatchToProps)(MovieDetailsContainer);

操作(两个请求相同 - 针对结果、错误和加载的操作):

 ...
export function moviesDataSuccess(items) {
    return {
        type: MOVIES_RESULTS,
        isLoading: false,
        items: items
    };
}

       export function dicoverMovies() {
        return (dispatch) => {
            dispatch(moviesIsLoading(true));

            API.get(URL_MOVIE_DISCOVER)
              .then(response => {
                dispatch(moviesIsLoading(false));
                dispatch(moviesDataSuccess(response.data));
              })
              .catch(e => {
                console.log(e);
                dispatch(moviesHasErrored(true))
              });
        };
    }

    export function getMovieDetails(id) {
        return (dispatch) => {
            dispatch(moviesIsLoading(true));

            API.get(URL_MOVIE_DETAILS(id))
              .then(response => {
                dispatch(moviesIsLoading(false));
                dispatch(moviesDataSuccess(response.data));
              })
              .catch(e => {
                console.log(e);
                dispatch(moviesHasErrored(true))
              });
        };
    }

Reducers:

   export function movieHasErrored(state = false, action) {
    switch (action.type) {
        case MOVIES_ERROR:
            return action.hasErrored;
        default:
            return state;
    }
}

export function movieIsLoading(state = false, action) {
    switch (action.type) {
        case MOVIES_LOADING:
            return action.isLoading;
        default:
            return state;
    }
}

export function movieItems(state = {}, action) {
    switch (action.type) {
        case MOVIES_RESULTS:
            return action.items;
        default:
            return state;
    }
}

const rootReducer = combineReducers({
    movieHasErrored, 
    movieIsLoading, 
    movieItems
});

我很乐意接受所有建议。谢谢。

2个回答

React router 首先匹配关闭的 URL,我认为问题在于组件的顺序。 我建议您像这样对它们进行排序,它应该会得到解决:

const Routes = () => {
    return <div>
        <Route path="/details/:id" component={MovieDetailsContainer} />
        <Route exact path="/" component={Main} />
    </div>;
}

此外,React Router 有一个用于在 URL 之间切换的组件,如果您的 URL 不明确,您可以利用它:

 import { BrowserRouter as Router, Route, Link, Switch } from "react-router-dom";
 ...
 <Switch>
        <Route path="/about" component={About} />
        <Route path="/company" component={Company} />
        <Route path="/:user" component={User} />
 </Switch>

PS: 抱歉,如果这不是您想要的答案,我只是想发表评论而不是发布这个答案,我只是还没有直接评论的声誉:|

Amin Paks
2018-07-09

我的错误在其他地方。

问题是,我对列表结果和详细信息结果使用了相同的操作类型。

当我打开一个页面以获取详细信息结果时,我从上一个页面的列表结果中获得了道具(因为它们使用相同的操作类型)。之前我的详细信息组件会获取新数据。

并且我的代码在渲染时因数据错误而失败。

asf
2018-07-09