Why is JSHINT complaining that this is a strict violation?

by

JSHint says “Possible strict violation” because you are using this inside something that, as far as it can tell, is not a method.

In non-strict mode, calling gotoPage(5) would bind this to the global object (window in the browser). In strict mode, this would be undefined, and you would get in trouble.

Presumably, you mean to call this function with a bound this context, e.g. gotoPage.bind(myObj)(5) or gotoPage.call(myObj, 5). If so, you can ignore JSHint, as you will not generate any errors. But, it is telling you that your code is unclear to anyone reading it, because using this inside of something that is not obviously a method is quite confusing. It would be better to simply pass the object as a parameter:

function gotoPage(sorter, s) {
    if (s <= sorter.d && s > 0) {
        sorter.g = s;

        sorter.page((s - 1) * sorter.p.size);
    }
}

function pageChange(event, sorter) {
    var dd = event.currentTarget;
    gotoPage(sorter, dd[dd.selectedIndex].value);
}

More Related Contents:

  1. JavaScript function order: why does it matter?
  2. Should I use JSLint or JSHint JavaScript validation? [closed]
  3. What does "use strict" do in JavaScript, and what is the reasoning behind it?
  4. Why avoid increment (“++”) and decrement (“–“) operators in JavaScript?
  5. What is the difference between `new Object()` and object literal notation?
  6. How to initialize an array’s length in JavaScript?
  7. How to fix jslint error ‘Don’t make functions within a loop.’?
  8. Turning off eslint rule for a specific line
  9. Immediate function invocation syntax
  10. (…()) vs. (…)() in javascript closures [duplicate]
  11. How to split a long regular expression into multiple lines in JavaScript?
  12. What’s wrong with var x = new Array();
  13. What is array literal notation in javascript and when should you use it?
  14. JSLint is suddenly reporting: Use the function form of “use strict”
  15. Why should you not use Number as a constructor? [duplicate]
  16. JSlint error ‘Don’t make functions within a loop.’ leads to question about Javascript itself
  17. JSlint: unexpected ‘for’ [duplicate]
  18. JSLint error: Move all ‘var’ declarations to the top of the function
  19. Not recommended to use “use strict” in ES6?
  20. Don’t make functions within a loop [duplicate]
  21. JSLint “insecure ^” in regular expression
  22. How to disable the warning ‘define’ is not defined using JSHint and RequireJS
  23. How to remove global “use strict” added by babel
  24. JSLint Error: Unexpected ‘this’
  25. Get current function name in strict mode
  26. What does the JSLint error ‘body of a for in should be wrapped in an if statement’ mean?
  27. Strict Violation using this keyword and revealing module pattern
  28. JSLint: was used before it was defined
  29. JSLint: Using a function before it’s defined error
  30. JSLint: control comments (selective ignore)

Leave a Comment