回答編集履歴

修正

2016/03/06 11:25

投稿

スコア2092

test CHANGED Viewed

@@ -218,7 +218,7 @@
         $li = $('<li />'),
-        $img = $('<img />').attr('src', src);
+        $img = $('<img />').attr('src', src),
         $text = $('<p />').text(name);

追記

2016/03/06 11:25

投稿

スコア2092

test CHANGED Viewed

@@ -285,3 +285,33 @@
 });
 ```
+追記ですが、groupVal, placeVal, genreValを求める部分を共通の関数で処理するよう
+下記の形に書き換えることも出来ます。
+```js
+  getVal: function(key, value){
+    return DB[key][value] || [];
+  },
+  changeList: function() {
+    var groupVal = this.getVal('group', this.$group.val()),
+      placeVal = this.getVal('place', this.$place.val()),
+      genreVal = this.getVal('genre', this.$genre.val()),
+// 以下同様
+```

修正

2016/02/28 00:28

投稿

スコア2092

test CHANGED Viewed

@@ -218,15 +218,11 @@
         $li = $('<li />'),
-        $img = $('<img />'),
+        $img = $('<img />').attr('src', src);
-        $text = $('<p />');
+        $text = $('<p />').text(name);
-      $img.attr('src', src);
-      $text.text(name);
       $li.append($img, $text);
@@ -248,37 +244,31 @@
   changeList: function() {
-    this.$searchcontainer.empty();
     var groupVal = DB.group[this.$group.val()] || [],
       placeVal = DB.place[this.$place.val()] || [],
       genreVal = DB.genre[this.$genre.val()] || [],
-      result;
+      result = $.unique(groupVal.concat(placeVal, genreVal));
     console.log(groupVal, placeVal, genreVal);
+    console.log('result', result);
-    result = $.unique(groupVal.concat(placeVal, genreVal));
-    console.log('result', result);
     if(result.length === 0){
       result = DB.all;
     }
+    this.$searchcontainer.empty();
     this.showResult(result);