아이고 자바스크립트 고수분들 도와주세요!!!

Inv['plugin']['event'] 에 정의된 함수에 문제가 있습니다.

 

for(var i = 0; i < Inv.getElements('event').length; i++){

    Inv.getElements('event')[i].onclick = function() {

        if(Inv.getValue("event", i) === "refresh")

            location.reload();

    }

}

 

 

여기서 Element.onclick 함수 안에서 i 변수를 사용하셨는데,

반복문에서 사용되는 반복카운터는 함수 안에서 사용하시면 안됩니다.

 

저 i 값은 클로져 변수라 for 블럭 안에서만 사용되는 것은 맞으나, 그것이 반복할 때마다 새롭게 생성되는건 아닙니다.

이전에 실행되었던 반복코드에서도 i 값에 접근한다면, 돌아오는 값은 지금까지 진행된 카운트 수가 됩니다.

예를 들어, for(i = 0 to 10) { setTimeout( { alert(i) }, 1000); } 이라는 의사코드를 실행한다면,

출력되는 값은 어지간한 조선컴퓨터가 아닌 이상 10 이라는 숫자가 10회 출력된다는 것이죠.

 

 

하남님이 원하시는(또는 그렇게 추정되는) 방식으로 구현하려면,

저 i 값을 보존하는 변수를 하나 더 만들어서, 그 변수를 i 대신 사용해야겠죠.

 

for(var i = 0; i < Inv.getElements('event').length; i++){

    var ti = i;

    Inv.getElements('event')[ti].onclick = function() {

        if(Inv.getValue("event", ti) === "refresh")

            location.reload();

    }

}

 

같은 느낌으로요.

 

솔직히, 제가 생각하기에는 그리 이상적인 방법은 아니라고 생각합니다.

요즘 PC에서는 웬만해선 부담되지 않겠지만, 그래도 많은 Element 들이 생긴다면, 부담이 될겁니다.

해당 Attribute 가 있는 개수만큼 변수를 생성해버릴테니까요.

 

저라면 getValue 에서 사용 가능한 파라미터 종류를 하나 더 만들듯 싶네요.

파라미터의 타입이 Element 라면, getElement를 거치지 않고, 바로 파라미터에서 Attribute를 가져오는 식으로요.