Javascript中对象过滤的变异问题

Javascript中对象过滤的变异问题,javascript,reactjs,immutability,Javascript,Reactjs,Immutability,几个月前,我开始使用react,我认为直到现在我对突变的理解还不够透彻 我有一个函数,它接受一个对象、选定的itemId和parentId(可选)。数据的嵌套元素最多可以达到1级。如果itemId匹配,我需要做的是更改键selected的值。如果itemId在父对象中,那么我需要在该对象中设置selected 另一个用例是,如果itemId是父项的id。在这种情况下,我还需要设置所有子项的selected值。这就是问题所在。 当我试图设置子对象的选定值时,函数正在改变我发送的数据,因此返回错误的

几个月前,我开始使用react,我认为直到现在我对突变的理解还不够透彻

我有一个函数,它接受一个对象、选定的itemId和parentId(可选)。数据的嵌套元素最多可以达到1级。如果itemId匹配,我需要做的是更改键
selected
的值。如果itemId在父对象中,那么我需要在该对象中设置
selected

另一个用例是,如果itemId是父项的id。在这种情况下,我还需要设置所有子项的selected值。这就是问题所在。 当我试图设置子对象的
选定值时,函数正在改变我发送的数据,因此返回错误的结果

以下是所讨论的功能:

function returnSelectedItems(data, itemId, parentId) {
  // created this object trying to rule out mutation problem in the tests. @see tests file.
  const originalData = data.map(item => item);
  if (parentId) {
    // if parent id is present, a child element has been selected, then directly go to the element
    // go through all the items
    return originalData.map(parentItem => {
      // see if the id matches the parent item
      if (parentItem.id === parentId) {
        // if yes, check if it contains any children
        if (parentItem.children && parentItem.children.length > 0) {
          // iterate through their children and assign the resulting array as the children for the parent
          parentItem.children = parentItem.children.map(childItem => {
            // find the child with the id selected and change its value
            if (childItem.id === itemId) {
              childItem.selected = !childItem.selected;
            }
            return childItem;
          });
        }
      }
      return parentItem;
    });
  } else {
    return originalData.map(item => {
      if (item.id === itemId) {
        // set the item to the opposite it was before
        item.selected = !item.selected;

        // if it has children, we have to set the values for all of them.
        if (item.children && item.children.length > 0) {
          item.children = item.children.map(childItem => {
            childItem.selected = item.selected;
            return childItem;
          });
        }
      }
      return item;
    });
  }
}
下面是我创建的测试,以测试这是否如预期的那样工作:

test('WITHOUT children test for selectedItems function', () => {
  const data = [
    {
      id: 1,
      name: 'test',
      selected: false
    },
    {
      id: 2,
      name: 'test',
      selected: false
    },
    {
      id: 3,
      name: 'test',
      selected: false
    },
    {
      id: 4,
      name: 'test',
      selected: false
    },
    {
      id: 5,
      name: 'test',
      selected: false
    }
  ];
  expect(returnSelectedItems(data, 4)).toEqual(
    data.map(item => {
      if (item.id === 4) {
        item.selected = true;
      }
      return item;
    })
  );
  expect(returnSelectedItems(data, 5)).toEqual(
    data.map(item => {
      if (item.id === 5) {
        item.selected = true;
      }
      return item;
    })
  );
  expect(
    returnSelectedItems(
      data.map(item => {
        if (item.id === 4) {
          item.selected = true;
        }
        return item;
      }, 4)
    )
  ).toEqual(data);
  expect(
    returnSelectedItems(
      data.map(item => {
        if (item.id === 5) {
          item.selected = true;
        }
        return item;
      }, 5)
    )
  ).toEqual(data);
});

test('WITH children test for selectedItems function', () => {
  const data = [
    {
      id: 1,
      name: 'test',
      selected: false,
      children: [{id: 1, name: 'test', selected: false}, {id: 2, name: 'test', selected: false}, {id: 3, name: 'test', selected: false}]
    },
    {
      id: 2,
      name: 'test',
      selected: false,
      children: [{id: 1, name: 'test', selected: false}, {id: 2, name: 'test', selected: false}, {id: 3, name: 'test', selected: false}]
    },
    {
      id: 3,
      name: 'test',
      selected: false,
      children: [{id: 1, name: 'test', selected: false}, {id: 2, name: 'test', selected: false}, {id: 3, name: 'test', selected: false}]
    },
    {
      id: 4,
      name: 'test',
      selected: false,
      children: [{id: 1, name: 'test', selected: false}, {id: 2, name: 'test', selected: false}, {id: 3, name: 'test', selected: false}]
    },
    {
      id: 5,
      name: 'test',
      selected: false,
      children: [{id: 1, name: 'test', selected: false}, {id: 2, name: 'test', selected: false}, {id: 3, name: 'test', selected: false}]
    }
  ];

  const backupData = [
    {
      id: 1,
      name: 'test',
      selected: false,
      children: [{id: 1, name: 'test', selected: false}, {id: 2, name: 'test', selected: false}, {id: 3, name: 'test', selected: false}]
    },
    {
      id: 2,
      name: 'test',
      selected: false,
      children: [{id: 1, name: 'test', selected: false}, {id: 2, name: 'test', selected: false}, {id: 3, name: 'test', selected: false}]
    },
    {
      id: 3,
      name: 'test',
      selected: false,
      children: [{id: 1, name: 'test', selected: false}, {id: 2, name: 'test', selected: false}, {id: 3, name: 'test', selected: false}]
    },
    {
      id: 4,
      name: 'test',
      selected: false,
      children: [{id: 1, name: 'test', selected: false}, {id: 2, name: 'test', selected: false}, {id: 3, name: 'test', selected: false}]
    },
    {
      id: 5,
      name: 'test',
      selected: false,
      children: [{id: 1, name: 'test', selected: false}, {id: 2, name: 'test', selected: false}, {id: 3, name: 'test', selected: false}]
    }
  ];

  // this console log is correct
  console.log(data.filter(item => item.id === 4)[0]);
  const dataBack = returnSelectedItems(data, 4);
  // this console is also correct
  console.log(dataBack.filter(item => item.id === 4)[0]);
  // but here the value of the data is changed
  console.log(data.filter(item => item.id === 4)[0]);

  // this passes, but it shouldn't
  expect(returnSelectedItems(data, 4)).toEqual(
    data.map(item => {
      if (item.id === 4) {
        item.selected = true;
      }
      return item;
    })
  );

  // this should pass but it does not
  expect(backupData).toEqual(data);
});

谢谢你的帮助

const originalData=data.map(item=>item)
只创建
数据
数组的浅拷贝。因此,您在
原始数据
数组中修改的
对象
项将导致
数据
数组发生突变。因此,您必须深度复制
数据
数组。为方便起见,请使用类似的库。

常量originalData=data.map(item=>item)仅创建
数据
数组的浅拷贝。因此,您在
原始数据
数组中修改的
对象
项将导致
数据
数组发生突变。因此,您必须深度复制
数据
数组。为方便起见,请使用类似的库。

太好了!谢谢这就解决了问题。顺便说一句,对于其他有类似问题的人,我只在我的应用程序中导入了
lodash.clonedeep
,使用
warn add lodash.clonedeep
。然后
从'lodash.cloneDeep'导入cloneDeep;const originalData=cloneDeep(数据)完美!谢谢这就解决了问题。顺便说一句,对于其他有类似问题的人,我只在我的应用程序中导入了
lodash.clonedeep
,使用
warn add lodash.clonedeep
。然后
从'lodash.cloneDeep'导入cloneDeep;const originalData=cloneDeep(数据)