如何在 React 的卸载组件中设置状态

Question

我们如何 setState in componentDidMount 挂钩在一个参数中？

我的问题是 React notify 的警告：

Warning: Can't call setState (or forceUpdate) on an unmounted 
component. This is a no-op, but it indicates a memory leak in your 
application. To fix, cancel all subscriptions and asynchronous tasks in the 
componentWillUnmount method.
in DiscoverPage (created by Route)

在我的DiscoverPage 组件中：

import React, { Component } from 'react';
import { Switch, Route, Redirect } from 'react-router-dom';
import TvShows from './TvShows';
import Movies from './Movies';
import MovieDetail from "../MoviePage/MovieDetail";
import TvDetail from "../TvShowPage/TvDetail";

class DiscoverPage extends Component {

  constructor(props) {
    super(props);
    this.state = {
      data: {}
    }
  }

  getDataItem = (data) => {
    this.setState({ data: data });
  };

  render() {
    return (
      <Switch>
        <Route exact path="/discover/movie" render={props => <Movies data={this.getDataItem} {...props} />} />
        <Route path="/discover/tv" render={props => <TvShows data={this.getDataItem} {...props} />} />
        <Route path="/movie/:movie" render={props => <MovieDetail data={this.state.data} {...props} />} />
        <Route path="/tv/:tv" render={props => <TvDetail data={this.state.data} {...props} />} />
        <Redirect to="/discover/movie" />
      </Switch>
    );
  }
}

export default DiscoverPage;

在子组件中（本例为Movies组件）：

import React, { Component } from 'react';
import requestApi from '../api';
import MovieList from "../MoviePage/MovieList";

class Movies extends Component {

  constructor(props) {
    super(props);
    this.state = {
      data: {
        page: 1,
        results: []
      }
    }
  }

  componentDidMount() {
    requestApi.fetchData('discover', 'movie').then(response => {
      this.setState({ data: response.data, isLoading: false });
    });
  }

  getMoviebyId = (id) => {
    requestApi.fetchDataById('movie', id).then(response => {
      this.props.data(response.data);
    });
  };

  nextPage = (e) => {
    const page = this.state.data.page + 1;
    requestApi.fetchDataPaginate('discover', 'movie', page).then(response => {
      this.setState({ data: response.data });
    });
  };
  prevPaginate = (e) => {
    const page = this.state.data.page - 1;
    requestApi.fetchDataPaginate('discover', 'movie', page).then(response => {
      this.setState({ data: response.data });
    });
  };

  render() {
    return (
      <div className="container">
        <div className="ss_media">
          <h2 className="title">Discover New Movies & TV Shows</h2>
          <MovieList
            movie={this.getMoviebyId}
            routeProps={this.props}
            prevPaginate={this.prevPaginate}
            nextPaginate={this.nextPage}
            moviesList={this.state.data} />
        </div>
      </div>
    );
  }
}

export default Movies;

Answer 1

我认为您缺少从 react-router-dom 导入 BrowserRouter

import { Switch, Route, Redirect, BrowserRouter } from 'react-router-dom'; // Add BrowserRouter

class DiscoverPage extends Component {

...
render() {
    return (
    <BrowserRouter> // Add this 
        <Switch>
            <Route exact path="/discover/movie" render={props => <Movies data={this.getDataItem} {...props} />} />
            <Route path="/discover/tv" render={props => <TvShows data={this.getDataItem} {...props} />} />
            <Route path="/movie/:movie" render={props => <MovieDetail data={this.state.data} {...props} />} />
            <Route path="/tv/:tv" render={props => <TvDetail data={this.state.data} {...props} />} />
            <Redirect to="/discover/movie" />
        </Switch>
    </BrowserRouter>
    );
}
}

Answer 2

我们可以在 componentDidMount 中设置状态，例如：

componentDidMount(){
    var {state, setState} = this.props;
    setState({doNotRender: true});
}

但改为使用 componentDidUpdate，例如：

componentDidUpdate(){
    var {state, setState} = this.props;
    if(!state.doNotRender){ //any_condition ur choice
       setState({doNotRender: true});
    }
}

Answer 3

您的 DiscoverPage 组件在路由更改时卸载，之后您调用 getDataItem 方法。这就是 react 抛出错误的原因。

我建议使用 react-router 组件方法而不是渲染并将数据移动到存储区，然后任何组件都可以访问该存储区。参考react-router render methods

例子：

<Route path="/user/:username" component={User}/>

Answer 4

需要做一些更改，您使用的逻辑效率不高
就我从你的问题中了解到的那样，这会很好

import React, { Component } from 'react';
import { Switch, Route, Redirect } from 'react-router-dom';
import TvShows from './TvShows';
import Movies from './Movies';
import MovieDetail from '../MoviePage/MovieDetail';
import TvDetail from '../TvShowPage/TvDetail';

class DiscoverPage extends Component {
    constructor(props) {
        super(props);
        this.state = {
            data: {}
        };
    }
    getALLDATA() {
        //update your state after your request
        request().then(res => {
            this.setState({
                data: res.data
            });
        });
    }
    componentDidMount() {
        this.getALLDATA();
    }
    getDataItem = () => {
        return this.state.data;
    };

    render() {
        return (
            <Switch>
                <Route exact path="/discover/movie" render={props => <Movies data={this.getDataItem} {...props} />} />
                <Route path="/discover/tv" render={props => <TvShows data={this.getDataItem} {...props} />} />
                <Route path="/movie/:movie" render={props => <MovieDetail data={this.state.data} {...props} />} />
                <Route path="/tv/:tv" render={props => <TvDetail data={this.state.data} {...props} />} />
                <Redirect to="/discover/movie" />
            </Switch>
        );
    }
}

export default DiscoverPage;

Answer 5

您收到的警告是告诉您不能在未安装的组件（即当前未渲染的组件）上调用 setState。最有可能的是，您的其他组件之一（Movies、TvShows 等）正在调用您作为 data 属性传递的方法。当这些组件之一被渲染时，DiscoveryPage 组件不会被渲染，因此通过 data 属性调用 getDataItem 将导致在未渲染时调用 DiscoveryPage 的 setState 方法。为了解决这个问题（假设你想要做的是在组件之间传递数据）我建议也许使用localStorage 来存储一个共享变量（或者，正如@Anu 所建议的，一些其他类型的存储）。

我不明白您所说的“在参数中”是什么意思，但是，回答您的问题，您可以在任何 component... 方法中调用 setState（即使在 componentWillUnmount 中调用它几乎没有意义，因为这个方法只在组件即将被卸载时调用）。

编辑： 查看Movie 组件。使用localStorage 作为我们的存储，在getMovieById 方法中，而不是：

getMoviebyId = (id) => {
    requestApi.fetchDataById('movie', id).then(response => {
        //this next sentence will call DiscoveryPage's setState
        //DiscoveryPage is not mounted, so a warning will be raised
        this.props.data(response.data);
    });
};

你可以有类似的东西：

getMoviebyId = (id) => {
    requestApi.fetchDataById('movie', id).then(response => {
        //any component can access this store
        localStorage.setItem("data", response.data);
    });
};

然后，在其他组件或方法中，像这样访问它：

localStorage.getItem("data");

您的代码有点混乱。我在理解您在哪里使用DiscoveryPage 的data 时遇到了一点麻烦，这是您在调用data 方法时设置的那个。但无论如何，希望这会有所帮助。

再补充一点：使用localStorage 时，管理它是一种很好的做法，我的意思是在您使用完data 值后调用localStorage.removeItem("data")。最好这样做，例如，在一些 componentWillUnmount 方法中。