I have a function along the lines of the following:
doSomething: function () {
var parent = null;
if (this === null) {
parent = 'some default value';
} else {
parent = this.SomeValue();
}
}
Could parent ever be set to 'some default value' or is the check for null superfluous?
Alternatively, what if I used the less restrictive:
doSomething: function () {
var parent = this ? this.SomeValue() : 'some default value';
}
Could parent ever be set to 'some default value' in this case?
In non-strict mode, this
has undergone an Object(this)
transformation, so it's always truthy. The exceptions are null
and undefined
which map to the global object. So this
is never null
and always truthy, making both checks superfluous.
In strict mode, however, this
can be anything so in that case you'd have to watch out. But then again you have to opt in for strict mode yourself, so if you don't do that there are no worries.
(function() { return this; }).call(null); // global object
(function() { "use strict"; return this; }).call(null); // null
The specification of ES5 says:
The thisArg value is passed without modification as the this value. This is a change from Edition 3, where a undefined or null thisArg is replaced with the global object and ToObject is applied to all other values and that result is passed as the this value.