Значит ли jshint Angular?

Question

jshint выдает ошибку при определении углового модуля (или директивы или фабрики), как рекомендовано направляющими углового типа (по John Papa или Todd Motto). Например, для контроллера, как это:

(function() { 
    'use strict'; 

    angular 
     .module('myApp') 
     .controller('myAppCtrl', theController); 

    function theController() {...} 

})(); 

... jshint выдает эту ошибку:

'theController' was used before it was defined. 

Угловая приложение работает отлично, несмотря на эти ошибки. Однако я не знаю, почему протесты jshint ...

Что мне не хватает? Интересно, является ли jshint хорошим оценщиком качества углового кода (несмотря на то, что он включен в популярные пакеты как генератор-угловые), или это я, что я делаю что-то неправильно (хотя мое приложение работает).

Заранее благодарен!

Answer 1

хорошо первый из включают в себя угловой в ваших «глобалам переменных», например:

"globals": { // Global variables. 
     "jasmine": true, 
     "angular": true, 
     "browser": true, 
     "element": true, 
     "by":true, 
     "io":true, 
     "_":false, 
     "$":false 
    } 

затем переместить определение функции, прежде чем ссылаться на него угловой.

(function() { 
    'use strict'; 

    function theController() {...} 

    angular 
     .module('myApp') 
     .controller('myAppCtrl', theController); 
})(); 

Answer 2

Ваш код должен работать, но jshint будет пытаться заставить вас код в очень строгой манере. По крайней мере, это «хорошая практика», чтобы ваши функции были определены до их использования. Как я уже упоминал в комментарии выше, я думаю, что более старые javascript-движки выполняются сверху донизу (не помню наверняка, хотя и не могут протестировать) - поэтому, если вы собираетесь поддерживать как можно большую поддержку, вам нужно будет послушайте jshint.

Что-то здесь следует отметить, что если вы используете var ключевое слово, чтобы определить свою функцию, вы получите ошибку - лучше всего объяснить на примере:

Это работает (http://jsfiddle.net/ryanwheale/kr8L825p/)

(function() { 
    try { 
     foo(); 
    } catch(ex) { 
     alert("ahhhhh, what's going on?!?!?\n\n" + ex.message); 
    } 

    function foo() { 
     alert("I was hoisted to the top of this scope before execution :)"); 
    } 
})(); 

.. ., но это не (http://jsfiddle.net/ryanwheale/kr8L825p/4/)

(function() { 
    try { 
     foo(); 
    } catch(ex) { 
     alert("ahhhhh, what's going on?!?!?\n\n" + ex.message); 
    } 

    var foo = function() { 
     alert("I was hoisted to the top of this scope before execution :)"); 
    } 
})(); 

Answer 3

Используйте latedef свойство и установить его на false. Это позволяет поднимать функции, которые ИМО в порядке. Но все же сообщается о подъеме варов, что плохо. IMO

Answer 4

Таким образом, еще один вариант, который делает каждый счастливый листинг, заключается в объявлении переменной, которая сначала будет удерживать функцию, использовать ее как параметр, а затем определить ее.

Но лично я не уверен, что мне нравится поток здесь. Я думаю, мне нравится ответ Джек лучше, но это немного ближе к тому, что, по-видимому, предпочитает папа, если вы с его style guide. На самом деле, я не уверен, почему это не то, что он рекомендует, если он действительно хочет, чтобы функции появлялись после того, как они были использованы (и он это делает). В противном случае вы не можете использовать его стиль с latedef установлен в true в JSHint - или JSLint на всех.

(function() { 
    'use strict'; 
    var theController; 

    angular 
     .module('myApp') 
     .controller('myAppCtrl', theController); 


    theController = function() { 
     return "so jslint doesn't complain about empty blocks"; 
    }; 
}());