Dlaczego jshint nie rozpoznaje przypisania jako wyrażenia?

Question

Jak muszę zmodyfikować te linie, aby jshint był szczęśliwy?

Przypisanie jest wyrażeniem. Dlaczego jshint nie rozumie tego? Oczywiście, że tłumacz.

Line 572: while(bookmark_element=bookmark_list[iterator++]) 

Expected a conditional expression and instead saw an assignment. 


Line 582: while(bookmark_element=bookmark_list[iterator++]) 

Expected a conditional expression and instead saw an assignment. 


Line 623: while(element_iterator=element_iterator.nextSibling) 

Expected a conditional expression and instead saw an assignment. 

Answer 1

Jeśli naprawdę chcesz słuchać JSHint, konwertowanie wyrażenie do wartości logicznej przez:

while (!!(bookmark_element=bookmark_list[iterator++])) 

! means: Something that evaluates to true is converted to false, 
     something that evaluates to false is converted to true. 

Więc !! oznacza: Konwersja coś do reprezentacji warunkowej.

Answer 2

/*jshint boss:true */ 

Eksperymentuj z options.

Answer 3

Jestem pewien, że jshint rozumie wyrażenie dobrze, po prostu większość ludzi, którzy piszą if (a = b), faktycznie oznacza if (a == b), a to generuje ostrzeżenie.

Ponieważ kod jest co zamierzałeś można dodać wyraźnie Test:

while ((element_iterator = element_iterator.nextSibling) !== null) { ... } 

Answer 4

Jest wyrazem i można go zmodyfikować, aby pracować z JSHint (chociaż to nie jest miłe) tak:

while(element_iterator.nextSibling) { 
    element_iterator = element_iterator.nextSibling; 

Dla Twojego ostatniego przykładu. Jednak nie musisz tego robić. JSHint to tylko narzędzie, które pomoże ci ulepszyć nawyki dotyczące kodowania i poprawić błędy, ale biorąc pod uwagę, że to, co masz, jest jasne, zwięzłe i (moim zdaniem) najlepszy sposób robienia tego - po prostu zignoruj ​​te wiadomości.

Answer 5

jshint nie może stwierdzić, czy naprawdę chciałeś wykonać zadanie w bloku warunku, czy też rzeczywiście miało to być porównanie. Obawiano się, że inni ludzie mogą mieć takie same wątpliwości.

W przypadku iteratora, myślę, że wszystko w porządku.

Answer 6

miałem ten błąd, bo miałem z tyłu przecinek w deklaracji poprzedzających funkcję:

this.foo = "bar", // <- Error was here 

this.myfunc = function() { 
    ... // <- Error reported on this line 
}; 

_{(trudno było znaleźć, ale wzmacnia moją opinię, że Linters są zwykle rację, to jest mój kod, który jest źle Gdybym wyłączył ostrzeżeń globalnie. - lub nawet w tym miejscu - bug nadal byłby obecny </lecture>)}

Answer 7

Istnieją co najmniej dwa sposoby, aby rozwiązać zgłoszony problem, według JSHint docs..

1. Dodaj /*jshint boss:true */ przed instrukcją warunkową
2. Wrap oświadczenie w dodatkowych nawiasach, tj while ((element_iterator = element_iterator.nextSibling)) {...}

Osobiście uważam, że otaczający z dodatkowych nawiasach jest najlepsza praktyka, ponieważ utrzymuje kontrolę błędów, ale nadal ma sens pod względem kodu. Dodanie do tego !! w rzeczywistości niczego nie zmienia, lecz przekształca wyrażenie na true/false dwa razy.