如何在reactjs

时间:2017-05-16 10:15:37

标签: javascript ajax reactjs

我目前正在使用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;

2 个答案:

答案 0 :(得分:2)

if(prevState.data !== this.state.data) 并不比较实际对象的相等性。这就是我假设$.ajax请求在componentDidUpdate中不断被触发的原因,即使实际数据是相同的。

如果您真的想要采用昂贵的方式,可以使用lodash_.isEqual)来比较对象(或jQuery中的内置方法)。最终,出于性能原因,它是比较特定数据的更好方法。

答案 1 :(得分:0)

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

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

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

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