Question

我有一个对象，其中数字月份是其他对象的键，其中包含键的年份和初始值的零。

MonthRow : {   
   1 : {2017:0,2018:0},
   2 : {2017:0,2018:0},
   3 : {2017:0,2018:0},
   4 : {2017:0,2018:0},
   5 : {2017:0,2018:0},
   6 : {2017:0,2018:0}
}

查询后，我使用以下代码为每个对象设置值

 Object.keys(MainData.MonthRow).forEach(function(key){
    MainData.block.forEach(function(element,i,block){
      if(key == element.month){
        MainData.year.forEach(function(year, j, years){
          if(element.year == year && key == element.month){
           console.log("This is the Sale: ", element.Sale, "This is the year ", year, key);
            console.log(MainData.MonthRow[key], "This is the Month Key");
            console.log(MainData.MonthRow[key][year], "This is the year and key");
            MainData.MonthRow[key][year]=element.Sale;
            console.log(MainData.MonthRow)
          }
        })   
      }
    });

但是在使用MonthRow [key] [year] = element.Sale分配值之后；它将值分配给所有月份。

我的守时性问题是如何为obj.key.year = value分配一个值，其中key和year是变量？

在JSFiddle中重新创建，获得了预期的结果，但是在Sails框架上无法正常工作

JSFiddle Test

Answer 1

problem是MonthRow内的所有子对象都引用同一个对象（MainData.years），换句话说就是MainData.years === MonthRow['1'] === MonthRow['2'] === ...。因此，对这些子对象之一的更改也将反映在MainData.years的所有子对象上。这是问题的证明：

var objA = {};

var objB = objA;            // objA is not copied to objB, only a reference is copied
                            // objA and objB are pointing/referencing the same object

objB.foo = "bar";           // changes to one of them ...

console.log(objA);          // ... are reflected on the other

要解决此问题，您需要先克隆对象MainData.years，然后再分配给对象MonthRow的每个属性，因此子对象将都是不同的对象。您可以像这样使用Object.assign：

MonthRow = {
  '1': Object.assign({}, MainData.years),
  '2': Object.assign({}, MainData.years),
  ...
}

旁注：

可以将问题中的代码重构为较短的代码，因为您不需要循环MonthRow或MainData.year的键，只需要循环MainData.block，对于每个元素，您只需检查当前元素的年份是否包含在MainData.year中（使用indexOf或includes），然后使用元素的年份更新MainData.MonthRow月：

MainData.block.forEach(function(element) {
  if(MainData.year.indexOf(element.year) !== -1) {
    MainData.MonthRow[element.month][element.year] = element.sale;
  }
});

为另一个具有动态分配键的对象内的对象分配值

1 个答案: