javascriptのメソッドreduceについて

Question

オブジェクトの配列から要素propの文字列が空ではないものの総数を取得したくて以下のようなコードを書きました。(インデント4すみません)

ary = [
   {
       prop: "",
       foo: "",
   },
   {
       prop: "aaa",
       foo: "",
   },
   {
       prop: "",
       foo: "",
   }
]

const sum = ary.reduce((count, obj) => {
    if (obj.prop !== "") {
      return count + 1;
    }
}, 0);

期待する値は1です。
しかし上記だと結果はundefinedになってしまいます。
reduceについて調べてみたのですが、どのように書けば期待した結果が取得できるかわかりません。
どなたかわかる方おられましたらお教えください。
(※ちなみに、reduceメソッドでなくてもスマートな書き方がありましたら教えてほしいです)

Answer 1

こんにちは
ベストアンサー決定後に失礼します。

(※ちなみに、reduceメソッドでなくてもスマートな書き方がありましたら教えてほしいです)

とのことですので、reduce を使わないコードの一例を回答します。filter を使えば、 ary の要素で prop が空文字列ではないものだけを集めた配列を作ることができるので、それの length が得るべき値になります。

const sum = ary.filter(e => e.prop).length

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

Comments

[ginuser]

すごいスッキリしたコードですね！
しかもなかなか直感的で良いです、ありがとうございました！

[jun68ykt]

どういたしまして????
> しかもなかなか直感的で良いです
とのことでよかったです ????

コツとしては、
reduce はもちろん便利ですが、
「reduce を使おうか？」
と思ったときに、reduceを使ったコードを書き始める前に、
「 filter (や map) で済ませられないか？」
と、ちょっと立ち止まって考えてみるとよいかもしれません。

[ginuser]

なるほど。
ありがとうございます、その考え、次から生かしてみます！

Answer 2

ifの中だけreturnしてますね

const sum = ary.reduce((count, obj) =>count+(obj.prop!==""?1:0),0);

Comments

[ginuser]

ありがとうございます！
盲目でした。

Answer 3

if部分しかないけど、else return count;が要りますね。

Comments

[ginuser]

ありがとうございます！
他の方の解答が同じ趣旨だったので解答時間の早い方をベストアンサーとさせていただきました！