teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

3

修正

2016/03/06 11:25

投稿

yamato_hikawa
yamato_hikawa

スコア2092

answer CHANGED
@@ -108,7 +108,7 @@
108
108
  src = data[0],
109
109
  name = data[1],
110
110
  $li = $('<li />'),
111
- $img = $('<img />').attr('src', src);
111
+ $img = $('<img />').attr('src', src),
112
112
  $text = $('<p />').text(name);
113
113
 
114
114
  $li.append($img, $text);

2

追記

2016/03/06 11:25

投稿

yamato_hikawa
yamato_hikawa

スコア2092

answer CHANGED
@@ -141,4 +141,19 @@
141
141
  $(function() {
142
142
  new filtering();
143
143
  });
144
+ ```
145
+
146
+ 追記ですが、groupVal, placeVal, genreValを求める部分を共通の関数で処理するよう
147
+ 下記の形に書き換えることも出来ます。
148
+
149
+ ```js
150
+ getVal: function(key, value){
151
+ return DB[key][value] || [];
152
+ },
153
+ changeList: function() {
154
+ var groupVal = this.getVal('group', this.$group.val()),
155
+ placeVal = this.getVal('place', this.$place.val()),
156
+ genreVal = this.getVal('genre', this.$genre.val()),
157
+ // 以下同様
158
+
144
159
  ```

1

修正

2016/02/28 00:28

投稿

yamato_hikawa
yamato_hikawa

スコア2092

answer CHANGED
@@ -108,11 +108,9 @@
108
108
  src = data[0],
109
109
  name = data[1],
110
110
  $li = $('<li />'),
111
- $img = $('<img />'),
111
+ $img = $('<img />').attr('src', src);
112
- $text = $('<p />');
112
+ $text = $('<p />').text(name);
113
113
 
114
- $img.attr('src', src);
115
- $text.text(name);
116
114
  $li.append($img, $text);
117
115
 
118
116
  return $li;
@@ -123,22 +121,19 @@
123
121
  this.$searchcontainer.append($list);
124
122
  },
125
123
  changeList: function() {
126
- this.$searchcontainer.empty();
127
-
128
124
  var groupVal = DB.group[this.$group.val()] || [],
129
125
  placeVal = DB.place[this.$place.val()] || [],
130
126
  genreVal = DB.genre[this.$genre.val()] || [],
131
- result;
127
+ result = $.unique(groupVal.concat(placeVal, genreVal));
132
128
 
133
129
  console.log(groupVal, placeVal, genreVal);
134
-
135
- result = $.unique(groupVal.concat(placeVal, genreVal));
136
130
  console.log('result', result);
137
131
 
138
132
  if(result.length === 0){
139
133
  result = DB.all;
140
134
  }
135
+
141
-
136
+ this.$searchcontainer.empty();
142
137
  this.showResult(result);
143
138
  }
144
139
  };