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

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

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

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

Q&A

解決済

1回答

1674閲覧

Javascript Todoリストでタスクごとにインデックスを表示する

mikeikeikename

総合スコア15

JavaScript

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

0グッド

0クリップ

投稿2020/06/10 15:15

前提・実現したいこと

Todoリストにおいて、タスクを入力するとIDの下に①番号(0,1,2とタスクを追加するごとに追加される)、②入力したタスク、③作業中のボタン、④削除のボタンを表示したい

発生している問題

②、③、④は作ることができたのですが、①のタスクを追加するごとに数値を出す作業が上手く出来ません。

該当のソースコード

HTML

1 2<!DOCTYPE html> 3<html lang="en"> 4<head> 5 <meta charset="UTF-8"> 6 <meta name="viewport" content="width=device-width, initial-scale=1.0"> 7 <link rel ="stylesheet" href="css/style.css"> 8 <title>Todoリスト</title> 9</head> 10<body> 11 <h1>Todoリスト</h1> 12 <p> 13 <input type="radio" name="status" value="1" checked="checked">全て 14 <input type="radio" name="status" value="2">作業中 15 <input type="radio" name="status" value="3">完了 16 </p> 17 18 <h3>ID コメント 状態</h3> 19 <ul class="addTask-target"> 20 21 </ul> 22 23 <h2>新規タスクの追加</h2> 24 <input class="addTask-value" type="text" /> 25 <button class="addTask-trigger" type="button">追加</button> 26 <script src="js/main.js"></script> 27</body> 28</html> 29

Javascript

1 2'use strict'; 3 4 // 必要なDOM要素を取得。上から登録、空のリスト、登録用インプット 5 const addTaskTrigger = document.getElementsByClassName('addTask-trigger')[0]; 6 const addTaskTarget = document.getElementsByClassName('addTask-target')[0]; 7 const addTaskValue = document.getElementsByClassName('addTask-value')[0]; 8 9 //タスクを追加した時の挙動をaddTaskと定義 10 const addTask = task => { 11 // listItemと各ボタンを定義 p要素とボタン要素をHTMLに加える 12 const listItem = document.createElement('p'); 13 const removeButton = document.createElement('button'); 14 const addButton = document.createElement('button'); 15 16 // それぞれ作業中、削除という言葉をボタンに入れる 17 addButton.innerText = '作業中'; 18 removeButton.innerText = '削除'; 19 //ボタンを押したら以下の作業をする 20 removeButton.addEventListener('click', () => removeTask(removeButton)); 21 22 listItem.append(1); 23 listItem.innerText = task; 24 // 作業中ボタンを追加 25 listItem.append(addButton); 26 // 削除ボタンを追加 27 listItem.append(removeButton); 28 //入力タスクを表示 29 addTaskTarget.appendChild(listItem); 30 }; 31 32 // 追加ボタンに対して、タスク登録イベントを設定 33 addTaskTrigger.addEventListener('click', event => { 34 const task = addTaskValue.value; 35 addTask(task); 36 addTaskValue.value = ''; 37 }); 38

試したこと

・IDの部分に数字を出せないに関して
→const indexをfor文を使って定義してみたが上手く行かず

const index =for(let i =0; i < 99; i++)

・ID、コメント、状態を3つ同時に出せるように配列を使ってみたが
動作せず。

以上です。
お手数ですが宜しくお願い致します。

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

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

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

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

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

guest

回答1

0

ベストアンサー

こんにちは
いろいろな修正方法が考えられますが、とり急ぎ、ご質問にあるコードに最小限の追加、修正をして、0から始まるIDを表示するようにするための一例を回答します。

1. 次の新規タスクに割り当てるID

次の新規タスクに割り当てるIDを保持する変数 nextId を追加して、0で初期化しておきます。追加する場所は、 const addTask の上あたりにしておきます。

diff

1+ // 次の新規タスクに割り当てるID 2+ let nextId = 0; 3 4 //タスクを追加した時の挙動をaddTaskと定義 5 const addTask = (task, id) => {

2. addTask の第2引数として、nextId を渡す。

addTask の第2引数として、(1)で追加したnextId を渡すようにします。現在の nextId の値が参照された後に、nextIdが1増えるように、後置インクリメントしておきます。

diff

1 // 追加ボタンに対して、タスク登録イベントを設定 2 addTaskTrigger.addEventListener('click', event => { 3 const task = addTaskValue.value; 4- addTask(task); 5+ addTask(task, nextId ++);

3. addTask の修正

3.1 第2引数 id を追加

(2)の修正によって渡されてくるid を表示できるように、addTask を修正します。まずは、引数を受け取れるようにします。

diff

1 //タスクを追加した時の挙動をaddTaskと定義 2- const addTask = task => { 3+ const addTask = (task, id) => {

3.2 listItem を <li>要素に修正

次に、listItem の作成部分です。listItem は、<ul> であるaddTaskTarget の子要素として追加される要素なので、p ではなく、li 要素として作成するように修正します。

diff

1- const listItem = document.createElement('p'); 2+ const listItem = document.createElement('li');

3.3 IDとタスクの表示部分

ID とタスクの表示部分で、現状では、

javascript

1listItem.append(1); 2listItem.innerText = task;

となっていますが、これだと、仮で表示させようとしている、IDの1が、その後の listItem.innerText = task; によって上書きされて消えてしまいます。ですので、IDとタスクのそれぞれを表示するための <span>を作って、listItemにappendするように修正します。表示するIDの値としては、引数で渡されてくる id を使用します。

diff

1- listItem.append(1); 2- listItem.innerText = task; 3+ // IDを表示するspan要素を作成して listItem に追加 4+ const idSpan = document.createElement('span'); 5+ idSpan.innerText = id; 6+ listItem.append(idSpan); 7+ 8+ // タスクを表示するspan要素を作成して listItem に追加 9+ const taskSpan = document.createElement('span'); 10+ taskSpan.innerText = task; 11+ listItem.append(taskSpan);

ここまでの修正で、とりあえず、タスクの直前に、0始まりのIDが表示されるようになります。以下は、上記の修正を行ったコードを動作確認するサンプルです。

修正後のコードについてもリファクタリングの余地があるとは思いますが、とりあえず現状のコードの作り方にそった追加、修正によって意図通り、0から始まるIDを表示させる一例になります。

以上、参考になれば幸いです。

投稿2020/06/10 18:49

jun68ykt

総合スコア9058

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

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

mikeikeikename

2020/06/11 03:04

丁寧なご確認ありがとうございます。 無事解決致しました。ベストアンサーとさせていただきました。
jun68ykt

2020/06/11 03:28

どういたしまして???? > 無事解決致しました。 とのことでよかったです。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問