React useState钩子不会更新状态

时间:2019-02-20 14:56:38

标签: javascript reactjs react-hooks

我遇到了useState问题。以下是在codeandbox中运行的代码

https://codesandbox.io/s/kmznl0345r

这是代码本身;

import React, { Fragment, useState } from "react";
import ReactDOM from "react-dom";

type FormElem = React.FormEvent<HTMLFormElement>;

interface ITodo {
  text: string;
  complete: boolean;
}

export default function App(): JSX.Element {
  const [value, setValue] = useState<string>("");
  const [todos, setTodos] = useState<ITodo[]>([]);

  const handleSubmit = (e: FormElem): void => {
    e.preventDefault();
    addTodo(value);
    setValue("");
  };

  const addTodo = (text: string): void => {
    const newTodos: ITodo[] = [...todos, { text, complete: false }];
    setTodos(newTodos);
  };

  const completeTodo = (index: number): void => {
    const newTodos: ITodo[] = todos;
    newTodos[index].complete = !newTodos[index].complete;
    setTodos(newTodos);
  };

  return (
    <Fragment>
      <h1>Todo List</h1>
      <form onSubmit={handleSubmit}>
        <input
          type="text"
          value={value}
          onChange={e => setValue(e.target.value)}
          required
        />
        <button type="submit">Add Todo</button>
      </form>
      <section>
        {todos.map((todo: ITodo, index: number) => (
          <Fragment key={index}>
            <div>{todo.text}</div>
            <button type="button" onClick={() => completeTodo(index)}>
              {" "}
              {todo.complete ? "Incomplete" : "Complete"}{" "}
            </button>
          </Fragment>
        ))}
      </section>
    </Fragment>
  );
}

const root = document.getElementById("root");

ReactDOM.render(<App />, root);

单击完成按钮时,它仅运行completeTodo函数一次,即使setTodos函数已运行并且待办事项已更新,它也不会再次运行。

这曾经在react 16.7.0-alpha-2中起作用,但是现在版本为16.8.1,它似乎没有更新。

让我知道您有任何建议,这里再次是在codeandbox中运行的代码;

https://codesandbox.io/s/kmznl0345r

1 个答案:

答案 0 :(得分:4)

您当前正在todo函数中对completeTodo对象进行突变。如果您改为使用todos中的所有元素创建一个新数组,并在其中切换了complete的位置创建一个新对象,它将按预期工作。

const completeTodo = (index: number): void => {
  const newTodos: ITodo[] = [...todos];
  newTodos[index] = {
    ...newTodos[index],
    complete: !newTodos[index].complete
  };
  setTodos(newTodos);
};