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

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

ただいまの
回答率

88.81%

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

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 323

mikeikeikename

score 14

前提・実現したいこと

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

発生している問題

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

該当のソースコード

<!DOCTYPE html>
<html lang="en">
<head>
  <meta charset="UTF-8">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <link rel ="stylesheet" href="css/style.css">
  <title>Todoリスト</title>
</head>
<body>
  <h1>Todoリスト</h1>
  <p>
    <input type="radio" name="status" value="1" checked="checked">全て
    <input type="radio" name="status" value="2">作業中
    <input type="radio" name="status" value="3">完了
    </p>

    <h3>ID コメント 状態</h3>
    <ul class="addTask-target">

    </ul>

    <h2>新規タスクの追加</h2>
    <input class="addTask-value" type="text" />
    <button class="addTask-trigger" type="button">追加</button>
  <script src="js/main.js"></script>
</body>
</html>
'use strict';

        // 必要なDOM要素を取得。上から登録、空のリスト、登録用インプット
        const addTaskTrigger = document.getElementsByClassName('addTask-trigger')[0];
        const addTaskTarget = document.getElementsByClassName('addTask-target')[0];
        const addTaskValue = document.getElementsByClassName('addTask-value')[0];

        //タスクを追加した時の挙動をaddTaskと定義
        const addTask = task => {
            // listItemと各ボタンを定義 p要素とボタン要素をHTMLに加える
              const listItem = document.createElement('p');
              const removeButton = document.createElement('button');
              const addButton = document.createElement('button');

              // それぞれ作業中、削除という言葉をボタンに入れる
        addButton.innerText = '作業中';
        removeButton.innerText = '削除';
        //ボタンを押したら以下の作業をする
        removeButton.addEventListener('click', () => removeTask(removeButton));

        listItem.append(1);
        listItem.innerText = task;
        // 作業中ボタンを追加
        listItem.append(addButton);
        // 削除ボタンを追加
        listItem.append(removeButton);
        //入力タスクを表示
        addTaskTarget.appendChild(listItem);
      };

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

試したこと

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

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

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

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

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

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

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

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

+ // 次の新規タスクに割り当てるID
+ let nextId = 0;

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

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

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

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

3. addTask の修正

3.1 第2引数 id を追加

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

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

3.2 listItem を <li>要素に修正

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

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

3.3 IDとタスクの表示部分

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

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


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

- listItem.append(1);
- listItem.innerText = task;
+ // IDを表示するspan要素を作成して listItem に追加
+ const idSpan = document.createElement('span');
+ idSpan.innerText = id;
+ listItem.append(idSpan);
+
+ // タスクを表示するspan要素を作成して listItem に追加
+ const taskSpan = document.createElement('span');
+ taskSpan.innerText = task;
+ listItem.append(taskSpan);

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/06/11 12:04

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

    キャンセル

  • 2020/06/11 12:28

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

    キャンセル

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

  • ただいまの回答率 88.81%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る