人力検索はてな
モバイル版を表示しています。PC版はこちら
i-mobile

【超至急】jQueryで動的サイトをappendで作成する際、タグが勝手に変換されて困っています。
■ソースコード
<head>
<script>
var list = ['a', 'b', 'c', ...];
$.each(list, function(i, item) {
var max_length = item.length;
var m = '';
if (i == 0) { … [A処理]
m += '<div>';
m += '<div>';
m += '<div>';
}
<****> … [B処理]
if (i == max_length) { … [C処理]
m += '</div>';
m += '</div>';
m += '</div>';
}
$("#test").append(m);
</script>
</head>
<body>
<div id="test"></div>
</body>
■実行結果
<div><div><div> ← [a結果]
<****>
</div></div></div> ← [b結果]
<****>
<****>
;
<****> ← [c結果]ループの最後
■環境
IE10, FireFox22, Chrome28,IOS6(Safari)
■質問
ループの最後に[C処理]が行われてほしいのですが、[C処理]は実際には行われていません。
このためか、[b結果]に[C処理]があるように見えますが、これはブラウザ?が勝手につけています。これは、PCブラウザのデバッガで確認しております。max_lengthも正しいですし、固定値にしても結果は変わりません。
なぜこのようなことが起こるのでしょうか?
実際にはもう少し複雑な分岐がありますが、上記の基本動作が動かないために非常に困っております。
至急で申し訳ありませんが、アドバイスや問題点をご指摘願います!!

●質問者: cochoo
●カテゴリ:ウェブ制作
○ 状態 :終了
└ 回答数 : 3/3件

▽最新の回答へ

1 ● Cherenkov
●34ポイント ベストアンサー

タグのとじ忘れとかがあるのかな。
あと、$("#test").append(m);の時点で#testが存在することが保証されていないのでお決まりの

$(function(){
 document読み込み後に実行したい処理
});

が必要かと。

文字列でHTMLを組むのではなくDOMElement的(?)に組んだほうがわかりやすいと思います。

var m = $('<div>');
m.append($('<div>'));

これで↓が生成される。

<div><div></div></div>

cochooさんのコメント
ご回答ありがとうございます。 ご指摘の処理で期待したとおりに動作することを確認しました! mのスコープには気が付きませんでした!

2 ● Lhankor_Mhy
●33ポイント

括弧の対応がずれているのでこれでエラーなく動くとは思えません。コピペミスってませんか?

正しいコードの構造が分かりませんのでなんとも言えませんが、変数をウォッチしてみるといいかもしれませんね。
たとえば

var list = ['a', 'b', 'c'];

$.each(list, function(i, item) {
 var max_length = item.length;
 console.log(i,max_length)
})
//実行結果(itemは配列ではない)
//0 1
//1 1
//2 1

あるいは

var list = ['a', 'b', 'c'];
var max_length = list.length;

$.each(list, function(i, item) {
 console.log(i,max_length)
})
//実行結果(iはmax_lengthとイコールにならない)
//0 3
//1 3
//2 3

のようなことが起きているのでは。


cochooさんのコメント
ご回答ありがとうございます。 本当ですね。なぜこうなるのでしょうか? でも、ヒントをいただいて助かりました!!感謝です!

3 ● a-kuma3
●33ポイント

実際のコードは、このままではないのかもしれませんが、確実におかしなところが二つあります。

$.each(list, function(i, item) {
 var max_length = item.length;
 var m = ''; // ※(1)
 if (i == 0) {
 [A処理]
 }

 [B処理]

 if (i == max_length) { // ※(2)
 [C処理]
 }
});

※(1)
ループの中にしかスコープがありません。
なので、二回目 (i == 1) のループのときには、A処理でつけた は DIV は消えてしまいます。

※(2)
each のコールバックに渡される第一引数 (i) は、0始まりです。
つまり、最後のループでは i == max_length - 1 です。

修正したのがこちら。

var max_length = item.length;
var m = ''; // ★(1) ループの外に出す
$.each(list, function(i, item) {
 if (i == 0) {
 [A処理]
 }

 [B処理]

 if (i == max_length - 1) { // ★(2) max_length - 1 と比較
 [C処理]
 }
});

さらに、この順番で処理すれば良いのであれば、A処理、C処理は、ループの中に入れる必要はないと思います。

var max_length = item.length;
var m = ''; // ★(1) ループの外に出す

[A処理]

$.each(list, function(i, item) {

 [B処理]

});

[C処理]


実物と見比べてみてください。


a-kuma3さんのコメント
もろ被りなうえに、<span style="color: red;">item</span>.length を見落としてるか…… orz

Lhankor_Mhyさんのコメント
あ、自分もmのスコープについて見落としてましたw

cochooさんのコメント
ご回答ありがとうございます。 mスコープについては気が付きませんでしたが、 なるほど、こうして順を追って確認していただくととてもわかりやすいです。 しかし、思い込みや知らなかったことばかりで質問側として恥ずかしいかぎりです。 ありがとうございました!!
関連質問

●質問をもっと探す●



0.人力検索はてなトップ
8.このページを友達に紹介
9.このページの先頭へ
対応機種一覧
お問い合わせ
ヘルプ/お知らせ
ログイン
無料ユーザー登録
はてなトップ