回答編集履歴
3
jQuery#html 関連リンクを追加
answer
CHANGED
|
@@ -7,9 +7,11 @@
|
|
|
7
7
|
|
|
8
8
|
### jQuery#html の仕様
|
|
9
9
|
|
|
10
|
-
説明すると長くなるので
|
|
10
|
+
説明すると長くなるのでQiitaにまとめました。
|
|
11
11
|
|
|
12
12
|
- [jQuery.prototype.html の仕様を調べてみた - Qiita](http://qiita.com/think49/items/affdd59c1791e2a04b6c)
|
|
13
|
+
- [.html() | jQuery API Documentation](http://api.jquery.com/html/)
|
|
14
|
+
- [jquery.js v3.1.1 · jquery/jquery-ui](https://github.com/jquery/jquery-ui/blob/master/external/jquery-3.1.1/jquery.js#L5978)
|
|
13
15
|
|
|
14
16
|
### 修正コード
|
|
15
17
|
|
2
jQuery#html の仕様を追記
answer
CHANGED
|
@@ -5,6 +5,12 @@
|
|
|
5
5
|
- `jQuery#html` は関数です。プロパティではありません。従って、`div.html = String(key * 10);` は期待通りに動作しません。
|
|
6
6
|
- `jQuery#html` は DocumentFragment を認識しません。DOM API の `appendChild` を使ってください。⇒これは誤りでした。`jQuery#html` はDOMノードも扱えるようです(詳細は後述)。
|
|
7
7
|
|
|
8
|
+
### jQuery#html の仕様
|
|
9
|
+
|
|
10
|
+
説明すると長くなるので下記にまとめました。
|
|
11
|
+
|
|
12
|
+
- [jQuery.prototype.html の仕様を調べてみた - Qiita](http://qiita.com/think49/items/affdd59c1791e2a04b6c)
|
|
13
|
+
|
|
8
14
|
### 修正コード
|
|
9
15
|
|
|
10
16
|
質問文のコードの体裁を残したコードがこちら。
|
|
@@ -69,4 +75,9 @@
|
|
|
69
75
|
|
|
70
76
|
更に `Object.defineProperty` で `dict` に getter/setter を与えてやると良い感じになりそうです。
|
|
71
77
|
|
|
78
|
+
### 更新履歴
|
|
79
|
+
|
|
80
|
+
- 2017/01/17 17:06 修正コードを追記
|
|
81
|
+
- 2017/01/17 19:25 jQuery#html の仕様を追記
|
|
82
|
+
|
|
72
83
|
Re: Meganezaru さん
|
1
修正コードを追記
answer
CHANGED
|
@@ -1,6 +1,72 @@
|
|
|
1
|
+
### 雑感
|
|
2
|
+
|
|
1
3
|
DOM API と jQuery API を混同して覚えているような印象を受けます。
|
|
2
4
|
|
|
3
5
|
- `jQuery#html` は関数です。プロパティではありません。従って、`div.html = String(key * 10);` は期待通りに動作しません。
|
|
4
|
-
- `jQuery#html` は DocumentFragment を認識しません。DOM API の `appendChild` を使ってください。
|
|
6
|
+
- `jQuery#html` は DocumentFragment を認識しません。DOM API の `appendChild` を使ってください。⇒これは誤りでした。`jQuery#html` はDOMノードも扱えるようです(詳細は後述)。
|
|
5
7
|
|
|
8
|
+
### 修正コード
|
|
9
|
+
|
|
10
|
+
質問文のコードの体裁を残したコードがこちら。
|
|
11
|
+
|
|
12
|
+
```HTML
|
|
13
|
+
<div id="wrapper">
|
|
14
|
+
<div id="1">-</div>
|
|
15
|
+
<div id="2">-</div>
|
|
16
|
+
<div id="3">-</div>
|
|
17
|
+
</div>
|
|
18
|
+
<script>
|
|
19
|
+
'use strict';
|
|
20
|
+
jQuery(function (jQuery) {
|
|
21
|
+
var dict = {"1": 1,"2": 2,"3": 3},
|
|
22
|
+
documentFragment = document.createDocumentFragment();
|
|
23
|
+
|
|
24
|
+
Object.keys(dict).forEach(function (key) {
|
|
25
|
+
var value = key * 10,
|
|
26
|
+
div = jQuery("#" + key).clone();
|
|
27
|
+
|
|
28
|
+
div.text(value);
|
|
29
|
+
dict[key] = value;
|
|
30
|
+
documentFragment.appendChild(div[0]);
|
|
31
|
+
});
|
|
32
|
+
jQuery('#wrapper').html(documentFragment);
|
|
33
|
+
});
|
|
34
|
+
</script>
|
|
35
|
+
```
|
|
36
|
+
|
|
37
|
+
上記コードには下記問題があります。
|
|
38
|
+
|
|
39
|
+
- id属性値が数字から始まっている(HTML5 では許されるが、セマンティック上好ましくない)
|
|
40
|
+
- 変数 `dict` が再構成されている(動的に生成するなら初期値が不要)
|
|
41
|
+
- 生成した DocumentFragment を jQuery オブジェクトでラップしているので変換コストが勿体ない
|
|
42
|
+
- 要素ノードをごっそり入れ替えているので当該要素ノードにイベント定義されていたら全てはがされてしまう
|
|
43
|
+
|
|
44
|
+
問題を修正したコードがこちら。
|
|
45
|
+
|
|
46
|
+
```HTML
|
|
47
|
+
<div id="wrapper">
|
|
48
|
+
<div id="sample-1">-</div>
|
|
49
|
+
<div id="sample-2">-</div>
|
|
50
|
+
<div id="sample-3">-</div>
|
|
51
|
+
</div>
|
|
52
|
+
<script>
|
|
53
|
+
'use strict';
|
|
54
|
+
jQuery(function () {
|
|
55
|
+
var dict = Object.create(null);
|
|
56
|
+
|
|
57
|
+
jQuery('#wrapper>div').text(function (index) {
|
|
58
|
+
var id = this.id,
|
|
59
|
+
value = id.replace(/^[a-z-]+/, '') * 10;
|
|
60
|
+
|
|
61
|
+
dict[this.id] = value;
|
|
62
|
+
return value;
|
|
63
|
+
});
|
|
64
|
+
|
|
65
|
+
console.log(JSON.stringify(dict)); // {"sample-1":10,"sample-2":20,"sample-3":30}
|
|
66
|
+
});
|
|
67
|
+
</script>
|
|
68
|
+
```
|
|
69
|
+
|
|
70
|
+
更に `Object.defineProperty` で `dict` に getter/setter を与えてやると良い感じになりそうです。
|
|
71
|
+
|
|
6
72
|
Re: Meganezaru さん
|