forEachで取得した値の加算

Question

前提・実現したいこと

forEachで取得した値を加算して合計を求めたいです。

発生している問題・エラーメッセージ

firestoreから抽出したデータは表示されます。(該当の日付にあるデータ)
consoleで確認するとforEachで取得できたデータが順に表示されます。
色々ググって試しましたが、合計を出すことができませんでした。

該当のソースコード

for(var d = startDate; d <= endDate; d.setDate(d.getDate()+1)) {
  var formatedDate = d.getFullYear()+""+(d.getMonth()+1)+""+d.getDate();
 
  dateList.push(formatedDate);

}

dateList.forEach(function(element){
 db.collection("users").doc(userId).collection("time").doc(element).get().then((snapshot)=>{

      if(snapshot.data() !== undefined){

        scores = snapshot.data().toMinutes;
　　　　
　　　　//ここに式を入れればいいと思ったのですが…
　　　　//以下は色々試したなかの一つです。

        let result = [scores];
        let total = result.reduce(function(sum, element){
          return sum + element;
        }, 0);
console.log(total)
　　　　//ここまで

      }else{～～～

(追記：kis9aさんに教えて頂いたコード）
for (var d = startDate; d <= endDate; d.setDate(d.getDate() + 1)) {
var formatedDate =
d.getFullYear() + "" + (d.getMonth() + 1) + "" + d.getDate();

dateList.push(formatedDate);
}

console.log(dateList); // month.html 78

const getTotalOfMonth = (dateList) => {
let totalOfMonth = 0

dateList.forEach((date) => {
db.collection("users").doc(userId).collection("time").doc(date).get().then((snapshot) => {

    if (snapshot.data() !== undefined) {
      scores = snapshot.data().toMinutes;

      // ここに式を入れればいいと思ったのですが…
      // 以下は色々試したなかの一つです。

      let result = [scores];

      let totalOfDate = result.reduce(function (sum, element) {
        return sum + element;
      }, 0);

      totalOfMonth = totalOfMonth + totalOfDate;

★☆★☆★☆★☆★☆★☆★☆★☆★☆★☆★☆★☆★☆★☆
//追加
let array1 = [];
array1.push(totalOfMonth)
console.log(array1)
★☆★☆★☆★☆★☆★☆★☆★☆★☆★☆★☆★☆★☆★☆
} else {
}
});
});
return getTotalOfMonth
};

console.log(getTotalOfMonth(dateList))

上記イメージにあるようにコンソールには表示されます。

補足情報（FW/ツールのバージョンなど）

初歩的な内容かもしれないのですが、私の検索能力では限界だったのでお助け頂けると助かります。よろしくお願いします。

Answer 1

forEach で指定するのはコールバックですからね。

素直に for (x of dateList) { ... } 使えば?
--- 追記 ---
コールバックではループで使用する「関数」を指定しますから、ループの中と外では参照/更新できるものとできないものがあるので、そのへん慎重にコーディングする必要があります。
一方、上記を使えばそのような制限はないので手軽にコードを書けます。

for (element of dateList) {
    db.collection("users").doc(userId).collection("time").doc(element).get().then((snapshot)=>{

        if (snapshot.data() !== undefined) {

            scores = snapshot.data().toMinutes;

            //ここに式を入れればいいと思ったのですが…
            //以下は色々試したなかの一つです。

            let result = [scores];
            let total = result.reduce(function(sum, element){
                return sum + element;
                }, 0);
            console.log(total)
            //ここまで

      } else {
      
      }
}

たとえば、ループ内で計算した値を集計したいならば、ループ前に初期化してループ内で順次集計すればいいだけです。
ちなみに、全角空白がまじってるので半角空白に修正した方がいいと思います。

Comments

[teddy1121]

ご回答ありがとうございます。
「for of datalist」
でググったのですが私の理解が追い付かずやり方がわかりませんでした。
例などを見せて頂けたら有難いです。
お手数をおかけして申し訳ありませんがよろしくお願いします。

[teddy1121]

ご回答ありがとうございます。
全角と半角…
お恥ずかしい‥
修正したいと思います。
ご提案頂いたコードも試してみたいと思います。
ご丁寧にありがとうございました！

Answer 2

すみません内容が少し分かりにくいです。
タイトル通りの意味だと下のように思いますし。

const a = [1, 2, 3];

let res = 0;

a.forEach((v) => {
  res = res + v;
});


console.log(res); // 6

例にあるようにArray.reduceを使って合計を求めることもできると思います。

let res = a.reduce((v, e) => {
  return v + e;
}, 0);

console.log(res); // 6

どの変数にどのような値が入っているのかを具体的にしていただくか、
もう少し、コードを抽象化して返信いただけますか？
エラーが出ているのであればエラーのメッセージも共有していただけますか？

追)

for (var d = startDate; d <= endDate; d.setDate(d.getDate() + 1)) {
  var formatedDate =
    d.getFullYear() + "" + (d.getMonth() + 1) + "" + d.getDate();

  dateList.push(formatedDate);
}

console.log(dateList); // month.html 78

const getTotalOfMonth = (dateList) => {
  let totalOfMonth = 0

  dateList.forEach((date) => {
    db.collection("users").doc(userId).collection("time").doc(date).get().then((snapshot) => {

        if (snapshot.data() !== undefined) {
          scores = snapshot.data().toMinutes;

          // ここに式を入れればいいと思ったのですが…
          // 以下は色々試したなかの一つです。

          let result = [scores];

          let totalOfDate = result.reduce(function (sum, element) {
            return sum + element;
          }, 0);

          totalOfMonth = totalOfMonth + totalOfDate;

          console.log(total); // month.html 92

        } else {
        }
      });
  });
  return getTotalOfMonth
};

console.log(getTotalOfMonth(dateList))

Comments

[teddy1121]

ご返答ありがとうございます！説明が下手で申し訳ありません。
具体的にいくと、"dateList"で今回の場合202191~2021930という形で日付を呼び出しています。
その値を元にfirestoreからデータのある日付を抽出し、その日にある分数を取得しています。
コンソール画面には分数が縦に並びます。
配列も調べたのですが、取得した分数を配列内[]に格納する方法がわからず…
ご提案頂いた方法が今試せないので後でトライしてみます！
どう質問したら伝わるかもわからずすみません…

[teddy1121]

返信遅くなり申し訳ありません。
質問部分を編集しました。
console.log(total)は画像のmonth.html:92です。
コンソールには画像のように分数が出るのですが、これを配列にすればいいと思ったのですが違いますか？
ご提案頂いた内容だとv,eの部分がこれに当たるのかと思ったのですが、値をv,eに入れる方法がわりません。
本当に初歩的な内容かと思うのですがご教示頂けたら幸いです。

[退会済みユーザー]

つまり、155 + 289 + ... と console.log(total) で出力された値を合計したいと言うことでしょうか？
値をv,eに入れる方法 、reduceの仕様に関しては、
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/Reduce を見てください。
コードも追記したので確認してください。

[退会済みユーザー]

配列に格納する場合。forEachの外で空の配列の変数を用意して、Array.push() すればどうでしょうか？

[teddy1121]

遅くなり申し訳ありません。
はい、155+289+という形で対象期間内の日ごとの分数を合計して1か月の総計を出したいです。
追記頂いた文をそのままコピペすると「console.log(getTotalOfMonth(dateList))」には「(datelist) =>{」以降のコードがそのまま表示されます。コピペの仕方が違うのでしょうか？
自分でもいろいろ弄ってみますが併せてご教授いただければ幸いです。

[teddy1121]

kis9aさんに教えて頂いたようにArray.push()で配列にtotalOfMonthを追加したら合計数を取得できました！
この方法があっているのかわかりませんが、とりあえず動いたのでkis9aさんをベストアンサーにさせて頂きます！教えて頂きありがとうございました。もしより良い方法があれば教えて頂けたら嬉しいです！