Javascript：for..in 循环运行的次数超出预期

Question

在下面的代码中，user.roles 的实际长度为 1。但是，循环运行了两次。

当我输出 i 的值时，它在第二次迭代中显示为 'diff'。 切换到普通的 for 循环解决了这种情况。 但是，我想知道 for..in 循环有什么问题。

 for (var i in user.roles) {
                if (user.roles[i].school.equals(schoolId)) {
                    for (var j in user.roles[i].permissions) {
                        for (var k in accessType) {
                            if (user.roles[i].permissions[j].feature == featureKey) {
                                if (user.roles[i].permissions[j][accessType[k]]) {
                                    return true;
                                }
                            }
                        }
                    }
                }
            }

更新：用户是一个对象，角色是一个对象数组。导致问题的角色实例如下所示：

{
  "_id": "582d3390d572d05c1f028f53",
  "displayName": "Test Teacher Attendance",
  "gender": "Male",
  "roles": [
    {
      "_id": "57a1b3ccc71009c62a48a684",
      "school": "57a1b3ccc71009c62a48a682",
      "role": "Teacher",
      "__v": 0,
      "designation": true,
      "permissions": [
        {
          "feature": "User",
          "_id": "57ac0b9171b8f0b82befdb7d",
          "review": false,
          "view": true,
          "delete": false,
          "edit": false,
          "create": false
        },
        {
          "feature": "Notice",
          "_id": "57ac0b9171b8f0b82befdb7c",
          "review": false,
          "view": true,
          "delete": false,
          "edit": false,
          "create": false
        },

      ]
    }
  ],
}

Answer 1

user.roles 似乎是一个数组。而对于数组，你不应该使用 for in。

简单示例

var arr = [2];
arr.s = 3;

for (var i  in arr) {
console.log("here"); // paints twice
}

来自MDN，for...in 语句以任意顺序迭代对象的可枚举属性。对于每个不同的属性，可以执行语句。

如何选择迭代器的类型，这里参考iterators

编辑

根据更新后的问题，如果以下代码中的某处存在，则上述内容只能带有 diff 属性

Array.prototype.diff = .....

Answer 2

我想这就是你要找的。 我假设您的 accessTypes 是一个包含以下项目的数组：

var accessTypes = ["review", "view", "delete", "edit", "create"];

编辑以提高效率。

var schoolId = "57a1b3ccc71009c62a48a682";
var featureKey = "Notice";
var accessTypes = ["review", "view", "delete", "edit", "create"];

var user = {
  "_id": "582d3390d572d05c1f028f53",
  "displayName": "Test Teacher Attendance",
  "gender": "Male",
  "roles": [{
    "_id": "57a1b3ccc71009c62a48a684",
    "school": "57a1b3ccc71009c62a48a682",
    "role": "Teacher",
    "__v": 0,
    "designation": true,
    "permissions": [{
      "feature": "User",
      "_id": "57ac0b9171b8f0b82befdb7d",
      "review": false,
      "view": true,
      "delete": false,
      "edit": false,
      "create": false
    }, {
      "feature": "Notice",
      "_id": "57ac0b9171b8f0b82befdb7c",
      "review": false,
      "view": true,
      "delete": false,
      "edit": false,
      "create": false
    }]
  }]
};

user.roles.forEach(function(roleItem) {
  // console.log('This is a role: ' + roleItem.school);
  if (roleItem.school == schoolId) {
    roleItem.permissions.forEach(function(permissionItem) {
      // console.log('This is a permission: ' + permissionItem.feature);
      // console.log('This is a accessType: ' + accessType);
      if (permissionItem.feature == featureKey) {
        accessTypes.forEach(function(accessType) {
          if (permissionItem[accessType]) {
            console.log('accessType: ' + accessType + ' -> true');
            return true;
          }
        });
      }
    });
  }
});

forEach 接受迭代器函数。为数组中的每个条目调用迭代器函数，按顺序跳过稀疏数组中不存在的条目。

forEach 还有一个好处是，您不必在包含范围内声明索引和值变量，因为它们作为参数提供给迭代函数，因此很好地限定了该迭代。

如果您担心为每个数组条目进行函数调用的运行时成本，请不要担心；更多technical details.

如果您仍然认为forEach 的速度基本上较慢，您可以使用一个简单的for 循环，正如我在另一个my answers 中解释的那样。

希望对您有所帮助。