Question

これはなぜ動作しないのか誰にでも教えてもらえますか？

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

firebugでは、setTimeout（）に渡されたelement_idが定義されていないというエラーが表示されます。

Answer 1

変数element_idは、文字列になります。

より良い方法は、このように、クロージャを使用することです：

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

文字列内のコードを渡すとnot recommendedであることに留意すべきです。

Answer 2

試してみてください。私はあなたがgetElementByIdに渡している文字列パラメータに引用符を追加しました

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

注意してください。 MYIDは引用符にする必要がありますが、それはないということ

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

注意：お使いのタイムアウトコードは次のようになりますので

Answer 3

これを行うための最善の方法は、閉鎖

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

である