質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.51%
JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

Q&A

解決済

3回答

5217閲覧

チェックボックスを.htmlで動的に変更し、配列を取得すると値が重複するのですが。。。

noripi

総合スコア34

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

0グッド

0クリップ

投稿2016/10/17 07:26

編集2016/10/18 06:51

複数のチェックボックスを任意の順番に並び替えしたのち、チェックがついているものの値を一括取得したいのですが、チェックがついた状態のものの順番を変更すると一括取得した値が重複してしまいます。
対処方法わかる方おりませんでしょうか。

《追記》
チェックボックスのチェックがない状態で順序を変更後、チェックボックスにチェックを入れる場合は問題ないようです。
チェックボックスにチェックが入ったものを順序変更した場合のみ、チェックが入った値が1つ増えます(重複)します。その後はいくら順序を変更しても重複した値を増えません。

javascript

1 $(function(){ 2 //表示順序を下に変更 3 $("#custom-display-post-widget #item li").on("click", ".down", function() { 4 var now_index = $(this).closest("li").index() + 1; 5 var next_index = $(this).closest("li").next().index() + 1; 6 var now_obj = $(this).closest("li").html(); 7 var next_obj = $(this).closest("li").next().html(); 8 $("#custom-display-post-widget #display-setting #item ul li:nth-child("+next_index+")").html(now_obj); 9 $("#custom-display-post-widget #display-setting #item ul li:nth-child("+now_index+")").html(next_obj); 10 $("#custom-display-post-widget #item li input").removeAttr('checked');//追記 11 checkVal(); 12 }); 13 14 //表示順序を上に変更 15 $("#custom-display-post-widget #item li").on("click", ".up", function() { 16 var now_index = $(this).closest("li").index() + 1; 17 var prev_index = $(this).closest("li").prev().index() + 1; 18 var now_obj = $(this).closest("li").html(); 19 var prev_obj = $(this).closest("li").prev().html(); 20 $("#custom-display-post-widget #display-setting #item ul li:nth-child("+prev_index+")").html(now_obj); 21 $("#custom-display-post-widget #display-setting #item ul li:nth-child("+now_index+")").html(prev_obj); 22 $("#custom-display-post-widget #item li input").removeAttr('checked');//追記 23 checkVal(); 24 }); 25 26 // 27 28 //表示設定を取得 29 $("#custom-display-post-widget #item li input").live("change", function() { 30 31 //チェックされていた場合 32 if($(this).prop('checked')){ 33 $(this).attr('checked','checked'); 34 } 35 36 //チェックされていない場合 37 if(!$(this).prop('checked')){ 38 $(this).removeAttr('checked'); 39 } 40 41 checkVal(); 42 }); 43 44 function checkVal(){ 45 var checkBox=[]; 46 // チェックされた値を取得 47 $("#custom-display-post-widget #item li input:checked").each(function(){ 48 checkBox.push($(this).val()); 49 }); 50 //取得した値を input text にセット 51 $("#custom-display-post-widget input.display-post-set").val(checkBox); 52 } 53 });

HTML

1 <div id="display-setting"> 2 <p> 3 <input id="id_display_order" class="display-post-set" name="display_order" type="text" value="" size="30"> 4 </p> 5 <div id="item"> 6 <span class="title">アイテム:</span> 7 <ul> 8 <li><input class="date display-item" type="checkbox" name="date" value="date">date<span class="order"><span class="up">▲</span><span class="down">▼</span></span></li> 9 <li><input class="thumbnail display-item" type="checkbox" name="thumbnail" value="thumbnail">thumbnail<span class="order"><span class="up">▲</span><span class="down">▼</span></span></li> 10 <li><input class="title display-item" type="checkbox" name="title" value="title">title<span class="order"><span class="up">▲</span><span class="down">▼</span></span></li> 11 <li><input class="excerpt display-item" type="checkbox" name="excerpt" value="excerpt">excerpt<span class="order"><span class="up">▲</span><span class="down">▼</span></span></li> 12 <li><input class="category display-item" type="checkbox" name="category" value="category">category<span class="order"><span class="up">▲</span><span class="down">▼</span></span></li> 13 <li><input class="link display-item" type="checkbox" name="link" value="link">link<span class="order"><span class="up">▲</span><span class="down">▼</span></span></li> 14 </ul> 15 </div> 16 </div>

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答3

0

val関数内でcheck配列を初期化してないのが原因ではないでしょうか?

投稿2016/10/17 14:50

attercop

総合スコア246

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

noripi

2016/10/18 00:49

ご回答ありがとうございます。 早速、配列の初期化と同時に念のため関数名と配列名を変更してみましたが、解決しませんでした。。。 何か処理のタイミングが原因なんでしょうか。。。
attercop

2016/10/18 02:28

配列を初期化する位置がおかしいです。 eachの中で初期化すると、配列の初期化と配列への追加を繰り返すことになりますよね。
guest

0

初期化位置がおかしいです。

JavaScript

1function checkVal(){ 2 var checkBox = []; //ADD 3 // チェックされた値を取得 4 $( "#custom-display-post-widget #item li input:checked" ).each( function() { 5 // var checkBox = []; // delete 6 checkBox.push( $( this ).val() ); 7 } ); 8 //取得した値を input text にセット 9 $( "#custom-display-post-widget input.display-post-set" ).val( checkBox ); 10} 11```**動くサンプル:**[https://jsfiddle.net/crtbe6v1/1/](https://jsfiddle.net/crtbe6v1/1/) 12 13--- 14 15また、jQuery().liveはver1.9から廃止されていますので、jQuery().onでコーディングするのが良いと思います。 16 17【jQueryのonをliveの代わりに使う場合の書き方 - 無理しない感じ】 18[http://hogesuke.hateblo.jp/entry/2014/02/04/101641](http://hogesuke.hateblo.jp/entry/2014/02/04/101641) 19 20【jQueryのclickとbindとliveとdelegateとonの違い - Qiita】 21[http://qiita.com/smzk/items/5eed5a90c4b32ca8b23a](http://qiita.com/smzk/items/5eed5a90c4b32ca8b23a)

投稿2016/10/18 02:33

kei344

総合スコア69357

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

noripi

2016/10/18 06:45

ご回答ありがとうございます。 初期化位置ですが、こちらは私の投稿追記ミスでした。申し訳ありません。 liveの廃止は知りませんでした。。。 教えて頂いたサイトを参考にonへ変更しましたが状況は変わりませんでした。 orz
kei344

2016/10/18 06:52

「動くサンプル」は御覧になりましたか?その挙動で問題があれば、問題再現の手順を書いてください。
noripi

2016/10/18 07:03

すみません。「動くサンプル」上の挙動で問題ないです。 ですが、本番環境では問題が解決できませんでした。 もう少し調査してみます。
guest

0

自己解決

理由はよくわかりませんが、JavaScriptコード内の「//追記」部分により問題が解消されました。
他のスクリプトとの競合が原因?

投稿2016/10/18 06:55

編集2016/10/18 07:09
noripi

総合スコア34

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

attercop

2016/10/19 04:25

今更ですがattr、removeAttrでchecked属性を操作いないと正常動作しないとなると、 DOMが汚染されているのかもしれません。 通常はpropでcheckedを操作した場合、属性値も書き換わっているはずなのでattrでの操作は必要ないはずです。 エレメントの移動はhtmlを書き換えるのではなくinsertAfter、insertBefore等を使用されるとよいかもしれません。 http://cly7796.net/wp/javascript/to-insert-or-move-by-operating-the-dom/
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.51%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問