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

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

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

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

JavaScript

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

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

Q&A

解決済

1回答

2813閲覧

【Todoリスト】「追加」ボタンを押すとタスクが表示→非表示を繰り返す

退会済みユーザー

退会済みユーザー

総合スコア0

HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

JavaScript

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

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

0グッド

0クリップ

投稿2020/09/27 20:11

JavaScriptでTodoリストを作成しているのですが、現在の状況としては以下の様に「追加」ボタンを押すとタスクが表示→非表示を繰り返します。
イメージ説明

●ソースコード

HTML

1<!DOCTYPE html> 2<html lang="ja"> 3<head> 4 <meta charset="UTF-8"> 5 <meta name="viewport" content="wclassth=device-wclassth, initial-scale=1.0"> 6 <title>Document</title> 7 <link rel="stylesheet" href="https://stackpath.bootstrapcdn.com/bootstrap/4.3.1/css/bootstrap.min.css" integrity="sha384-ggOyR0iXCbMQv3Xipma34MD+dH/1fQ784/j6cY/iJTQUOhcWr7x9JvoRxT2MZw1T" crossorigin="anonymous"> 8 <link rel="stylesheet" href="style.css"> 9</head> 10<body> 11 <div class="container"> 12 <h1>Todoリスト</h1> 13 14 <table class="todos"> 15 <tr> 16 <th>ID</th> 17 <th>コメント</th> 18 <th>状態</th> 19 </tr> 20 </table> 21 22 <form class="add my-4"> 23 <label class="d-block font-weight-bold">新規タスクの追加</label> 24 <input type="text" class="form-control col-4 d-inline-block add-text" name="add" placeholder="New Text"> 25 <button type="button" class="add-btn d-inline-block col-1">追加</button> 26 </form> 27 </div> 28 29 <script src="https://code.jquery.com/jquery-3.3.1.slim.min.js" integrity="sha384-q8i/X+965DzO0rT7abK41JStQIAqVgRVzpbzo5smXKp4YfRvH+8abtTE1Pi6jizo" crossorigin="anonymous"></script> 30 <script src="https://cdnjs.cloudflare.com/ajax/libs/popper.js/1.14.7/umd/popper.min.js" integrity="sha384-UO2eT0CpHqdSJQ6hJty5KVphtPhzWj9WO1clHTMGa3JDZwrnQq4sF86dIHNDz0W1" crossorigin="anonymous"></script> 31 <script src="https://stackpath.bootstrapcdn.com/bootstrap/4.3.1/js/bootstrap.min.js" integrity="sha384-JjSmVgyd0p3pXB1rRibZUAYoIIy6OrQ6VrjIEaFf/nJGzIxFDsf4x0xIM+B07jRM" crossorigin="anonymous"></script> 32 <script src="js/main.js"></script> 33</body> 34</html>

JavaScript

1const todos = []; 2const input = document.querySelector('.add-text'); 3const addButton = document.querySelector('.add-btn'); 4const table = document.querySelector('.todos'); 5 6const showTodo = () => { 7 //一旦すべて削除 8 const todo = document.querySelector('.todo'); 9 while(todo) { 10 table.removeChild(todo); 11 } 12 13 todos.forEach((todo, index) => { 14 const tr = document.createElement('tr'); 15 tr.className = 'todo'; 16 const tdId = document.createElement('td'); 17 const tdComment = document.createElement('td'); 18 tdId.textContent = `${index + 1}`; 19 tdComment.textContent = `${todo}`; 20 tr.appendChild(tdId); 21 tr.appendChild(tdComment); 22 23 // 状態ボタンの作成 24 const stateButton = document.createElement('button'); 25 const tdStateButton = document.createElement('td'); 26 stateButton.textContent = '作業中'; 27 stateButton.className = 'state-btn'; 28 tdStateButton.appendChild(stateButton); 29 30 //削除ボタンの作成 31 const removeButton = document.createElement('button'); 32 const tdRemoveButton = document.createElement('td'); 33 removeButton.textContent = '削除'; 34 removeButton.className = 'remove-btn'; 35 tdRemoveButton.appendChild(removeButton); 36 37 //作成した要素をHTML上に表示させる 38 tr.appendChild(tdStateButton); 39 tr.appendChild(tdRemoveButton); 40 table.appendChild(tr); 41 }); 42}; 43 44addButton.addEventListener('click', event => { 45 const todo = input.value; 46 input.value = ''; 47 if(todo && todo.match(/\S/g)){ 48 todos.push(todo); 49 showTodo(); 50 } 51});

いったんすべての要素を削除してから、再度表示するというやり方をとっているのですが、削除するときにうまくいってないのだと思います。
しかしそこを重点的に確認しても、どこが悪いのかがわかりません。
素人質問ですみませんが、わかる方がいれば是非教えてください。

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

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

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

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

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

guest

回答1

0

ベストアンサー

以下の様にしたらうまくいきました。

JavaScript

1while(document.querySelector('.todo')) { 2 table.removeChild(document.querySelector('.todo')); 3}

またquerySelectorAllを使うと以下のようにもできます。

JavaScript

1const children = document.querySelectorAll('.todo'); 2children.forEach(child => { 3 if(child) { 4 table.removeChild(child); 5 } 6});

querySelectorAllはNodeListというもので配列ではないが、forEachは使えるとのことです。
原因がいまだ分からないので、根本的な解決にはなっていません。

投稿2020/09/27 20:45

編集2020/09/27 22:26
退会済みユーザー

退会済みユーザー

総合スコア0

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.35%

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

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

質問する

関連した質問