多重連想配列のfilter検索方法改善

Question

JavaScriptで配列データの検索機能実装を行おうとしております。
この配列は、多重連想配列になっており、1次元目のカラムと2次元目のカラム双方が一致する場合のデータを抽出するものになります。
この、2次元目の検索を行うデータの数が可変になります。
一旦動く処理にはしたのですが、filterしたのち、多重ループという有様でして・・・。
こちら、もう少し簡略化とかできないものなのでしょうか？

コード

//item_listからname先頭文字がsearch_name、かつcategoryのいずれかの情報に合致するものを抽出する.
function searchBunk_agent(item_list,category,search_name) {
    var item_result = [];
    var category_list = category.split(",");
    var search_name_Result = item_list.filter(function(value){
        return value.name.indexOf(search_name,0) == 0;
    });
    for(var i = 0; i < search_name_Result.length; i++){
        var val = search_name_Result[i];
        for(var j = 0; j < category_list.length; j++){
            if(val['category'] == category_list[j]){
                item_result.push('名前：' + val['name'] + 'カテゴリ：' + val['name']);
                break;
            }
        }
    }
}

item_list = [
    {"name":"みかん", "category":"00"},
    {"name":"みしん", "category":"01"},
    {"name":"みどり", "category":"02"}
];

期待値

パターン１
search_name:"み"
category:"00"
結果：”みかん”の要素情報がitem_resultにpushされる

パターン２
search_name:"み"
category:"00,02"
結果：”みかん”と"みどり"の2つの要素情報がitem_resultにpushされる

filter1つのみで検索は完了。その後検索結果でループして、そこではpush実行のみ。
という処理にできれば、作るコード上は検索を何度もするものではなくなり、
見やすくなるのだろうか？と思っていたりします。
ただ、categoryが可変(OR条件が可変)になることへの対策方法が出てこず・・・。

私の方で挙げたコードが妥当な作りでは？や、上記のfilterでcategory可変対応をしても似たような処理をfilterに入れ込むだけだから大して変わらない、などももしありましたら、情報頂ければと思います。

Answer 1

category_listのループをコードから消すならincludesを使えばよいと思いました。

また、item_listをfilterしなくてもよいなら、全件検索したほうがすっきりします。

function searchBunk_agent(item_list,category,search_name) {
    var item_result = [];
    var category_list = category.split(",");
    item_list.forEach(function(val){
        if(val['name'].startsWith(search_name) && category_list.includes(val['category'])){
            item_result.push('名前：' + val['name'] + 'カテゴリ：' + val['category']);
        }
    });
}

Comments

[draq]

forEach 内で item_result へ追加するんじゃなくて、filter して map した結果を直接 item_result へ代入した方がシンプルですね。

const category_list = category.split(",");
const item_result = item_list
.filter(item => item.name.startsWith(search_name) && category_list.includes(item.category))
.map(item => `名前: ${item.name} カテゴリ: ${item.category}`);

[JanTh1989]

ご回答ありがとうございます。
前者、後者共にコードがシンプルになりましたね。
後者はなおシンプルです。
includesもmapも使ったことがなかったので、そういう方法もあるのかと勉強になりました。
後者側でやっていることを整理しつつ、コード改善してみようと思います。