Question

我有一个从数据库加载的数据对象。用户可以在页面上选择选项，对于初始对象中不存在的每个选项，我需要创建它们。

我写了一些伪造的代码，用于我在这里尝试的内容，它会导致内存问题并崩溃。

这里的目标是我遍历每个task＆gt; Roles数组并检查是否需要创建任何缺少的对象。需要创建的对象来自selectedRoles。

期望的结果：

在第一个任务中，它应该创建3个角色，因为它当前为空。在第二个任务中，它将创建角色1和3，因为已存在id为2的角色。

var tasks = [{
  "User": {
    "FirstName": "Joe",
    "LastName": "Dirt",
    "selected": false,
    "marked": false
  },
  "Tool": {
    "id": 31,
    "text": "Admin",
    "ToolID": "31",
    "ToolName": "Admin",
    "ToolSuite": "Enterprise Product"
  },
  "Roles": []
}, {
  "User": {
    "FirstName": "Bart",
    "LastName": "Simpson",
    "selected": false,
    "marked": false
  },
  "Tool": {
    "id": 35,
    "text": "Wordpress",
    "ToolID": "35",
    "ToolName": "Wordpress",
    "ToolSuite": "Enterprise Product"
  },
  "Roles": [{
    RoleName: 'Role 2',
    Role: 2,
    RoleID: 2
  }]
}];

// New selected roles from list
var selectedRoles = [1, 2, 3];

////////////////////////////////////////////////////////////////////////

/* 
  Loop over the configured tasks and
  if there is not an existing role
  matching a role id from "SelectedRoles",
  create a new role within that task.
*/

// Loop over the tasks
tasks.forEach((task) => {
    // If we have roles, loop over them
  if (task.Roles.length) {
    for (let i = 0; i < task.Roles.length; i++) {
            // If this roleID does not exist in our selectedRoles, create the task
      if(selectedRoles.indexOf(task.Roles[i].RoleID) >= 0){
        // Create this role and add it to our task
        task.Roles.push(createRole('Role ' + task.Roles[i].RoleID, task.Roles[i].RoleID, task.Roles[i].RoleID));
      }
    }
  }
});

console.log(tasks)

function createRole(RoleName, RoleID, Role){
    return {
    RoleName: RoleName,
    RoleID: RoleID,
    Role: Role
  }
}

有关更清洁（和工作）方式的任何想法吗？

Answer 1

var tasks = [{
  "User": {
    "FirstName": "Joe",
    "LastName": "Dirt",
    "selected": false,
    "marked": false
  },
  "Tool": {
    "id": 31,
    "text": "Admin",
    "ToolID": "31",
    "ToolName": "Admin",
    "ToolSuite": "Enterprise Product"
  },
  "Roles": []
}, {
  "User": {
    "FirstName": "Bart",
    "LastName": "Simpson",
    "selected": false,
    "marked": false
  },
  "Tool": {
    "id": 35,
    "text": "Wordpress",
    "ToolID": "35",
    "ToolName": "Wordpress",
    "ToolSuite": "Enterprise Product"
  },
  "Roles": [{
    RoleName: 'Role 2',
    Role: 2,
    RoleID: 2
  }]
}];

// New selected roles from list
var selectedRoles = [1, 2, 3];

////////////////////////////////////////////////////////////////////////

/* 
  Loop over the configured tasks and
  if there is not an existing role
  matching a role id from "SelectedRoles",
  create a new role within that task.
*/
tasks.forEach((task) => {
    if(task.Roles.length == 0){
      for(var i = 0; i < selectedRoles.length; i++){
        task.Roles.push(createRole('Role ' + selectedRoles[i], selectedRoles[i], selectedRoles[i]));
      }
    }
    else{
      var concatRoleArr = [];
      var roleIdArr = [];
      for(var i = 0; i < task.Roles.length; i++){
         roleIdArr.push(task.Roles[i].RoleID);
      }
      for(var i = 0; i < selectedRoles.length; i++){
        var roleIndex = roleIdArr.indexOf(selectedRoles[i]);
        if(roleIndex < 0){
          concatRoleArr.push(createRole('Role ' + selectedRoles[i], selectedRoles[i], selectedRoles[i]));
        }
      }
      task.Roles = task.Roles.concat(concatRoleArr);
    }
});

console.log(tasks);


function createRole(RoleName, RoleID, Role){
    return {
    RoleName: RoleName,
    RoleID: RoleID,
    Role: Role
  }
}

以下是您更新后的代码。我们改变了你的循环。我们首先检查是否有任何角色。如果没有，我们循环遍历selectedRoles并将每个角色添加到角色中。

如果任务中存在角色，我们创建一个新数组（concatRoleArr），将所有roleIds收集到一个新数组中，然后再次遍历selectedRoles。如果现有角色ID的数组不包含当前查看的selectedRole（indexOf为-1），则将其添加到新的concatRoleArr，否则，继续。完成后，使用concatRoleArr连接task.Roles数组。

您遇到了内存问题，因为您的原始代码具有无限循环。你可以在task.Roles中添加角色，所以循环只是继续进行，因为当你不断添加更多内容时，你的for循环无法完成任务结束。并且它继续添加更多因为如果它在数组中的EXISTS，那么检查indexOf与＆gt; = 0将是真的，你的角色总是因为你只是循环遍历现有角色。检查＆lt; 0检查它是否不存在，因为它将返回-1。

Javascript循环对象并从数组

1 个答案: