Recommendation¶

If the global variable reference is intentional, consider adding a JSLint /*global ...*/ directive or an externs declaration to declare the variable.

If the global variable reference is unintentional, qualifying the reference with exports will make it refer to the exported definition instead.

Example¶

In the following example, the module exports two functions checkOne and checkList. The latter is also stored in a variable of the same name that is local to the module, but the former is not. Hence the call checkOne(xs[i]) on line 7 does not refer to the function defined on line 1, but to an otherwise undeclared global variable also called checkOne.

exports.checkOne = function(x) {
  if (!x) throw new Error();
};

var checkList = exports.checkList = function(xs) {
  for (var i=0; i<xs.length; ++i)
    checkOne(xs[i]);
};

Assuming that the intention is to call the checkOne function defined on line 1, the call should be qualified with exports like this:

exports.checkOne = function(x) {
  if (!x) throw new Error();
};

var checkList = exports.checkList = function(xs) {
  for (var i=0; i<xs.length; ++i)
    exports.checkOne(xs[i]);
};

References¶

• Node.js: Modules.

• JSLint Help: JSLint Directives.

• Closure Compiler: Advanced Compilation and Externs.