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

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

新規登録して質問してみよう
ただいま回答率
85.48%
React.js

Reactは、アプリケーションのインターフェースを構築するためのオープンソースJavaScriptライブラリです。

Q&A

解決済

1回答

957閲覧

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

chigu4ps5

総合スコア7

React.js

Reactは、アプリケーションのインターフェースを構築するためのオープンソースJavaScriptライブラリです。

0グッド

0クリップ

投稿2019/12/03 06:45

編集2019/12/04 09:28

###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>

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

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

自己解決

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

投稿2019/12/04 12:55

chigu4ps5

総合スコア7

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問