noForIn

Esta página aún no está disponible en tu idioma.

JavaScript (and super languages)

Summary

Diagnostic Category: lint/nursery/noForIn
This rule doesn’t have a fix.
The default severity of this rule is information.
Sources:
- Inspired from @typescript-eslint/no-for-in-array

How to configure

1
{
2
  "linter": {
3
    "rules": {
4
      "nursery": {
5
        "noForIn": "error"
6
      }
7
    }
8
  }
9
}

Description

Disallow iterating using a for-in loop.

A for-in loop (for (const i in o)) iterates over the properties of an Object. While it is legal to use for-in loops with array values, it is not common. There are several potential bugs with this:

It iterates over all enumerable properties, including non-index ones and the entire prototype chain. For example, RegExp.prototype.exec returns an array with additional properties, and for-in will iterate over them. Some libraries or even your own code may add additional methods to Array.prototype (either as polyfill or as custom methods), and if not done properly, they may be iterated over as well.
It skips holes in the array. While sparse arrays are rare and advised against, they are still possible and your code should be able to handle them.
The “index” is returned as a string, not a number. This can be caught by TypeScript, but can still lead to subtle bugs.

You may have confused for-in with for-of, which iterates over the elements of the array. If you actually need the index, use a regular for loop or the forEach method.

Examples

Invalid

1
for (const i in array) {
2
  console.log(i, array[i]);
3
}

code-block.js:1:1 lint/nursery/noForIn ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ℹ Unexpected for-in loop.
  
  > 1 │ for (const i in array) {
      │ ^^^^^^^^^^^^^^^^^^^^^^^^
  > 2 │   console.log(i, array[i]);
  > 3 │ }
      │ ^
    4 │ 
  
  ℹ For-in loops are confusing and easy to misuse. You likely want to use a regular loop, for-of loop or forEach instead.

Valid

1
for (const value of array) {
2
  console.log(value);
3
}

1
for (let i = 0; i < array.length; i += 1) {
2
  console.log(i, array[i]);
3
}

1
array.forEach((value, i) => {
2
  console.log(i, value);
3
});

1
for (const [i, value] of array.entries()) {
2
  console.log(i, value);
3
}