let sumData = []; menu.value.forEach((item) => { const list = item.options.filter((v) => v.isActive); if (list.length > 0) sumData.push({ type: item.type, list, }); }); 优化

可以使用 Array.prototype.reduce() 方法来优化代码，避免使用额外的变量（如 sumData），同时也可以减少代码行数。具体实现如下：

const sumData = menu.value.reduce((acc, item) => {
  const list = item.options.filter((v) => v.isActive);
  if (list.length > 0) {
    acc.push({
      type: item.type,
      list,
    });
  }
  return acc;
}, []);

这里使用了 reduce() 方法，初始值为一个空数组。对于 menu.value 数组中的每个元素，我们都会执行一次回调函数。该回调函数中会对每个元素的 options 数组进行过滤，获得 isActive 为 true 的元素，然后将这些元素和 type 属性一起组成一个对象，最后将该对象 push 进 acc 数组中。最后，reduce() 方法返回的就是 acc 数组。

相关问题

menu.value.forEach((item) => { item.options.filter((v) => (v.isActive = false)); }); 优化

首先需要指出的是，这段代码的问题在于 filter 方法的回调函数中使用了“赋值”操作符，将 v.isActive 赋值为 false，这样会将所有的选项的 isActive 属性都设置为 false，而不是过滤出 isActive 为 false 的选项。

如果想过滤出 isActive 为 false 的选项，可以将回调函数中的“赋值”操作符改为“比较”操作符。另外，可以使用 map 方法替代 forEach 方法，将每个菜单项中的选项进行过滤，同时可以使用 const 声明变量，避免不必要的变量声明。改进后的代码如下：

menu.value = menu.value.map((item) => {
  const options = item.options.filter((v) => v.isActive === false);
  return {
    ...item,
    options
  };
});

这样会将每个菜单项中 isActive 属性为 false 的选项过滤掉，而不会改变选项的属性值。同时使用 map 方法将菜单项转化为需要的数据结构。