Consider this simple code:
"use strict";
var obj = {
f: function() {
this.prop = 'value';
g.bind( this )();
}
};
function g() {
console.log( this.prop );
}
If I try to validate this code, jshint gives me the error Possible strict violation.
where I call console.log( this.prop );
. This is because this
is undefined in strict mode in a function.
But I'm binding this function before calling it, so this
is the correct object.
I'm using this "design pattern" to avoid cluttering the main object. Passing the properties in the parameters will also clutter the function, so I refuse to do this. Besides, this is exactly what bind
is for.
Is there a way for JSHint to let me do this?
It is extremely hard to detect this case without running the code. You can use option validthis
to suppress this warning:
"use strict";
var obj = {
f: function() {
this.prop = 'value';
g.bind( this )();
}
};
function g() {
/*jshint validthis:true */
console.log( this.prop );
}
It is to be noted that jshint comments are function scoped. So the comment will work for the function g
and its inner functions, not just the next line.
You can also achieve the same effect if you modify your code to the following to avoid using this
all together.
"use strict";
var obj = {
f: function() {
this.prop = 'value';
g.bind( null, this )();
}
};
function g(self) {
console.log( self.prop );
}
Here's a simpler solution that doesn't require any change of pattern or specific markup for jshint:
"use strict";
var obj = {
f: function() {
this.prop = 'value';
G.bind( this )();
}
};
function G() {
console.log( this.prop );
}
jshint assumes that you're following the convention that functions starting with an uppercase letter are classes which will be instantiated and always having this
available.
Try:
"use strict";
var obj = {
f: function() {
this.prop = 'value';
g.bind( this )();
}
};
var g = function() {
console.log( this.prop );
}
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With