在React中的现有状态转换错误期间无法更新

Leo*_*ban 5 javascript jsx reactjs

在我的渲染return()里面我有这些:

<button onClick={ this.selectTimeframe('Today') }
        className={ this.setActiveIf('Today') } type="button">Today</button>
Run Code Online (Sandbox Code Playgroud)

这是什么功能:

selectTimeframe(timeframe) {
    // this.setState({ timeframe });
}
Run Code Online (Sandbox Code Playgroud)

^我现在必须注释掉setState否则我会得到上面发布的那个错误并且应用程序中断了.

我在构造函数中有这个:

this.selectTimeframe = this.selectTimeframe.bind(this);

我在这里找到了这个答案,但它没有意义,我想如何传递变量?或者他是说每个独特的按钮都需要一个独特的功能?为了避免在渲染中调用它?

完整代码

import React from 'react';

export class TimeframeFilter extends React.Component {

    constructor(props) {
        super(props);

        this.state = {
            timeframe: 'Today'
        };

        this.selectTimeframe = this.selectTimeframe.bind(this);
        this.setActiveIf = this.setActiveIf.bind(this);
    }

    componentDidMount() {
        console.log('%c TimeframeFilter componentDidMount', 'background: #393939; color: #bada55', this.state);
    }

    selectTimeframe(timeframe) {
        // this.setState({ timeframe });
    }

    setActiveIf(timeframe) {
        return this.state.timeframe === timeframe ? 'btn btn-default active' : 'btn btn-default';
    }

    render() {
        return (
            <div className="fl">
                <span className="label label-default mr10">Timeframe</span>
                <div className="btn-group btn-group-sm mr20" role="group">
                    <button onClick={ this.selectTimeframe('Today') }
                            className={ this.setActiveIf('Today') } type="button">Today</button>
                    <button onClick={ this.selectTimeframe('Week') }
                            className={ this.setActiveIf('Week') } type="button">Week</button>
                    <button onClick={ this.selectTimeframe('Month') }
                            className={ this.setActiveIf('Month') } type="button">Month</button>
                    <button onClick={ this.selectTimeframe('Year') }
                            className={ this.setActiveIf('Year') } type="button">Year</button>
                    <button onClick={ this.selectTimeframe('All') }
                            className={ this.setActiveIf('All') } type="button">All</button>
                </div>
            </div>
        );
    }
}

export default TimeframeFilter;
Run Code Online (Sandbox Code Playgroud)

Shu*_*tri 3

您应该使用该值将值传递给onClick函数arrow function or bind method,否则每次重新渲染发生时它都会执行返回值,onClick 如果您setState在其中使用,它将调用 arerender并再次调用该函数,从而重复该过程。

这样做就像

<button onClick={ ()=> this.selectTimeframe('Today') }
        className={ this.setActiveIf('Today') } type="button">Today</button>
Run Code Online (Sandbox Code Playgroud)

或者

<button onClick={ this.selectTimeframe.bind(this, 'Today') }
        className={ this.setActiveIf('Today') } type="button">Today</button>
Run Code Online (Sandbox Code Playgroud)

希望我能够向您解释清楚。:)

编辑:虽然上述方法解决了问题,但由于性能原因并不是最佳方法。我们应该避免在渲染内部绑定方法,因为在重新渲染期间它将创建新方法而不是使用旧方法,这会影响性能。

请参阅有关如何避免在渲染方法内绑定的答案