Переменная функции передана setTimeout не работает?

Question

Может кто-нибудь сказать мне, почему это не работает?

function changeBG(element_id){
        document.getElementById(element_id).className= "arrowActive";
        setTimeout("document.getElementById("+element_id+").className= 'arrow'", 300);
}

В firebug я получаю сообщение о том, что element_id, переданный setTimeout (), не определен

Answer 1

Переменная element_id будет строкой, поэтому ваш код тайм-аута будет выглядеть так:

document.getElementById(myId).className = ...

Обратите внимание, что myId должен быть в кавычках, но это не так.

Лучше использовать закрытие, например:

function changeBG(element_id)
{
        var elm = document.getElementById(element_id);
        elm.className = "arrowActive";
        setTimeout(function() { elm.className= 'arrow'; }, 300);
}

Следует отметить, что передача кода в виде строки не рекомендуется .

Answer 2

Лучший способ сделать это с закрытием

function changeBG(element_id)
{
        var elem = document.getElementById(element_id);
        elem.className= "arrowActive";
        setTimeout( function(e)
        {
                return function()
                {
                        e.className = 'arrow';
                }
        }( elem ), 300);
}

Answer 3

Попробуйте:

setTimeout("document.getElementById('" + element_id + "').className= 'arrow'", 300);

Обратите внимание, что я добавил кавычки в строковый параметр, который вы передаете getElementById.