2

我正在尝试通过实现一个简单的待办事项应用程序来学习反应和函数式编程。我没有使用通量，因为我只是想了解在父母和孩子之间传递信息的概念。我正在尝试在子项中的单击事件上触发父项中的函数。但是，我从 React 中得到了一个关于使用纯函数和状态的非常讨厌的错误。有人可以解释我做错了什么以及正确的做事方式吗？我的功能有什么不纯洁之处，我看不到我正在创造的副作用。这是我的代码：

var React = require('react');
var Todo = require('./Todo');

const todos = [ {task: "Eat", completed: false},
            {task: "Breathe", completed: false},
            {task: "Sleep", completed: false}];

var TodoList = React.createClass({
    getInitialState: function() {
        return {todos: todos};
    },
    changeTodoStatus (task) {
        var updatedTodos = this.state.todos.map(function(todo){
            if (task.task === todo.task) {
                return {task: todo.task, completed: !todo.completed};
            } else {
                return todo;
            }
        });
        this.setState({todos: updatedTodos});
    },
    render: function() {
        var _that = this;
        return(
            <div className="container">
                <div className="row list-of-things">
                    <ul className="list-group">
                        {
                          this.state.todos.map( (todo, index) => {
                                return (<Todo clickHandler={ this.changeTodoStatus } key={index} todo={todo} />);
                            })
                        }
                    </ul>
                </div>
            </div>
        );
    }
});

module.exports = TodoList;

var Todo = React.createClass({
    handleClick( todo ){
        this.props.clickHandler( todo );
    },
    render: function() {
        if( this.props.todo.completed === true){
            return ( <li onClick={ this.handleClick(this.props.todo.task) } className="list-group-item list-group-item-success"><strike>{this.props.todo.task}</strike></li> );
        } else {
            return ( <li onClick={ this.handleClick(this.props.todo.task) } className="list-group-item"> {this.props.todo.task} </li> );
        }
    }
});

module.exports = Todo;

非常感谢任何帮助/澄清！

这是错误：bundle.js:9139 警告：setState(...)：在现有状态转换期间无法更新（例如在render或其他组件的构造函数内）。渲染方法应该是 props 和 state 的纯函数；构造函数副作用是一种反模式，但可以移至componentWillMount.

4

2 回答 2

2

在 todo 的 onclick 处理程序中，您实际上是在调用这些函数而不是引用。我要做的是在您的 Todo 组件内部：

handleClick() {
  this.props.clickHandler( this.props.todo );
{

在你的渲染中做

render: function() {
    if( this.props.todo.completed === true){
        return ( <li onClick={ this.handleClick } className="list-group-item list-group-item-success"><strike>{this.props.todo.task}</strike></li> );
    } else {
        return ( <li onClick={ this.handleClick } className="list-group-item"> {this.props.todo.task} </li> );
    }
}

您现在拥有它的方式实际上是在调用它，this.handleClick(this.props.todo)因此一旦组件呈现它就会调用该函数，该函数会立即在父组件中设置状态，这与反应模式 AKA 您没有在渲染方法中设置状态

于 2016-09-22T04:22:34.460 回答
1

对于给出的答案finalfreq

您还必须将函数与this

<li onClick={ this.handleClick.bind(this) }

抱歉，我没有 50 声望可以发表评论。所以写在这里

于 2016-10-04T07:06:26.907 回答