Question

44

Strict Violation using this keyword and revealing module pattern

rated 0 times [ 49] [ 5] / answers: 1 / hits: 25185 / 13 Years ago, fri, june 10, 2011, 12:00:00

Having trouble getting the following to pass jslint/jshint

/*jshint strict: true */

var myModule = (function() {

    use strict;



    var privVar = true,

        pubVar = false;



    function privFn() {

        return this.test; // -> Strict violation.

    }



    function pubFn() {

        this.test = 'public'; // -> Strict violation.

        privFn.call(this); // -> Strict violation.

    }



    return {

        pubVar: pubVar,

        pubFn: pubFn

    };



}());



myModule.pubFn();

I understand it's being caused by the use of this in a function declaration, but I read something Crockford wrote and he said the violation is meant to prevent global variable pollution - but the only global variable here is the one I'm explicitly defining... myModule. Everything else is held in the immediate function scope, and I should be able to use this to refer to the module.

Any ideas how I can get this pattern to pass?

Update: if I use a function expression instead of a declaration, this seems to work, ie

var pubFn = function () { ...

I'm not a fan of this format though, prefer to have the function name and named params closer and the declaration looks/feels cleaner. I honestly don't see why this is throwing the violation - there's no reason for it in this pattern.

Answers

Only authorized users can answer the question. Please sign in first, or register a free account.

mattieparisp

Add To Favorites

Follow

Total Points: 612

Total Questions: 109

Total Answers: 104

Location: Trinidad and Tobago

Member since Fri, May 8, 2020

4 Years ago

answered 13 Years ago hallie · Accepted Answer

JSHint has an option called validthis, which:

[...] suppresses warnings about possible strict violations when the code is running in strict mode and you use this in a non-constructor function [...], when you are positive that your use of this is valid in strict mode.

Use it in the function that JSHint is complaining about, which in your case, would look like this:

function privFn() {

    /*jshint validthis: true */

    return this.test; // -> No Strict violation!

}



function pubFn() {

    /*jshint validthis: true */

    this.test = 'public'; // -> No Strict violation!

    privFn.call(this); // -> No Strict violation!

}

It might seem like a pain to have to specify that in each function where it applies, but if you set the option at the top of your module function, you may hide genuine strict mode violations from yourself.