In the eslint rule guard-for-in , use for in
directly is incorrect. The good practice is
for (key in foo) {
if (Object.prototype.hasOwnProp
They'll never1 lead to different results, which is why both of them are considered correct.
The object {}
always inherits from the object prototype and has no own keys, so accessing the .hasOwnProperty
method on it will get you what you want. And Object.prototype
is a non-writable property of the global Object
constructor, so it always refers to the object prototype as well.
Now, there are some cases where the expression does not do what you wanted it to do:
Object
variableObject
identifier in your scope so that it doesn't refer to the globalObject.prototype.hasOwnProperty
with something elseFunction.prototype.call
with something else1: Obviously, the first two edge cases only mess with the Object.prototype
reference, but they're still edgy enough to be ignored.