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

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

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

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

jQuery

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

Q&A

1回答

197閲覧

jQueryでフィルタリングのコードを整理したい

ysfree14

総合スコア19

JavaScript

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

jQuery

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

0グッド

1クリップ

投稿2019/11/14 07:54

編集2019/11/16 05:52

jsonを読み込みコードを生成、jQueryでフィルタリング表示するコンテンツをページの2か所に設置しています。
フィルタリングに関しては初期表示は8個、もっと見るをクリックでさらに8個ずつ追加表示というようにしています。

1つ目のコードをそのまま複製するような形で下記のように複製してみたのですが、もっとスマートにjqueryを記述するにはどのようにすればいいのでしょうか?
よろしくお願い致します。

jquery

1$(function(){ 2 3 //1つ目 4 //変数設定 5 var $boxes = $("#boxes"); 6 var alldata = []; //ここはすべてのデータを保持しておく配列 7 var filterdata = []; //フィルターを掛けたデータを保持する配列 8 9 //最初に読み込んだ時に発生する関数 10 $.getJSON('1つ目のjsonのパス',init); 11 12 function init(data) { 13 //全データを保持しておく 14 alldata = data; 15 //とりあえず全データを代入しておく 16 filterdata = alldata; 17 //データを表示させる 18 display(0); 19 } 20 21 const MAX_DISPLAY_COUNT = 8; 22 23 //画像表示させるための関数 24 function display(startIndex) { 25 //$boxes.empty(); 26 $('.original-tab-more-btn').show(); 27 for (var i = startIndex; i < startIndex + MAX_DISPLAY_COUNT; i++) { 28 if(!filterdata[i]){ 29 $('.original-tab-more-btn').hide(); 30 return; 31 } 32 $('<div class="original-tab-elm matchHeight"><a href="' + filterdata[i].url + '" target="_top" class="block"><div class="product-img img-col1"><img src="' + filterdata[i].img + '" alt="" id="product-img"></div><div class="label-wrap"><p class="img-label' + filterdata[i].labelName + '">' + filterdata[i].dateText + '</p><p class="img-label' + filterdata[i].labelName2 + '">' + filterdata[i].dateText2 + '</p></div><p class="price-text">' + filterdata[i].price + '</p></a></div>').appendTo($boxes); 33 } 34 } 35 36 let currentDisplayCount = 0; 37 38 //ラジオボタンが変わった時にフィルタリングする関数 39 $('.original-filter input:radio').change(function(){ 40 41 $boxes.empty(); 42 43 //チェックボタンの状況によってフィルタリングする 44 var cate=$(this).val(); 45 //フィルターの配列をいったん空っぽにする 46 filterdata=[]; 47 48 if(cate==="all"){//allが選択されたら全データを保持 49 filterdata=alldata; 50 51 }else{ 52 filterdata=$.grep(alldata,function(n){ 53 return n.category===cate; 54 }); 55 } 56 57 currentDisplayCount = 0; 58 display(currentDisplayCount); 59 60 }); 61 62 //もっと見るで要素を追加させたい 63 $('.original-tab-more-btn').click(function () { 64 currentDisplayCount += MAX_DISPLAY_COUNT; 65 display(currentDisplayCount); 66 }); 67}); 68$(function(){ 69 70     //2つ目 71     //変数設定 72 var $boxes2 = $("#select-boxes"); 73 var alldata2 = []; //ここはすべてのデータを保持しておく配列 74 var filterdata2 = []; //フィルターを掛けたデータを保持する配列 75 76 //最初に読み込んだ時に発生する関数 77 $.getJSON('2つ目のjsonのパス',init); 78 79 function init(data2) { 80 //全データを保持しておく 81 alldata2 = data2; 82 //とりあえず全データを代入しておく 83 filterdata2 = alldata2; 84 //データを表示させる 85 display2(0); 86 } 87 88 const MAX_DISPLAY_COUNT2 = 8; 89 90 //画像表示させるための関数 91 function display2(startIndex2) { 92 //$boxes.empty(); 93 $('.select-tab-more-btn').show(); 94 for (var i = startIndex2; i < startIndex2 + MAX_DISPLAY_COUNT2; i++) { 95 if(!filterdata2[i]){ 96 $('.select-tab-more-btn').hide(); 97 return; 98 } 99 $('<div class="select-tab-elm matchHeight"><a href="' + filterdata2[i].url + '" target="_top" class="block"><div class="product-img img-col1"><img src="' + filterdata2[i].img + '" alt="" id="product-img"></div><div class="label-wrap"><p class="img-label' + filterdata2[i].labelName + '">' + filterdata2[i].dateText + '</p><p class="img-label' + filterdata2[i].labelName2 + '">' + filterdata2[i].dateText2 + '</p></div><p class="price-text">' + filterdata2[i].price + '</p></a></div>').appendTo($boxes2); 100 } 101 } 102 103 let currentDisplayCount2 = 0; 104 105 //ラジオボタンが変わった時にフィルタリングする関数 106 $('.select-filter input:radio').change(function(){ 107 108 $boxes2.empty(); 109 110 //チェックボタンの状況によってフィルタリングする 111 var cate2=$(this).val(); 112 113 //フィルターの配列をいったん空っぽにする 114 filterdata2=[]; 115 116 if(cate2==="all"){//allが選択されたら全データを保持 117 filterdata2=alldata2; 118 119 }else{ 120 filterdata2=$.grep(alldata2,function(n){ 121 return n.category===cate2; 122 }); 123 } 124 125 currentDisplayCount2 = 0; 126 display2(currentDisplayCount2); 127 128 }); 129 130 //もっと見るで要素を追加させたい 131 $('.select-tab-more-btn').click(function () { 132 currentDisplayCount2 += MAX_DISPLAY_COUNT2; 133 display2(currentDisplayCount2); 134 }); 135}); 136 137

json

1[ 2 3{ 4"category": "new", 5"url": "", 6"img": "", 7"labelName": "", 8"dateText": "", 9"labelName2": "", 10"dateText2": "", 11"price": "" 12}, 13 14{ 15"category": "re", 16"url": "", 17"img": "", 18"labelName": "", 19"dateText": "", 20"labelName2": "", 21"dateText2": "", 22"price": "" 23}, 24 25]

json

1[ 2 3{ 4"category": "new-s", 5"url": "", 6"img": "", 7"labelName": "", 8"dateText": "", 9"labelName2": "", 10"dateText2": "", 11"price": "" 12}, 13 14{ 15"category": "re-s", 16"url": "", 17"img": "", 18"labelName": "", 19"dateText": "", 20"labelName2": "", 21"dateText2": "", 22"price": "" 23}, 24 25]

html

1<!--1つ目--> 2<form> 3 <label> 4 <input type="radio" name="list" value="all" checked="checked"> 5 <span>All</span> 6 </label> 7 <label> 8 <input type="radio" name="list" value="new"> 9 <span>new</span> 10 </label> 11 <label> 12 <input type="radio" name="list" value="re"> 13 <span>re</span> 14 </label> 15</form> 16 17<div id="boxes" class="boxes cf"></div> 18<div class="original-tab-more-btn">もっと見る</div> 19 20<!--2つ目--> 21<form> 22 <label class="btn-on"> 23 <input type="radio" name="list" value="all" checked="checked"> 24 <span>All</span> 25 </label> 26 <label> 27 <input type="radio" name="list" value="new-s"> 28 <span>new</span> 29 </label> 30 <label> 31 <input type="radio" name="list" value="re-s"> 32 <span>re</span> 33 </label> 34</form> 35 36<div id="select-boxes" class="boxes cf"></div> 37<div class="select-tab-more-btn">もっと見る</div> 38

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

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

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

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

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

yambejp

2019/11/14 07:56

HTMLとjsonデータを例示して、想定する結果を説明してください
ysfree14

2019/11/14 09:56

追記しました、よろしくお願い致します。
guest

回答1

0

init関数が2度設置されたりしてちょっと整合性が取れていないと思います
$.getJSON()が2度実行されているのは、最初が1つ目のJSON、次のが2つ目のJSONを呼ぶのでしょうか?

sample

javascript

1<script> 2$(function(){ 3 a(); 4 function a(){ 5 console.log(1); 6 } 7 a(); 8 function a(){ 9 console.log(2); 10 } 11 a(); 12}); 13</script>

※競合した名前の関数を設定しても、上書きされてあとの方しか実行されません

投稿2019/11/14 10:13

編集2019/11/15 08:43
yambejp

総合スコア114814

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

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

ysfree14

2019/11/15 08:36

回答ありがとうございます。 はい、最初が1つ目のjsonで次が2つ目のjsonを読み込んでいます。 通常は複数のinit関数は使用しないのでしょうか?
yambejp

2019/11/15 08:44

sampleつけておきました
ysfree14

2019/11/16 05:55

回答ありがとうございます。 1つ目・2つ目の処理それぞれを$(function(){ ~ });で囲っていると正常に動作します。 これで一応は問題ないでしょうか?
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問