Báo cáo lỗi JSLint - Có vấn đề gì với điều này?

Question

Tôi gặp lỗi này và không biết nguyên nhân có thể là gì. Bất kỳ ý tưởng?

Sự cố ở dòng 2127 ký tự 18: Xấu trong biến thể thao. cho (môn thể thao trong sugested_sports)

   // make array 
     var sugested_sports = data.split(","); 

      // pre build DIV 
     var sporty_items = ''; 
     for (sport in sugested_sports) 
     { 
      if (sugested_sports.hasOwnProperty(sport)) { 
       sporty_items += '<a href="#'+identifier[1]+'">'+sugested_sports[sport]+'</a>'; 
      } 
     } 
      // insert DIV 
     DIVsuggestions.html(sporty_items); 

thx rất nhiều.

Answer 1

Hãy thử

var sport; 
for (sport in sugested_sports) 

này sẽ chăm sóc của các khai báo biến mất và đặt nó bên ngoài vòng lặp for (xem jsLint error "Cannot set property 'first' of undefined").

Answer 2

Câu trả lời của Pointy có lẽ là câu trả lời mà bạn đang phàn nàn.

---

Theo nguyên tắc chung, bạn nên cẩn thận khi sử dụng for (... in ...). Mọi người thường nhầm lẫn cấu trúc này với foreach từ C# hoặc các khái niệm tương tự khác bằng các ngôn ngữ khác, trong khi thực tế nó không liên quan. Cấu trúc javascript for in lặp lại mọi thành viên của đối tượng - thay vì chỉ các giá trị trong một bộ sưu tập - bao gồm các phương thức và thuộc tính. Hành vi này thường có thể dẫn đến các tác dụng phụ không mong muốn nếu bạn không biết nó hoạt động như thế nào trước.

Ví dụ:

x = ['one', 'two']; 
for (var value in x) { 
    alert(value); 
} 

Đó mang lại hai cảnh báo, các contaning đầu tiên 0 và thứ hai 1, đáng chú ý là chỉ số của bộ sưu tập.

Nếu chúng ta thay đổi điều đó lên một chút:

x = ['one', 'two']; 
x.method = function() {}; 
for (var value in x) { 
    alert(value); 
} 

Chúng tôi kết thúc với ba cảnh báo thời gian này, 0, 1, và method. Đây là hành vi bất ngờ mà tôi đang đề cập đến. Thật tuyệt khi sử dụng in nếu bạn biết nó làm gì, nhưng tôi đã thấy nó bắt được nhiều người hơn một lần.

Các công trình sau đây với cả hai ví dụ:

x = ['one', 'two']; 
for (var i = 0; i < x.length; i++) { 
    alert(i); 
} 

Answer 3

var sugested_sports = data.split(","), 
    sport, 
    sport_items = ''; 

    for (sport in sugested_sports) 
    { 
     // 
    } 

Answer 4

Tất cả các lỗi có nghĩa là trong JSHint/JSLint là bạn không tuyên bố chìa khóa của bạn/iterator biến. Như @Christopher suggests, JSLint muốn bạn phải khai báo nó ở phía trên của phạm vi của nó (google JavaScript hoisting để biết thêm về treo, like this link):

/*global data, identifier, DIVsuggestions */ 
// We'll pretend all of the above were passed in from a function's parameters 
// by using JSLint's "global" keyword -- now you can paste this code into 
// jslint.com and have it pass muster. 

// make array 
var sugested_sports = data.split(","), 
    sporty_items = '', // pre build DIV 
    sport; // <<<< **** DECLARE YOUR "KEY" HERE **** 

for (sport in sugested_sports) 
{ 
    if (sugested_sports.hasOwnProperty(sport)) { 
     sporty_items += '<a href="#'+identifier[1]+'">' 
      +sugested_sports[sport]+'</a>'; 
    } 
} 
// insert DIV 
DIVsuggestions.html(sporty_items); 

này bad for in variable lỗi ở đây giảm xuống còn giống như một lỗi 'sport' was used before it was defined nơi khác.

---

EDIT: Nó đáng nói rằng nếu for bạn đang ở trong một chức năng nội bộ, bạn cần phải khai báo biến for in của bạn trong bối cảnh đó tương tự. JSLint sẽ khiếu nại nếu bạn khai báo for in trong ngữ cảnh gốc.

Ví dụ:

function spam(d) 
{ 
    var fnTest, row; // `row` is defined "too early" 

    fnTest = function (data) { 
     for (row in data) 
     { 
      if (data.hasOwnProperty(row)) 
      { 
       console.log(data.row); 
      } 
     } 
    }; 

    fnTest(d); 
} 

Để thực hiện những điều hạnh phúc, di chuyển row vào chức năng nội bộ. Mặc dù về mặt kỹ thuật vẫn còn trong phạm vi, JSLint không thích "superscope" đã được sử dụng trước đó.

function spam(d) 
{ 
    var fnTest; 

    fnTest = function (data) { 
     var row; // and JSLint is happy! ;^D 
     for (row in data) 
     { 
      if (data.hasOwnProperty(row)) 
      { 
       console.log(data.row); 
      } 
     } 
    }; 

    fnTest(d); 
} 

---

Bằng cách này, mối quan tâm của James được bao phủ bởi hasOwnProperty séc OP đã chèn vào. Đi ra kiểm tra đó, và JSLint sẽ khiếu nại, "Cơ thể của một trong nên được gói trong một tuyên bố nếu để lọc các thuộc tính không mong muốn từ nguyên mẫu". Here's a little more on hasOwnProperty with for... in, if you're interested.