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

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

新規登録して質問してみよう
ただいま回答率
85.35%
JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

Q&A

2回答

1792閲覧

行を削除する処理の実装について

ruuuu

総合スコア176

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

0グッド

0クリップ

投稿2020/12/22 03:54

前提・実現したいこと

現在、Todoアプリを作成しているのですが、削除ボタンを押した際に、レコードが削除される処理を実装したいと考えています。



該当のソースコード

html

1<!DOCTYPE html> 2<html lang="ja"> 3<head> 4 <meta charset="UTF-8"> 5 <meta name="viewport" content="width=device-width, initial-scale=1.0"> 6 <link rel=”stylesheet” type="text/css" href="css/kadai-3.css"> 7 <title>課題3</title> 8</head> 9<body> 10 <h1>ToDoリスト</h1> 11 <div> 12 <input type="radio" name="todo_list" id="check_value_left" value="all" >すべて 13 <input type="radio" name="todo_list" id="check_value_middle" value="working" >作業中 14 <input type="radio" name="todo_list" id="check_value_right" value="complete" >完了 15 </div> 16 <table id="task_table"> 17 <thead> 18 <tr> 19 <th>ID</th> 20 <th>コメント</th> 21 <th>状態</th> 22 <th></th> 23 </tr> 24 </thead> 25 <tbody id="task_list"></tbody> 26 </table> 27 <h3>新規タスクの追加</h3> 28 <input type="text" id="task_text"> 29 <button type="submit" id="task_add_button">追加</button> 30</body> 31<script type="text/javascript" src="js/kadai-3.js"></script> 32</html>

js

1'use strict'; 2window.addEventListener('DOMContentLoaded', () => { 3 let id = 0; 4 const todos = []; 5 document.getElementById('task_add_button').addEventListener('click', () => { 6 const taskElem = document.getElementById('task_text'); 7 const taskText = taskElem.value; 8 if(!taskText){ 9 return 10 } 11 const todo = { 12 task: taskText, 13 status: '作業中' 14 } 15 todos.push(todo); 16 displayData(); 17 id += 1; 18 taskElem.value = ''; 19 }); 20 21 const displayData = () => { 22 const tbody = document.getElementById('task_list'); 23 const row = tbody.insertRow(-1); 24 // console.log(row.rowIndex); 25 const tdId = row.insertCell(); 26 const tdComment = row.insertCell(); 27 const tdStateButton = row.insertCell(); 28 const tdDeleteButton = row.insertCell(); 29 const stateButton = createStateButton(); 30 const deleteButton = createDeleteButton(); 31 tdId.textContent = id; 32 tdComment.textContent = todos[id].task; 33 tdStateButton.appendChild(stateButton); 34 tdDeleteButton.appendChild(deleteButton); 35 } 36 37 const createStateButton = () => { 38 const stateButton = document.createElement('button'); 39 stateButton.setAttribute('value', 'working'); 40 stateButton.textContent = todos[id].status; 41 return stateButton 42 } 43 const createDeleteButton = () => { 44 const deleteButton = document.createElement('button'); 45 deleteButton.setAttribute('value', '削除'); 46deleteButton.onclick = deleteRow(this); 47 deleteButton.textContent = '削除'; 48 return deleteButton 49 } 50const deleteRow = (obj) =>{ 51 // tr = obj.parentNode.parentNode; 52 // // trのインデックスを取得して行を削除する 53 // tr.parentNode.deleteRow(tr.sectionRowIndex); 54 console.log(obj); 55 } 56});

試したこと

こちらの記事を参考に、onlickを挿入する処理を挟んでみたのですが、行のオブジェクトが取得出来ずコンソールにオブジェクトの情報を出力することが出来ませんでした。
こちら、何か行を削除する良い方法などありましたら、アドバイス頂けましたら幸いです。

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

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

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

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

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

guest

回答2

0

関数は()をつけることで即時実行されるので、Consoleを見ていくと分かるように
createDeleteButton時に実行されています。

単に下記で良いのでは。

js

1 deleteButton.onclick = deleteRow;

私の手元の方ではこれで「ボタンクリック時」にイベント取得できました。
あとは

js

1 2 console.log(obj.currentTarget);

とでもすれば、「押したボタン自身」が取れます。

投稿2020/12/22 04:06

m.ts10806

総合スコア80875

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

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

ruuuu

2020/12/22 07:04 編集

ご回答ありがとうございます。 > deleteButton.onclick = deleteRow; こちらのように修正しました。 実は、以下の形では、エラーが発生してしまい、行を削除することが出来ませんでした tr = obj.parentNode.parentNode; // trのインデックスを取得して行を削除する tr.parentNode.deleteRow(tr.sectionRowIndex); エラーは以下の通りです。 「 Uncaught TypeError: Cannot read property 'parentNode' of undefined」 行を削除しようと思った場合、どのようにコードを修正すれば良いでしょうか...?
guest

0

deleteRow関数の引数として渡されるのはMouseEventオブジェクトです。

Eventオブジェクトからクリックされた要素を取得するには、Event.targetプロパティを読み取ります。

js

1const deleteRow = (obj) =>{ 2 tr = obj.target.parentNode.parentNode; 3 // trのインデックスを取得して行を削除する 4 tr.parentNode.deleteRow(tr.sectionRowIndex); 5 }

また、宣言してない変数trに代入しようとしているのでまたエラーとなっています。
変数宣言も追加する必要があります。

js

1const deleteRow = (obj) =>{ 2 const tr = obj.target.parentNode.parentNode; 3 // trのインデックスを取得して行を削除する 4 tr.parentNode.deleteRow(tr.sectionRowIndex); 5 }

投稿2020/12/22 07:22

azukiazusa

総合スコア112

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

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

ruuuu

2020/12/22 09:17 編集

ご回答ありがとうございます。 >const tr = obj.target.parentNode.parentNode; 「const」を付加してみたのですが、以下のエラーは消えず表示されてしまっている状況です... Uncaught TypeError: Cannot read property 'parentNode' of undefined
azukiazusa

2020/12/22 09:28

`console.log(obj)`で何が出力されますか?
ruuuu

2020/12/22 09:34 編集

現状、エラー出力のみで「console.log(obj)」の結果は表示されていませんでした...
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.35%

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

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

質問する

関連した質問