オブジェクト内で条件に一致する値を取得する方法

Question

下記のデータ内で、planList.plans.idが、指定した値(5)に一致する「score」の値のみ取得したいと考えているのですが、自分の実装方法だとうまく取得出来ません。

const planList = [
  {
    id: '1',
    name: 'aaaa',
    plans: [
        {
          id: '1',
          name: 'test1',
          score: '150',
        },
        {
          id: '2',
          name: 'test2',
          score: '220',
        },
        {
          id: '3',
          name: 'test3',
          score: '300',
        }
     ]
  },
  {
    id: '2',
    name: 'bbbb',
    plans: []
  },
  {
    id: '3',
    name: 'ccccc',
    plans: [
      {
        id: '4',
        name: 'test5',
        score: '100',
      },
      {
        id: '5',
        name: 'test6',
        score: '200',
      },
      {
        id: '6',
        name: 'test7',
        score: '900',
      }
    ]
  },
]

自分が実装したコードは下記になります

  planList.forEach(e1 => {
      const result = e1.plans.find(e2 => {
        if (e2.id === '5') {
          return e2.score
        } else {
          return false
        }

      })
      if(result) {
        const planPoint = result
      }
    })
    console.log('point' , planPoint)

こちら、どなたか解決方法につきまして、ご助言頂けましたら幸いです

Answer 1

ポイントはid=5となるオブジェクトがユニークとは限らないことです。
こんな感じにするとよいかも

const getAllValuesWithKeys=(obj,keys)=>{
  let ret=[];
  Object.entries(obj).forEach(ent=>{
    ret=ret.concat(
      ent[1] instanceof Object?
      getAllValuesWithKeys(ent[1],[ent[0]].concat(keys?keys:[])):
      [ent.concat(keys?[keys]:[[]])]
      );
  });
  return ret;
};
const tmp=getAllValuesWithKeys(planList);
const res=tmp.filter(x=>x[0]=="score"&&tmp.filter(x=>x[0]=="id" && x[1]=="5").map(x=>JSON.stringify(x[2])).includes(JSON.stringify(x[2]))).map(x=>x[1]);
console.log(res);

Comments

[ruuuu]

ご回答ありがとうございます。
> ポイントはid=5となるオブジェクトがユニークとは限らないことです。
確かにこちらの前提があった場合は、こちらを考慮し実装する必要があったかもしれません。
今回はidが5であるオブジェクトが複数あってもそれはユニークであるとの前提のつもりで質問しました。

より正確な情報を質問文に追記していくよう注意していきたいと思います。

[yambejp]

そういうことでしたらid=1,2,3が複数発生するサンプルはよろしくなかったですね

Answer 2

ブロック内でconst宣言した変数はブロックから抜けると破棄されますよ。
最初に変数宣言するといいです。

  let planPoint;
  planList.forEach(e1 => {
      const result = e1.plans.find(e2 => e2.id === '5');
      if(result){
          planPoint = result.score;
      }
  })
  console.log('point' , planPoint)

他の方法：

const planPoint = planList.map(plan => plan.plans).flat().find(plan => plan.id === '5')?.score;
console.log('point' , planPoint)

Comments

[ruuuu]

ご回答ありがとうございます。
> ブロック内でconst宣言した変数はブロックから抜けると破棄されますよ。
最初に変数宣言するといいです。
こちら、何故か、変数宣言したはずが何らか他コードの影響でエラー出ていたみたいでして、混乱してしまっていたのですが、再度検証してみましたら、エラーは表示されず実行できました。
しかし、ご提示頂いたコードの方が明らかにシンプルだし読みやすいですね。
ご提示頂いたコードで思い出したのですが、「flatMap」を最近どこかで聞いてこちらを取り入れてみたのですが、こちらの形でも値の取得が出来ました
const score = planList
.flatMap(plan => plan.plans)
.find(plan => plan.id === '5').score
エラーの問題もありましたし、冗長になってしまっていたので教えて頂き大変助かりました。

[shiracamus]

find で見つからないと .score がエラーになりますので、 ?.score と ? を付けておくといいですよ。

[ruuuu]

ありがとうございます。
検証してみました所、ご指摘の通りエラーとなってしまいました。
「?」こちらを付与しておきます