简体   繁体   中英

JSLint: Using a function before it's defined error

I'm using JSLint to verify most of my external Javascript files, but the largest amount of errors I'm getting is from functions being used before they're defined.

Is this really an issue I should worry about ?

It seems Firefox, IE7 and Chrome don't care. Functions like the popular init() (which I use often) normally stick at the top as that makes sense to me (I like to pretend it's analogous to main() ) will, according to JSLint, need to be pushed to the bottom of the file.

由于这是Google收视最高的命中,而其他人最初可能不会在jslint工具中看到它,因此有一个名为“ Tolerate misordered definitions”的选项可以让您隐藏这种类型的错误。

/*jslint latedef:false*/

If you declare functions using the function keyword, you can use them before they're declared. However, if you declare a function via another method (such as using a function expression or the Function constructor), you have to declare the function before you use it. See this page on the Mozilla Developer Network for more information.

Assuming you declare all your functions with the function keyword, I think it becomes a programming-style question. Personally, I prefer to structure my functions in a way that seems logical and makes the code as readable as possible. For example, like you, I'd put an init function at the top, because it's where everything starts from.

If you're using jshint you can set latedef to nofunc , which will ignore late function definitions only.

Documentation - http://www.jshint.com/docs/options/#latedef

Example usage:

/* jshint latedef:nofunc */

noop();

function noop() {}

Hope this helps.

From jslint's website ( http://www.jslint.com/lint.html ), you can read about a /*global*/ directive that allows you to set variables that are assumed to be declared elsewhere.

Here is an example (put this at the top of the file):

/*global var1,var2,var3,var4,var5*/

The :true :false is not actually needed from my experience, but it looks like it's recommended from what I read on the site.

Make sure the initial global statement is on the same line as /* , or else it breaks.

To disable this warning in jshint for all files, place this in your .jshintrc file:

{
   "latedef": false
}

在您的.jshintrc文件中,设置:

  "latedef": "nofunc",

it is very unfortunate the latedef option was removed. This is essential when trying to create a 'class' with an interface at the top, ie,

function SomeClass() {
   var self = this;
   self.func = func;

   function func {
      ...
   }
}

This style is very common but does not pass jsLint because func is 'used' before being defined. Having to use global for each 'member' function is a total pain.

You can always declare the offending function at the top

eg: var init;

.... but then you'll have to remove the "var" when you get to the true definition further down:

init = function() { };

The technical post webpages of this site follow the CC BY-SA 4.0 protocol. If you need to reprint, please indicate the site URL or the original address.Any question please contact:yoyou2525@163.com.

 
粤ICP备18138465号  © 2020-2024 STACKOOM.COM