ボタンを押すとJavaScriptのcreateElementでテーブルデータを増やす機能の実装

###質問したいこと：JavaScriptでボタンを押すとテーブルが作られそれぞれのtdタグごとに違う内容を表示したい。

現在、Todoアプリの実装のための練習として、ボタンを押すと、(1)ボタンを押した回数の数値(2)未完了と書かれたボタン(3)削除と書かれたボタン、の以上3つをそれぞれtdタグに入れるプログラムを実装しようとしています。

<!DOCTYPE html>
<html lang="ja">
<head>
  <meta charset="UTF-8">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <meta http-equiv="X-UA-Compatible" content="ie=edge">
  <title>Document</title>
</head>
<body>
  <button>増えるよ</button>
  <div id="hoge"></div>

  <script>
    let i = 1;
    let table = document.createElement("table");
    table.setAttribute("border", 1)
    document.body.appendChild(table);

    btn = document.querySelector("button");
    btn.addEventListener("click", increaseTbl);

    function increaseTbl() {
      let tr = document.createElement("tr");
      table.appendChild(tr);
      for (let j = 0; j < 3; j++) {
        let td = document.createElement("td");
        tr.appendChild(td);
        switch (j) {
          case 0:
            td.textContent = "<button>hoge</button>";
            console.log(i);
            td.setAttribute("id", "cont" + i);
          case 1:
            td.innerHTML = "<button>未完了</button>";
            td.setAttribute("id", "comp" + i);
            break;
          case 2:
            td.innerHTML = "<button>削除</button>";
            td.setAttribute("id", "del" + i);
        }
      }
      i++;
    }
  </script>
</body>
</html>

上記の<script></script>の中のswitchを用いて3つのtdそれぞれに適した内容を入れ込もうとしているのですが、どうしても一つ目のtdにも<button>未完了</button>およびid = "comp" + i の方が入ってします。(2つ目のtdと同じ内容が入ってしまう。)

これの原因は何なのでしょうか。
どなたかご教示お願いします。

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

回答2件

ベストアンサー

breakがないからなのかな

js
1        switch (j) {
2          case 0:
3            td.textContent = "<button>hoge</button>";
4            console.log(i);
5            td.setAttribute("id", "cont" + i);
6            break; // <- ここのbreakがないのがまずいと思う
7          case 1:
8            td.innerHTML = "<button>未完了</button>";
9            td.setAttribute("id", "comp" + i);
10            break;
11          case 2:
12            td.innerHTML = "<button>削除</button>";
13            td.setAttribute("id", "del" + i);
14            break; // <- ここのbreakは、まあ入れておいたほうがいいと思う
15        }

投稿2019/12/04 15:15

rururu3

総合スコア5545

digitalhimiko

2019/12/04 23:20 編集

ありがとうございます！確かにbreak忘れのぽかミスでした。。。

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

js
1        switch (j) {
2          case 0:
3            td.textContent = "<button>hoge</button>";
4            console.log(i);
5            td.setAttribute("id", "cont" + i);
6            break; // ADD
7          case 1:
8            td.innerHTML = "<button>未完了</button>";
9            td.setAttribute("id", "comp" + i);
10            break;
11          case 2:
12            td.innerHTML = "<button>削除</button>";
13            td.setAttribute("id", "del" + i);
14        }