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

前提・実現したいこと

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、コメント、状態を３つ同時に出せるように配列を使ってみたが
動作せず。

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

行動規範の内容に同意します

回答1件

ベストアンサー

こんにちは
いろいろな修正方法が考えられますが、とり急ぎ、ご質問にあるコードに最小限の追加、修正をして、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);