如何在reactjs中打破componentDidUpdate中的无限循环

Question

我目前正在使用reactjs制作仪表板，其中有4个选项卡或按钮图表正在正确创建，但唯一的问题是，当我们点击不同的仪表板并且它在相同的面板中具有相同的图表而不是图表未更新时我已经使用了componentDidMount（）和componentDidUpdate虽然它提供了适当的输出，但它创建了一个无限循环我也应用了一些条件，但它仍然在无限循环任何人都可以帮助我？

这是我的代码

class BarChartComponent extends React.Component 
{

   constructor(props) 
   {
      super(props);

      this.state = {
         data: null
      }
   };

   componentDidMount()
    {
            var that = this;
            $.ajax({
                    method: "GET",
                    url: "http://10.0.3.8:8050/querydata?query_id="+that.props.id,
                    success: function(data) 
                    {

                        console.log("component did mount method called");
                        console.log(1,JSON.stringify(data));
                        var BarChartData  = [
                        {
                            key: "totals",
                            values: []
                        }
                        ];
                        data.forEach(function (d)
                        {
                            d.value= +d.value
                            BarChartData[0].values.push(d)
                        }) 
                        console.log(2,JSON.stringify(BarChartData))
                        that.setState({data:BarChartData});
                    },
                    error: function(XMLHttpRequest, textStatus, errorThrown)
                    { 
                        console.error("Status: " + textStatus);
                        console.error("Error: " + errorThrown); 
                    }       
                });         
    }

    componentDidUpdate(prevProps, prevState)
    {
                if(prevState.data !== this.state.data )
                {
                    var that = this;
                    $.ajax({
                        method: "GET",
                        url: "http://10.0.3.8:8050/querydata?query_id="+that.props.id,
                        success: function(data) 
                        {

                            console.log("component did update method called");
                            var BarChartData  = [
                            {
                                key: "totals",
                                values: []
                            }
                            ];
                            data.forEach(function (d)
                            {
                                d.value= +d.value
                                BarChartData[0].values.push(d)
                            }) 

                            if( prevProps.id !== that.state.id )
                            {
                                that.setState({data:BarChartData});
                                console.log(3,JSON.stringify(BarChartData))
                                console.log(4,JSON.stringify(that.state.data))
                            }                   
                        },
                        error: function(XMLHttpRequest, textStatus, errorThrown)
                        { 
                            console.error("Status: " + textStatus);
                            console.error("Error: " + errorThrown); 
                        }       
                    });         

                }
    }



    render()
    {

        if (this.state.data==null)
        {
            return (<h1>Loading...</h1>);
        }
        return (


             <NVD3Chart type="discreteBarChart" datum={this.state.data} x="name" y="value"/>

        );
    }
}

window.BarChartComponent = BarChartComponent;

Answer 1

if(prevState.data !== this.state.data)没有比较实际对象的平等性。这就是为什么我假设你的$.ajax请求在componentDidUpdate中不断被触发，即使实际数据是相同的。

如果你真的想以昂贵的方式使用lodash（_.isEqual）来比较对象（或jQuery中的内置方法）。最终，出于性能原因，这是比较特定数据的更好方法。

Answer 2

我认为最好在componentDidUpdate的顶部使用此检查：

if (prevProps.id !== this.props.id)

因为您的请求仅取决于此参数。

然后你将没有无限循环，因为id具有标量值并且相等检查将起作用。