Как удалить выбранный элемент реагировать - PullRequest
0 голосов
/ 21 февраля 2019

У меня есть два компонента Display.jsx и DisplayList.jsx.Компоненты работают вместе для отображения значений из локального хранилища.Проблема в цикле метода handleDelete () DisplayList.JSX.

Я не понимаю, почему он удаляет первый элемент вместо выбранного?И как это исправить?

Github

Display.jsx

import {DisplayList} from './DisplayList';


class Display extends Component {
  constructor(props){
    let data = JSON.parse(localStorage.getItem('data'));
    super(props)
    this.state = {
      data: data,
  }

  // Methods
  this.displayValues = this.displayValues.bind(this);
  }

  displayValues(){

   return this.state.data.map((data1, index) =>
    <DisplayList
      key = {index}
      email = {data1.email}
      password = {data1.password}
      updateList = {this.updateList}
       /> 
    )
  }
  // This is the method that will be called from the child component.
  updateList = (data) => {
    this.setState({
      data
    });
  }
  render() {
    return (
      <ul className="list-group">
        {this.displayValues()}
      </ul>
    )
  }
}

export default Display;

DisplayList.jsx

import React, { Component } from 'react'
import {Button, Modal, Form} from 'react-bootstrap';


export class DisplayList extends Component {

    constructor(props){
        super(props)
        this.state = {
            email: '',
            password: '',
            show: false,
        };

        // Methods
        this.handleDelete = this.handleDelete.bind(this);
        this.onChange = this.onChange.bind(this);
    }

    onChange(event){
        this.setState({
            [event.target.name]: event.target.value
        })
    };   
    handleDelete(){
        const data = JSON.parse(localStorage.getItem('data'));
        for (let index = 0; index < data.length; index++) {
            if(this.props.email === data[index].email &&
                this.props.password === data[index].password){
                  console.log(this.props.email);
                  console.log(data[index]);
                data.splice(data[index], 1);
            }
            console.log('loop count');
        }
        localStorage.setItem('data', JSON.stringify(data));
        this.props.updateList(data);
    }


  render() {
    return (
    <div className = "mt-4">
        <li className="list-group-item text-justify">
            Email: {this.props.email} 
            <br /> 
            Password: {this.props.password}
            <br /> 
            <Button onClick = {this.handleShow} variant = "info mr-4 mt-1">Edit</Button>
            <Button onClick = {this.handleDelete} variant = "danger mt-1">Delete</Button>
        </li>
    </div>
    )
  }
}

Ответы [ 2 ]

0 голосов
/ 21 февраля 2019
handleDelete(){
  const data = JSON.parse(localStorage.getItem('data'));
  const filteredData = data.filter(
      x => !(x.email === this.props.email && x.password === this.props.password)
   );
  localStorage.setItem('data', JSON.stringify(filteredData));
  this.props.updateList(filteredData);
}
0 голосов
/ 21 февраля 2019

Вместо:

data.splice(data[index], 1);

Попробуйте переписать весь массив:

data = [
    ...data.slice(0, index),
    ...data.slice(index + 1)
];
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...