質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.92%

React初心者 Todoリスト 削除 エラー

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 286

chigu4ps5

score 7

React初心者 Todoリスト 削除

私はReact初心者で、Todoリストを作成しているのですが、削除機能を付けようとしたところ、エラーがでてしまい、出来ませんでした。

コードは以下の通りです。
App

import React, { Component } from 'react';
import TodoList from './TodoList';
import TodoAdd from './TodoAdd';

class App extends React.Component {
  constructor(props) {
    super(props);
    this.state = {
      tasks: [],
      text: "",
      id: 0,
      uniquId: 0
    }
    this.addTask = this.addTask.bind(this);
    this.deleteTodo = this.deleteTodo.bind(this);
  }

  addTask(text) {
    const { tasks, uniquId } = this.state;
    const task = {
      text,
      id: uniquId
    };

    let add = tasks.concat(task)

    this.setState({
      tasks: add,
      uniquId: uniquId + 1
    });
  }

   deleteTodo(i) {
    let deleTask = this.state.tasks.splice(i, 1);
    this.setState({
      tasks: deleTask
    });
  }



  render() {
    return (
      <div>
        <h1>Todoリスト</h1>
        <TodoList tasks={this.state.tasks} deleteTodo={this.deleteTodo}/>
        <TodoAdd addTask={this.addTask}/>
      </div>
    );
  }
}

export default App;

TodoAdd

import React, { Component } from 'react';

class TodoAdd extends React.Component {
  constructor(props) {
    super(props);
    this.state = {
      inputValue: null,
    }
    this.handleChange = this.handleChange.bind(this);
    this.handleClick = this.handleClick.bind(this);
  }

  handleChange(e) {
    this.setState({
      inputValue: e.target.value,
    });
  }

  handleClick() {
    const inputValue = this.state.inputValue;
    this.props.addTask(inputValue);
    this.setState({
      inputValue: "",
    })
  }

  render() {
    return (
      <div>
        <input
          type="text"
          value={this.state.inputValue}
          onChange={ this.handleChange }
        />
        <button onClick={this.handleClick}>送信</button>
      </div>
    );   
  }
}

export default TodoAdd;

export default ToDoList;

class ToDoList extends React.Component{
  constructor(props) {
    super(props);
  }

  render() {
    return (
      <div>
        <ul>
          {this.props.tasks.map((item) => <li key={item.id} >
            {item.text}
            <button onClick={this.props.deleteTodo(item.id)}>削除</button>
          </li>)}
        </ul>

      </div>

    );
  };
}

export default TodoList;

index

import React from 'react';
import ReactDOM from 'react-dom';
import './App.css';
import App from './App';
import * as serviceWorker from './serviceWorker';

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

// If you want your app to work offline and load faster, you can change
// unregister() to register() below. Note this comes with some pitfalls.
// Learn more about service workers: https://bit.ly/CRA-PWA
serviceWorker.unregister();

Todoを入力すると以下のエラーが出ます。

エラー

Error: Maximum update depth exceeded. This can happen when a component repeatedly calls setState inside componentWillUpdate or componentDidUpdate. React limits the number of nested updates to prevent infinite loops.
▶ 4 stack frames were collapsed.
App.deleteTodo
src/App.js:35
  32 | 
  33 |   deleteTodo(i) {
  34 |     let deleTask = this.state.tasks.splice(i, 1);
> 35 |     this.setState({
  36 |       tasks: deleTask
  37 |     });
  38 | }
View compiled
(anonymous function)
src/TodoList.js:15
  12 |        <ul>
  13 |          {this.props.tasks.map((item) => <li key={item.id} >
  14 |            {item.text}
> 15 |            <button onClick={this.props.deleteTodo(item.id)}>削除</button>
     | ^  16 |          </li>)}
  17 |        </ul>
  18 | 
View compiled
ToDoList.render
src/TodoList.js:12
   9 | render() {
  10 |   return (
  11 |     <div>
> 12 |       <ul>
     | ^  13 |         {this.props.tasks.map((item) => <li key={item.id} >
  14 |           {item.text}
  15 |           <button onClick={this.props.deleteTodo(item.id)}>削除</button>

削除の部分が間違っているように思いますが、初心者のため、はっきりとした原因が分かりません。
教えていただけると嬉しいです。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

check解決した方法

+1

ネットでエラーを調べ、何とか解決致しました。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.92%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る