teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

1

一部修正

2016/04/22 12:18

投稿

flat
flat

スコア617

answer CHANGED
@@ -1,6 +1,6 @@
1
1
  まだ解決されていないようなので、**tkturboさんの回答の補足として**コードの全体像を記載します。
2
2
 
3
- ちなみに現状では`add()`**に渡された**`indexno`**がそのまま`innerHTML`に含まれているのでエスケープした方が良い**とか、もっと言えば`innerHTML`**で追加している要素をできれば**`createElement`**と**`appendChild`**と**`createTextNode`**などを利用して生成・組み立てを行ってその要素を`appendChild`で追加した方が良い**とか、**同じボタンを押すたびに一意の識別子であるIDを持つ要素が重複して生成されてしまう**などの問題がありますが、動作だけで言えばこれで目的を達成できるはずです。
3
+ ちなみに現状では`add()`**に渡された**`indexno`**がそのまま`innerHTML`に含まれているので出力をエスケープ処理などで無害化した方が良い**とか、もっと言えば`innerHTML`**で追加している要素をできれば**`createElement`**と**`appendChild`**と**`createTextNode`**などを利用して生成・組み立てを行ってその要素を`appendChild`で追加した方が良い**とか、**同じボタンを押すたびに一意の識別子であるIDを持つ要素が重複して生成されてしまう**などの問題がありますが、動作だけで言えばこれで目的を達成できるはずです。
4
4
 
5
5
  なお、最初にもその旨を書いていますが、これは一部を除いて**ほぼすべてtkturboさんの回答にある内容**です。
6
6
  ```HTML