ReactJS Array.push函数在setState中不起作用

Question

到目前为止，我正在制作一个有3个问题的原始测验应用，无论是真是假。在我的handleContinue方法中，有一个调用将用户输入从无线电形式推送到userAnswers数组。它适用于第一次运行handleContinue，之后它会抛出一个错误：Uncaught TypeError: this.state.userAnswers.push is not a function(…)

import React from "react"

export default class Questions extends React.Component {
  constructor(props) {
    super(props)
    this.state = {
      questionNumber: 1,
      userAnswers: [],
      value: ''
    }

    this.handleContinue = this.handleContinue.bind(this)
    this.handleChange = this.handleChange.bind(this)
  }

  //when Continue button is clicked
  handleContinue() {
    this.setState({
      //this push function throws error on 2nd go round
      userAnswers: this.state.userAnswers.push(this.state.value),
      questionNumber: this.state.questionNumber + 1
    //callback function for synchronicity
    }, () => {
      if (this.state.questionNumber > 3) {
        this.props.changeHeader(this.state.userAnswers.toString())
        this.props.unMount()
      } else {
        this.props.changeHeader("Question " + this.state.questionNumber)
      }
    })
    console.log(this.state.userAnswers)
  }

  handleChange(event) {
    this.setState({
      value: event.target.value
    })
  }

  render() {
    const questions = [
      "Blargh?",
      "blah blah blah?",
      "how many dogs?"
    ]
    return (
      <div class="container-fluid text-center">
        <h1>{questions[this.state.questionNumber - 1]}</h1>
        <div class="radio">
          <label class="radio-inline">
            <input type="radio" class="form-control" name="trueFalse" value="true" 
            onChange={this.handleChange}/>True
          </label><br/><br/>
          <label class="radio-inline">
            <input type="radio" class="form-control" name="trueFalse" value="false" 
            onChange={this.handleChange}/>False
          </label>
          <hr/>
          <button type="button" class="btn btn-primary" 
          onClick={this.handleContinue}>Continue</button>
        </div>
      </div>
    )
  }
}

Answer 1

Do not modify state directly!一般来说，尝试avoid mutation。

Array.prototype.push()就地改变了阵列。所以基本上，当你将push放到setState中的一个数组时，你可以通过使用push来改变原始状态。因为push返回新的数组长度而不是实际的数组，所以你将this.state.userAnswers设置为数值，这就是你在第二次运行时得到Uncaught TypeError: this.state.userAnswers.push is not a function(…)的原因，因为你不能将push改为数字。

你需要使用Array.prototype.concat()代替。它不会改变原始数组，并返回带有新连接元素的新数组。这就是你想在setState里面做的事情。您的代码应如下所示：

this.setState({
  userAnswers: this.state.userAnswers.concat(this.state.value),
  questionNumber: this.state.questionNumber + 1
}

Answer 2

Array.push不返回新数组。尝试使用

this.state.userAnswers.concat([this.state.value])

这将返回新的userAnswers数组

参考文献：array push和array concat

Answer 3

您应该将状态对象视为不可变，但是您需要重新创建数组，使其指向新对象，设置新项，然后重置状态。

 handleContinue() {
    var newState = this.state.userAnswers.slice();
    newState.push(this.state.value);

    this.setState({
      //this push function throws error on 2nd go round
      userAnswers: newState,
      questionNumber: this.state.questionNumber + 1
    //callback function for synchronicity
    }, () => {
      if (this.state.questionNumber > 3) {
        this.props.changeHeader(this.state.userAnswers.toString())
        this.props.unMount()
      } else {
        this.props.changeHeader("Question " + this.state.questionNumber)
      }
    })
    console.log(this.state.userAnswers)
  }

上述解决方案的另一种替代方法是使用.concat()，因为它本身返回一个新数组。它等同于创建一个新变量，但是代码要短得多。

this.setState({
  userAnswers: this.state.userAnswers.concat(this.state.value),
  questionNumber: this.state.questionNumber + 1
}

Answer 4

我找到了解决方案。这应该适用于拼接和其他人。让我们说我有一个汽车阵列：

this.state = {
  cars: ['BMW','AUDI','mercedes']
};
this.addState = this.addState.bind(this);

现在，addState是我将用于向我的数组添加新项目的方法。这应该是这样的：

  addState(){

let arr = this.state.cars;
arr.push('skoda');
this.setState({cars: arr});
}

由于duwalanise，我找到了这个解决方案。我所要做的就是返回新阵列以推送新项目。我很长一段时间都面临着这样的问题。我将尝试更多的函数来查看它是否真的适用于通常不会的所有函数。如果有人更清楚如何使用更干净的代码实现这一目标，请随时回复我的帖子。

Answer 5

后来的React版本中推荐的方法是在修改状态时使用更新程序功能以防止竞争条件：

this.setState(prevState => ({
  userAnswers: [...prevState.userAnswers, this.state.value] 
}));