Tại sao jshint không nhận ra một bài tập như một biểu thức?

Question

Tôi cần phải sửa đổi những dòng này để làm cho jshint vui vẻ như thế nào?

Bài tập là một biểu thức. Tại sao jshint không hiểu điều này? Rõ ràng là thông dịch viên.

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

Nếu bạn thực sự muốn nghe JSHint, chuyển đổi biểu thức để một boolean bởi:

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. 

Vì vậy, !! nghĩa: Chuyển đổi một cái gì đó để các đại diện có điều kiện.

Answer 2

/*jshint boss:true */ 

Thử nghiệm với các options.

Answer 3

Tôi chắc chắn rằng jshint hiểu biểu thức tốt, nó chỉ là hầu hết những người viết if (a = b) thực sự có nghĩa là if (a == b) và do đó điều này tạo ra một cảnh báo.

Kể từ mã của bạn là những gì bạn dự định bạn có thể thêm một thử nghiệm rõ ràng:

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

Answer 4

Nó là một biểu hiện, và bạn có thể sửa đổi nó để làm việc với JSHint (mặc dù nó không đẹp) như sau:

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

Ví dụ cuối cùng của bạn. Tuy nhiên, bạn không cần phải làm điều này. JSHint chỉ là một công cụ giúp bạn cải thiện thói quen viết mã và sửa lỗi, nhưng cho rằng những gì bạn có là rõ ràng, súc tích, và (theo ý kiến ​​của tôi) là cách tốt nhất để làm điều đó - chỉ cần bỏ qua những thông điệp đó.

Answer 5

jshint không thể biết liệu bạn có thực sự muốn thực hiện nhiệm vụ trong khối điều kiện hay không hoặc liệu điều đó có thực sự được coi là so sánh hay không. Mối quan tâm sẽ là những người khác có thể có cùng một nghi ngờ.

Trong trường hợp của trình lặp, tôi nghĩ bạn ổn.

Answer 6

Tôi có lỗi này bởi vì tôi đã có một dấu phẩy trailing trong một tuyên bố trước các chức năng:

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

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

_{(Thật khó để tìm, nhưng củng cố quan điểm của tôi rằng xơ thường đúng, nó là mã của tôi đó là Nếu tôi đã vô hiệu hóa các cảnh báo trên toàn cầu - hoặc thậm chí tại chỗ đó - lỗi vẫn còn tồn tại. </lecture>)}

Answer 7

Có ít nhất hai cách để giải quyết lỗi được báo cáo, theo số JSHint docs.

1. Thêm /*jshint boss:true */ trước khi tuyên bố có điều kiện
2. tuyên bố Bọc trong ngoặc thêm, tức là while ((element_iterator = element_iterator.nextSibling)) {...}

Cá nhân, tôi nghĩ rằng xung quanh với ngoặc thêm là thực hành tốt nhất vì nó giữ việc kiểm tra lỗi nhưng vẫn làm cho tinh thần tốt mã khôn ngoan. Thêm !! trước khi thực sự không có gì ngoài việc chuyển đổi biểu thức thành true/false qua lại hai lần thêm.