回答編集履歴
1
edit
answer
CHANGED
|
@@ -7,4 +7,16 @@
|
|
|
7
7
|
|
|
8
8
|
itemが持ってるのがDOMで、todoItemsが持ってるのがJSONオブジェクトなので当然かもしれません。
|
|
9
9
|
data属性にユニークな情報（indexとかIDとなるもの）を持たせて照合させたほうが良いように思います。
|
|
10
|
-
場合により、LocalStorageに持たせるJSON文字列も実装上で扱いやすいように変更したほうがいいかもしれません。
|
|
10
|
+
場合により、LocalStorageに持たせるJSON文字列も実装上で扱いやすいように変更したほうがいいかもしれません。
|
|
11
|
+
|
|
12
|
+
別案：
|
|
13
|
+
今の実装だと
|
|
14
|
+
1.DOMに反映
|
|
15
|
+
2.ローカルストレージに反映
|
|
16
|
+
|
|
17
|
+
と二重管理になっていて、バグが起きやすい状況になってます。
|
|
18
|
+
それなら
|
|
19
|
+
1.ローカルストレージに反映
|
|
20
|
+
2.DOM全削除して反映したローカルストレージからDOMを構築
|
|
21
|
+
|
|
22
|
+
としたほうが実装もシンプルで短くなるんじゃないでしょうか。
|