jsで特定のキーを持つプロパティをカウントする方法

Question

java scriptでオブジェクトに対し、
値が入っているかをチェックしつつ、かつ特定のキーを持つプロパティをカウントする方法は、
どんなやり方が考えられるでしょうか？

・詳細
DBから取り出したデータの内、特定のキーを持つデータの数を数えたいです。
データの構成は以下の通りです。

problem{
// DBから取得したデータ配列
name: ...
category: ...
problem1: ...
problem2: ...
problem3: null
problem4: null
problem5: null
}

problem2以降は入力必須ではないため、値がnullの可能性もあります。

・やったこと
nullであろうとkeyは絶対あるので、for文とnullかどうかを判定しました。

ProblemAmount() {
  // return console.log(typeof this.drill);
  let amount = 0;
  for (let i = 1; i <= 10; i++) {
     if (!this.drill["problem".i] === null) {
      ++amount;
      console.log(amount);
      }
    }
  return amount;
  // 0
}

しかし、上記のコードを実行しても0が返ってきてしまい、対処に詰まってしまいました。

お手数かけますが、アドバイスをいただけますと幸いです。

Comments

[kei344]

問題を再現できるコードを提示されたほうが回答が付きやすいですよ。

Answer 1

const data = {
  a: 42,
  b: 43,
  c: null,
  d: null,
  e: 108,
  f: 109,
};

というデータを対象に、２例挙げます。

let counter = 0;
for (const key in data) {
  if (data[key] !== null) counter++;
}
console.log(counter);
//=> 4

console.log(Object.entries(data).reduce((acc, [key, value]) => acc + (value !== null ? 1 : 0), 0));
//=> 4

追記：
jun68yktさんの回答を見て、自分が質問の意図を取りそびれていたと気付かされました。
と、いうことは質問者さんのコード、以下の部分を…

if (!this.drill["problem".i] === null) {

以下のように修正すれば…

if (!(this.drill["problem"+i] === null)) {
// もしくは
if (this.drill["problem"+i] !== null) {

質問者さんの期待通り動くんじゃないでしょうか。

Comments

[marutto]

回答ありがとうございます。

仰る通り、真偽判定の仕方が間違っていたのを変えたらカウントできました！
jun68ykt様の回答とどちらにしようか迷いましたが、
こちらのコードを採用しようと思うので、ベストアンサーにさせていただきます。

Answer 2

こんにちは

以下のような data というオブジェクトがあるとして、

const data = {
  name: 'foo',
  category: 'bar',
  
  problem1: 'A',
  problem2: 'B',
  problem3: null,
  problem4: 'C',
  problem5: null,
  problem6: null,
  problem7: 'D',    
};

data のプロパティの中で、そのプロパティ名がproblem で始まり、かつ値が null(を含めfalsyな値）ではないものの数を求めるには、

Object.entries(data)
  .filter(([key, value]) => /^problem/.test(key) && value)
  .length;

とすればよいかと思います。冒頭挙げた data の例だと、 4 が得られます。

以下にて動作確認できます。

• 動作確認用 CodePen: https://codepen.io/jun68ykt/pen/povzgqq?editors=0012

参考になれば幸いです。

補足1

null ではないものは、（たとえfalsyな値であっても）全てカウントしたいのであれば、filterに与える条件の && の後を

value !== null

とします。

• 動作確認用 CodePen: https://codepen.io/jun68ykt/pen/rNaBxEB?editors=0012

補足2

蛇足ですが、もう一つ追記します。

配列やオブジェクトの操作で便利なライブラリ lodash の _.pickBy を使うと、与えられたオブジェクトから、ある条件を満たすプロパティだけをピックアップしたオブジェクトが得られます。

これを使うと、ご質問の要件は例えば以下のように書けます。

const isNonFalsyProblem = (value, key) => key.startsWith('problem') && !!value;

const availableProblems = _.pickBy(data, isNonFalsyProblem);

const numKeys = Object.keys(availableProblems).length;

• 動作確認用 CodePen: https://codepen.io/jun68ykt/pen/jOENBrY?editors=0012

Comments

[marutto]

回答ありがとうございます!

filterで１つづつ取り出して、
正規表現を使ってkeyがproblem + valueがある値のlengthプロパティを参照する

配列操作やグローバルオブジェクトの使い方、とても参考になります！
丁寧にCodePenまで付けてくださり、ありがとうございました！

[jun68ykt]

どういたしまして。解決したようで、よかったです。
オブジェクトの持つプロパティと値のペアを走査する場面で、Object.entries() は便利です。
それと、キーが "problem" で始まるという条件を、回答では正規表現を使って
/^problem/.test(key)
としましたが、正規表現を使わずとも、"〜で始まる" をチェックする専用のメソッドstartsWith があるので、
key.startsWith("problem")
でもよかったです。