Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

V2.0 #1067

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

V2.0 #1067

wants to merge 2 commits into from

Conversation

adrianocomp
Copy link

Hi,

I'm a PhD candidate and I'm studying whether static verification tools source code as JSLint and JSHint are really effective in finding problems in the source code. I checked the latest version of its source code with JSHint and JSLint and found some significant problems.

The images below show the names of errors find by JSLint and JSHint. The number next to the error, means the time of this error appears in your code.

errors jshint
errors jslint

This mistakes can lead to problems that are hard to solve. Solutions for this problems can be found here:

https://jslinterrors.com/

Do you agree with these problems? Yes or No?
You want to fix this problems?
Regards,

…r column headers. (cherry picked from commit a648d5a)"

This reverts commit 193c4d1.
@6pac
Copy link

6pac commented Jun 9, 2015

Hi adrianocomp,

I'm a little bit suspicious that you're mainly trying to get traffic to that website. But then, it's not a bad website.
I use WebStorm for js editing, which has the lint/hint tools built in. I checked it out and there were indeed about 10 reasonably serious errors it picked up.
However a lot of errors these tools flag are the result of them not understanding the syntax of more advanced javascript properly.
I'd categorise them as roughly:

  • actual correctly diagnosed syntax errors (2%)
  • harmless sloppiness errors (re-declarations, using !== to compare to null) (70%)
  • incorrect interpretation of valid code (28%)

So they are useful to a point, but can be quite annoying when you have to trawl through a bunch of incorrectly flagged errors to find the one true error. This is what tends to put people off.
I notice also that about 70% of the errors being flagged in my project are in the jQuery and other libraries. This is a sign in itself. These libraries tend to use nonstandard but highly optimised ('guru level') code structures. It would perhaps be good if the Lint user could flag their level of expertise (newbie, competent, guru) to tell the tool that suspicious but correct looking code is probably not a mistake but the work of a guru, or to at least allow different types of structure in this case.

Ben McIntyre

@adrianocomp
Copy link
Author

Hi Ben,

Thanks for comments. They are very helpful.

Regards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants