ある指定の同じキーの値を合算し、別の配列を生成する

Question

JavaScriptで、以下のような形でデータが入った連想配列にて、
同じキーの要素（今回の例では日付）のamountを合算し、
新しい配列を生成しなおすロジックを考えています。

例:元の5件の連想配列

var dataList = [data, data, data, data, data];

0 : data {id: 100, date: "2018-06-10", amount: 124}
1 : data {id: 105, date: "2018-06-15", amount: 354}
2 : data {id: 120, date: "2018-07-01", amount: 217}
3 : data {id: 129, date: "2018-07-01", amount: 320}
4 : data {id: 131, date: "2018-07-01", amount: 140}
5 : data {id: 183, date: "2018-07-12", amount: 156}

※idは再度採番
※日付が同日のデータをamountを合算して以下の配列を生成

var newDataList = [data, data, data, data];

0 : data {id: 1, date: "2018-06-10", amount: 124}
1 : data {id: 2, date: "2018-06-15", amount: 354}
2 : data {id: 3, date: "2018-07-01", amount: 677}
4 : data {id: 4, date: "2018-07-12", amount: 156}

filter, mapといったメソッドを使い色々試しているのですが、思い通りにいかず苦戦しています。
ご教授いただけないでしょうか。よろしくお願い致します。

Comments

[kei344]

ご自身で試されたコードを質問文に追記し、「何」が「どのように」わからないのか、コードのどの部分で詰まっているのかなどを具体的に追記されたほうが回答が望めると思います。

[SKSG]

> kei344様　遅くなり申し訳ありません。仰る通り、もう少し自身でトライしその結果コードのどこがどうわからないのか、解決しない場合追記したいと思います。コメントありがとう御座いました。

[think49]

「新しい配列を生成しなおすロジックを考えています。」と書きながら、ロジックが全く書かれていないのが気になるところですね。
「filter, mapといったメソッドを使い色々試しているのですが、思い通りにいかず苦戦しています。」とありますが、ロジックをどう考えて、filter, map を使おうと思い至ったのでしょう？

Answer 1

推測100%であえて書きますが、初心者の方は特に、ロジックを考える部分を適当なところで切り上げて、見切り発車で適当にコードを書き殴って「上手く動きません」と悩んでいる印象があります。
コードはただの成果物(結果)で、ロジック(経過)の方がよほど重要なのですが…。

'use strict';
const dataList = [{id: 100, date: "2018-06-10", amount: 124}, {id: 120, date: "2018-07-01", amount: 217}, {id: 129, date: "2018-07-01", amount: 320}, {id: 131, date: "2018-07-01", amount: 140}, {id: 183, date: "2018-07-12", amount: 156}],
      map = new Map;

for (let data of dataList) {
  const date = data.date, exist = map.get(date);

  exist ? exist.amout += data.amount : map.set(date, data);
}

console.log(JSON.stringify([...map.values()])); // [{"id":100,"date":"2018-06-10","amount":124},{"id":120,"date":"2018-07-01","amount":217,"amout":null},{"id":183,"date":"2018-07-12","amount":156}]

このコードを見て「出来ました！」でクローズするか、「これはどういうロジックなのでしょう？」と疑問に思うか、が境界線だと私は思います。

Re: sak71g さん

Answer 2

パフォーマンスは変わらないので普通にループがおすすめです

var keyList=[];
var newDataList=[];
for(var i=0;i<dataList.length;i++){
    var keyIndex=-1;
    for(var n=0;n<keyList.length;n++){
        if(keyList[n]==dataList[i].date){
            keyIndex=n;
            break;
        }
    }
    if(keyIndex>=0){
        newDataList[keyIndex].amount+=dataList[i].amount;
    }else{
        keyList.push(dataList[i].date);
        newDataList[i].push({id: newDataList.length, date:dataList[i].date, amount: dataList[i].amount});
    }
}