왜이 자바 스크립트가 올바르지 않습니까?

Question

내 IDE, RubyMine은,이 "잠재적으로 잘못된"입니다 말합니다.

내 IDE가 var top_up_miles 라인, " this의 무효 가능성이 사용하는 자바 스크립트 this위한이 검사에 말한다

function update_top_up_prices_via_localStorage(){ 
    var index = localStorage.getItem("volxs"); 
    $('.product-inner').each(function(){ 

     var product = this.getAttribute('data-product-miles'); 
     var cpm = ((localStorage.getItem("cpm_by_volxs_"+product)).split(","))[index]; 

     $(this).find('table tbody tr td').each(function(){ 
      var top_up_miles = Number(this.getAttribute('data-topup-miles')); 
      var price = Number(top_up_miles * cpm * 1.06); 
      price = price.toFixed(2); 
      $(this).text('\u00A3'+price); 
     }) 
    }); 
} 

이 같이 폐쇄에 동일로 외부 상황.

JS는?이 쓰기 수있는 더 나은 방법이 있나요?

. 그것은 IE8에서 작동하지 않습니다. 크롬과 IE9에서 잘 작동이 잘못인가

Answer 1

RubyMine은 $().each이 지정된 this 컨텍스트로 전달 된 함수를 실행한다는 것을 모릅니다. 당신이 그 경고를 피하려면,

$(/* ... */).each(function (i, elem) { 
    elem.getAttribute(/* ... */); 
}); 

을 사용하는 등

당신의 코드가 IE8에서 작동하지 않는 이유는 아마도 그와 아무 상관이있다

. 그러나 더 큰 호환성를 들어, 대신 네이티브 API 함수의 jQuery를 감싸 기능을 사용

Answer 2

그것은 당신이 자바 스크립트에서 this 키워드를 사용하여 조심해야한다는 경고에 관심을 그립니다 $(this).attr 대신 (감사 @freakish) this.getAttribute의로 this은 항상 함수 소유자를 나타냅니다.

자세한 내용은 this article on quirksmode on the JavaScript this keyword을 참조하십시오.