0

我正在制作一个时间表应用程序。index.js 呈现表格。表格的行从子数组中读取。children 数组读取状态以保持自身更新。AddRow() 工作正常。DeleteRow() 有一些问题。我打算从事件中获取 parentNode id，查找状态的数组，拼接它并让孩子自己更新——这反过来又更新了渲染。因此，当一行被删除时，它会从状态数组中拼接出来，然后在 children 数组中更新，然后再渲染。DeleteRow 运行除 array.splice 或 indexOf 部分之外的所有内容。我什至尝试了一个 for 循环，但它不起作用。我的方法正确吗？

index.js    
import React from "react";
import ReactDOM from "react-dom";
import HeaderAndFirstTableRow from "./headerandfirstrow";
import TableRow from "./tablerow";
import "./styles.css";

class App extends React.Component {
  constructor(props) {
    super(props);
    this.state = { noOfRows: [] };
    this.addRow = this.addRow.bind(this);
    this.deleteRow = this.deleteRow.bind(this);
    this.find = this.find.bind(this);
  }
  addRow() {
    let arr = this.state.noOfRows;
    let arrLength = arr.length;
    arr.push(arrLength + 1);
    this.setState({
      noOfRows: arr
    });
  }
  find(arr, id) {
    for (let i = 0; i < arr.length; i++) {
      if (arr[i] === id) {
        return i;
      }
    }
  }
  deleteRow(event) {
    let arr = this.state.noOfRows;
    let id = event.target.parentNode.id;
    let ix = arr.findIndex(id);
    arr.splice(ix, 1);
    this.setState({ noOfRows: arr });
  }
  render() {
    let children = [];
    for (let i = 0; i < this.state.noOfRows.length; i++) {
      children.push(
        <TableRow id={this.state.noOfRows.length} delete={this.deleteRow} />
      );
    }
    return (
      <HeaderAndFirstTableRow click={this.addRow}>
        {children}
      </HeaderAndFirstTableRow>
    );
  }
}
const rootElement = document.getElementById("root");
ReactDOM.render(<App />, rootElement);

headerandfirstrow.js

import React from "react";

import "./styles.css";

export default function HeaderAndFirstTableRow(props) {
  function handler() {
    props.click();
  }
  return (
    <table>
      <tr>
        <th>Feature</th>
        <th>Phase</th>
        <th>Comments</th>
        <th>Monday</th>
        <th>Tuesday</th>
        <th>Wednesday</th>
        <th>Thursday</th>
        <th>Friday</th>
        <th>Saturday</th>
        <th>Sunday</th>
        <th onClick={handler}>+</th>
      </tr>

      {props.children}
    </table>
  );
}

tablerow.js
import React from "react";
import "./styles.css";
export default function TableRow(props) {
  function del(e) {
    props.delete(e);
  }
  return (
    <tr id={props.id}>
      <td />
      <td />
      <td />
      <td />
      <td />
      <td />
      <td />
      <td />
      <td />
      <td />
      <td onClick={del}>-</td>
    </tr>
  );
}
4

3 回答 3

1

splice改变原始数组，你不应该改变状态，

你可以改成这个

deleteRow(event) {
    let arr = this.state.noOfRows;
    let id = event.target.parentNode.id;
    let filteredArray = arr.filter(val=> val !== id)
    this.setState({ noOfRows: filteredArray });
}
于 2019-06-01T06:43:57.940 回答
0

使用indexOf而不是findIndex. 此外，始终复制对象而不是直接将它们变异到状态中。

这是工作小提琴： https ://stackblitz.com/edit/react-6hfnww

于 2019-06-01T06:44:36.923 回答
0

拼接this.state.noOfRows数组会改变其元素。

在deleteRows中，这是设置相同的数组，该数组现在在组件状态中变回了。

而是使用Array.prototype.slice来获取没有该项的数组块，并从中组成一个新数组。

  deleteRow(event) {
    let arr = this.state.noOfRows;
    let id = event.target.parentNode.id;
    let ix = arr.findIndex(id);
    const noOfRows = [...arr.slice(0, ix), ...arr.slice(ix+1)] ;
    this.setState({ noOfRows });
  }
于 2019-06-01T06:44:46.327 回答