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

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

ただいまの
回答率

88.78%

Javascript Todoリスト タスクの追加とインデックスの初期化

解決済

回答 1

投稿

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

mikeikeikename

score 14

前提・実現したいこと

①Todoリストで、追加ボタンを押すとID(インデックス)、タスク、作業中、削除ボタンの4つがtdタグに順次追加されるようにしたい
②削除ボタンを押したらタスクが消えて、インデックスが初期化されているようにしたい

発生している問題・エラーメッセージ

①タスクを追加すると、下に追加されていくのではなく、同じ場所でインデックス番号のみ変わっていく仕様になっている。addTaskとaddButtonのオブジェクトは分けて実装したいです。

②初期化のメドッドを用いているが動作しない

該当のソースコード

<!DOCTYPE html>
<html lang="ja">
<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>
<p></p>
    <table>
      <thead>
        <th>ID</th>
        <th>コメント</th>
        <th>状態</th>
        <th></th>
      </thead>
      <tbody class ="addTask-target"></tbody>
    </table>
    <h2>新規タスクの追加</h2>
    <input class="addTask-value" type="text" />
    <button class="addTask-trigger" type="button">追加</button>
  <script src="js/main.js"></script>
</body>

</script>
</html>
    {
    document.addEventListener('DOMContentLoaded', function() {
    const addTaskTrigger = document.getElementsByClassName('addTask-trigger')[0];
    const addTaskTarget = document.getElementsByClassName('addTask-target')[0];
    const addTaskValue = document.getElementsByClassName('addTask-value')[0];
    let nextId = 0;
    const todos = [];

    //テーブル要素を生成する td要素を作る
    const tableItem = document.createElement('tr');
    const idSpanTd = document.createElement('td');
    const taskSpanTd = document.createElement('td');
    const createButtonTd = document.createElement('td');
    const removeButtonTd = document.createElement('td');

    const addTask = (task, id) => {    
      //要素内のHTML文章を変更する
      idSpanTd.innerText = id;
      taskSpanTd.innerText = task;
      //生成したテーブル要素をブラウザに表示する
      tableItem.append(idSpanTd);
      tableItem.append(taskSpanTd);
      addTaskTarget.appendChild(tableItem);
      return(task,id)
    };

    //Button要素を生成する
    const removeButton = document.createElement('button');
    const createButton = document.createElement('button');

    const addButton = (button) => {
      //要素内のHTML文章を変更する
      createButton.innerText = '作業中';
      removeButton.innerText = '削除';
      //生成したテーブル要素をブラウザに表示する
      tableItem.append(createButtonTd);
      tableItem.append(removeButtonTd);
      addTaskTarget.appendChild(tableItem);
       //生成したbutton要素を生成する
       createButtonTd.append(createButton);
       removeButtonTd.append(removeButton); 
       return(button)
      };

      //追加ボタンをクリックした際にタスクを追加する処理を行う
    addTaskTrigger.addEventListener('click', () => {
       const task = addTaskValue.value;
       addTask(task, nextId++);
       addButton();
       addTaskValue.value = '';
      });

    //チェックリスト用オブジェクト
    const todo = {
       task: 'taskSpanTd',
       status: '作業中'
      };
    todos.push(todo);

    removeButton.addEventListener('click', delete_element, false);

    // //削除ボタンを押した時にタスクを削除する
    function delete_element () {
      let tabletag = this.closest ('tr');
      if (tabletag)
      tabletag.remove ();
    }  
    });
  }

試したこと

①addTaskとaddButtonにそれぞれ以下の記述が無いためクリックした際に下に追加されないのではないかと思い、以下の要素を関数にしてaddTaskとaddButtonの中に入れましたが、上手く機能せずでした。

  //テーブル要素を生成する td要素を作る
    const tableItem = document.createElement('tr');
    const idSpanTd = document.createElement('td');
    const taskSpanTd = document.createElement('td');
    const createButtonTd = document.createElement('td');
    const removeButtonTd = document.createElement('td');

②以下の関数を使用していますが機能せずです。

  //  インデックスの初期化
    //   function initialize() {
    //   if(tr)
    //   tr.initialize();
    // };
    //   document.addEventListener("DOMContentLoaded", initialize.bind(this));

お手数ですがご確認宜しくお願い致します。

補足情報(FW/ツールのバージョンなど)

ここにより詳細な情報を記載してください。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+2

①タスクを追加すると、下に追加されていくのではなく、同じ場所でインデックス番号のみ変わっていく仕様になっている。
addTaskとaddButtonのオブジェクトは分けて実装したいです。 

同じ場所でインデックス番号のみ変わっていく動作になっている原因は、追加ボタンをクリックした際にtableItemなどに新しく作成した要素が代入されていないからです。
「試したこと」書いているコードは、tableItemなどと名前は同じであるが新たな別の定数を作ってそこに新しく作成した要素を代入するという意味のコードになっています。
単なる代入が行いたい場合は、頭にconstのキーワードはつけてはいけません。
またconstで宣言したものは定数であり代入が行えないので、constではなくletで宣言を行って下さい。

ちなみに、仕様というのは望む動作のことです。意図に反している動作のことは仕様とは呼びません。

②初期化のメドッドを用いているが動作しない 

メドッドという言葉はありません。メソッド(Method)です。

また「初期化」とは一体何がしたいのか分かりません。

「試したこと」にtr.initialize()というコードがありますが、trという変数は定義されていませんし、htmlの要素はinitializeというメソッドを持っていないので実行するとエラーになります。

それなりに動くコード:

    {
    document.addEventListener('DOMContentLoaded', function() {
    const addTaskTrigger = document.getElementsByClassName('addTask-trigger')[0];
    const addTaskTarget = document.getElementsByClassName('addTask-target')[0];
    const addTaskValue = document.getElementsByClassName('addTask-value')[0];
    let nextId = 0;
    const todos = [];

    let tableItem;
    let idSpanTd;
    let taskSpanTd;
    let createButtonTd;
    let removeButtonTd;

    const addTask = (task, id) => {    
      //要素内のHTML文章を変更する
      idSpanTd.innerText = id;
      taskSpanTd.innerText = task;
      //生成したテーブル要素をブラウザに表示する
      tableItem.append(idSpanTd);
      tableItem.append(taskSpanTd);
      addTaskTarget.appendChild(tableItem);
      return(task,id)
    };

    let removeButton;
    let createButton;

    const addButton = (button) => {
      //要素内のHTML文章を変更する
      createButton.innerText = '作業中';
      removeButton.innerText = '削除';
      //生成したテーブル要素をブラウザに表示する
      tableItem.append(createButtonTd);
      tableItem.append(removeButtonTd);
      addTaskTarget.appendChild(tableItem);
       //生成したbutton要素を生成する
       createButtonTd.append(createButton);
       removeButtonTd.append(removeButton); 
       return(button)
      };

      //追加ボタンをクリックした際にタスクを追加する処理を行う
    addTaskTrigger.addEventListener('click', () => {
       //テーブル要素を生成する td要素を作る
       //Button要素を生成する
       tableItem = document.createElement('tr');
       idSpanTd = document.createElement('td');
       taskSpanTd = document.createElement('td');
       createButtonTd = document.createElement('td');
       removeButtonTd = document.createElement('td');
       removeButton = document.createElement('button');
       createButton = document.createElement('button');

       const task = addTaskValue.value;
       addTask(task, nextId++);
       addButton();
       addTaskValue.value = '';

       removeButton.addEventListener('click', delete_element, false);

      });

    //チェックリスト用オブジェクト
    const todo = {
       task: 'taskSpanTd',
       status: '作業中'
      };
    todos.push(todo);

    // //削除ボタンを押した時にタスクを削除する
    function delete_element () {
      let tabletag = this.closest ('tr');
      if (tabletag)
      tabletag.remove ();
    }  
    });
  }

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/06/25 00:03

    ku__ra__ge様
    ご返信遅くなり申し訳ありません。
    ご教示ありがとうございます。
    思ったように動きませんが、引き続き模索してみます。

    キャンセル

  • 2020/06/25 00:13

    それなりに動くコードを追記しました。
    コードとしては問題が多いですが、差分がわかりやすいように最小限の変更にしてあるので自分の書いたコードと比較してみてください。

    キャンセル

  • 2020/06/25 17:59

    ご丁寧にありがとうございます。
    動作する要因を考えながら試してみたいと思います。

    キャンセル

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

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

関連した質問

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