jshint Angular'ı anlıyor mu?

Question

jshint, Açısal stil kılavuzları tarafından önerilen bir açısal modül (veya yönerge veya fabrika) tanımlarken hata veriyor (John Papa veya Todd Motto). Örneğin, böyle bir denetleyici için:

'theController' was used before it was defined. 

açısal uygulaması bu hataları rağmen mükemmel çalışıyor:

(function() { 
    'use strict'; 

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

    function theController() {...} 

})(); 

... jshint bu hatayı atar. Ancak neden jshint protesto ettiğini bilmiyorum ...

Neyi özlüyorum? Jshint'in açısal kodun kalitesinin iyi bir değerlendiricisi olup olmadığını merak ediyorum (jeneratör-açısal olarak popüler paketlere dahil olmasına rağmen) ya da ben yanlış bir şey yapıyorum (app'ım çalışmasına rağmen).

Şimdiden teşekkürler! iyi ilk

Answer 1

örneğin, "globaller değişkenler" in açısal şunlardır: Eğer köşeli onu referans önce

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

ardından işlevi tanımı taşıyın.

(function() { 
    'use strict'; 

    function theController() {...} 

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

Answer 2

Kodunuz çalışır, ancak jshint deneyin ve çok sıkı bir şekilde kodlamak için almak için gidiyor olmalıdır. En azından, işlevlerinizi kullanmadan önce tanımlamanız bir "iyi uygulama" dır. Yukarıda açıklamada belirtildiği gibi, daha yaşlı JavaScript motorları altındaki (gerçi kesin hatırlayamıyorum ve test edemez) yukarıdan yürütmek düşünüyorum - böylece geniş olarak-mümkün destek için gidiyoruz eğer isteyeceksiniz Jshint'i dinle. Burada kayda değer

şey size işlevini tanımlamak için var anahtar kelimeyi kullanırsanız, bir hata alacak - en iyi örnekle açıkladı:

Bu (http://jsfiddle.net/ryanwheale/kr8L825p/) çalışır

(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 :)"); 
    } 
})(); 

.. . Bu does not (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

Kullanım latedef mülkiyet ama veayarlayın. Bu, IMO'nun iyi olduğu işlevlerin yüklenmesine izin verir. Ancak yine de, IMP'lerin varsayılan raporlar, bu IMO

Answer 4

Bu nedenle, her linter'i mutlu kılan bir başka seçenek, önce işlevi tutacak olan değişkeni bildirmek, onu bir param olarak kullanmak ve sonra tanımlamaktır.

Ama şahsen ben buradaki akışı sevdiğimden emin değilim. Daha iyi Jack'in cevabını düşünüyorum, ama bu Papa aşağı onun style guide ile iseniz, tercih görüneni biraz daha yakın olduğunu. Bu gerçekten kullanıldıkları sonra görünmesini işlevleri istiyorsa o önerir (ve yaptığı) ne değildir neden Aslında ben emin değilim. Aksi takdirde, , stilini latedef JSHint - veya JSLint öğesinde öğesinde doğru olarak ayarlayamaz.