回答編集履歴

3

jQuery#html 関連リンクを追加

2017/01/17 10:27

投稿

think49
think49

スコア18166

test CHANGED
@@ -16,11 +16,15 @@
16
16
 
17
17
 
18
18
 
19
- 説明すると長くなるので下記にまとめました。
19
+ 説明すると長くなるのでQiitaにまとめました。
20
20
 
21
21
 
22
22
 
23
23
  - [jQuery.prototype.html の仕様を調べてみた - Qiita](http://qiita.com/think49/items/affdd59c1791e2a04b6c)
24
+
25
+ - [.html() | jQuery API Documentation](http://api.jquery.com/html/)
26
+
27
+ - [jquery.js v3.1.1 · jquery/jquery-ui](https://github.com/jquery/jquery-ui/blob/master/external/jquery-3.1.1/jquery.js#L5978)
24
28
 
25
29
 
26
30
 

2

jQuery#html の仕様を追記

2017/01/17 10:27

投稿

think49
think49

スコア18166

test CHANGED
@@ -9,6 +9,18 @@
9
9
  - `jQuery#html` は関数です。プロパティではありません。従って、`div.html = String(key * 10);` は期待通りに動作しません。
10
10
 
11
11
  - `jQuery#html` は DocumentFragment を認識しません。DOM API の `appendChild` を使ってください。⇒これは誤りでした。`jQuery#html` はDOMノードも扱えるようです(詳細は後述)。
12
+
13
+
14
+
15
+ ### jQuery#html の仕様
16
+
17
+
18
+
19
+ 説明すると長くなるので下記にまとめました。
20
+
21
+
22
+
23
+ - [jQuery.prototype.html の仕様を調べてみた - Qiita](http://qiita.com/think49/items/affdd59c1791e2a04b6c)
12
24
 
13
25
 
14
26
 
@@ -140,4 +152,14 @@
140
152
 
141
153
 
142
154
 
155
+ ### 更新履歴
156
+
157
+
158
+
159
+ - 2017/01/17 17:06 修正コードを追記
160
+
161
+ - 2017/01/17 19:25 jQuery#html の仕様を追記
162
+
163
+
164
+
143
165
  Re: Meganezaru さん

1

修正コードを追記

2017/01/17 10:25

投稿

think49
think49

スコア18166

test CHANGED
@@ -1,10 +1,142 @@
1
+ ### 雑感
2
+
3
+
4
+
1
5
  DOM API と jQuery API を混同して覚えているような印象を受けます。
2
6
 
3
7
 
4
8
 
5
9
  - `jQuery#html` は関数です。プロパティではありません。従って、`div.html = String(key * 10);` は期待通りに動作しません。
6
10
 
7
- - `jQuery#html` は DocumentFragment を認識しません。DOM API の `appendChild` を使ってください。
11
+ - `jQuery#html` は DocumentFragment を認識しません。DOM API の `appendChild` を使ってください。⇒これは誤りでした。`jQuery#html` はDOMノードも扱えるようです(詳細は後述)。
12
+
13
+
14
+
15
+ ### 修正コード
16
+
17
+
18
+
19
+ 質問文のコードの体裁を残したコードがこちら。
20
+
21
+
22
+
23
+ ```HTML
24
+
25
+ <div id="wrapper">
26
+
27
+ <div id="1">-</div>
28
+
29
+ <div id="2">-</div>
30
+
31
+ <div id="3">-</div>
32
+
33
+ </div>
34
+
35
+ <script>
36
+
37
+ 'use strict';
38
+
39
+ jQuery(function (jQuery) {
40
+
41
+ var dict = {"1": 1,"2": 2,"3": 3},
42
+
43
+ documentFragment = document.createDocumentFragment();
44
+
45
+
46
+
47
+ Object.keys(dict).forEach(function (key) {
48
+
49
+ var value = key * 10,
50
+
51
+ div = jQuery("#" + key).clone();
52
+
53
+
54
+
55
+ div.text(value);
56
+
57
+ dict[key] = value;
58
+
59
+ documentFragment.appendChild(div[0]);
60
+
61
+ });
62
+
63
+ jQuery('#wrapper').html(documentFragment);
64
+
65
+ });
66
+
67
+ </script>
68
+
69
+ ```
70
+
71
+
72
+
73
+ 上記コードには下記問題があります。
74
+
75
+
76
+
77
+ - id属性値が数字から始まっている(HTML5 では許されるが、セマンティック上好ましくない)
78
+
79
+ - 変数 `dict` が再構成されている(動的に生成するなら初期値が不要)
80
+
81
+ - 生成した DocumentFragment を jQuery オブジェクトでラップしているので変換コストが勿体ない
82
+
83
+ - 要素ノードをごっそり入れ替えているので当該要素ノードにイベント定義されていたら全てはがされてしまう
84
+
85
+
86
+
87
+ 問題を修正したコードがこちら。
88
+
89
+
90
+
91
+ ```HTML
92
+
93
+ <div id="wrapper">
94
+
95
+ <div id="sample-1">-</div>
96
+
97
+ <div id="sample-2">-</div>
98
+
99
+ <div id="sample-3">-</div>
100
+
101
+ </div>
102
+
103
+ <script>
104
+
105
+ 'use strict';
106
+
107
+ jQuery(function () {
108
+
109
+ var dict = Object.create(null);
110
+
111
+
112
+
113
+ jQuery('#wrapper>div').text(function (index) {
114
+
115
+ var id = this.id,
116
+
117
+ value = id.replace(/^[a-z-]+/, '') * 10;
118
+
119
+
120
+
121
+ dict[this.id] = value;
122
+
123
+ return value;
124
+
125
+ });
126
+
127
+
128
+
129
+ console.log(JSON.stringify(dict)); // {"sample-1":10,"sample-2":20,"sample-3":30}
130
+
131
+ });
132
+
133
+ </script>
134
+
135
+ ```
136
+
137
+
138
+
139
+ 更に `Object.defineProperty` で `dict` に getter/setter を与えてやると良い感じになりそうです。
8
140
 
9
141
 
10
142