Question

を働いていない私はこれがあります。

function tp_visible(action){ 
    if(action==1){ 
     document.getElementById("tp").style.display='block'; 
     document.getElementById("tp_action").onclick='tp_visible(0); 
     return false;'; 
    } 
    else { 
     document.getElementById("tp").style.display='none'; 
     document.getElementById("tp_action").onclick='tp_visible(1); 
     return false;'; 
    } 
    return false; 
} 

なぜonclickイベントを変更する上ではありませんか？ここで

私は Firebugを使用すると、イベントが同じまま

...

はHTMLです：

<a 
    id='tp_action' 
    name='tp_action' 
    href='#' 
    onclick='tp_visible(1); 
    return false;' 
>Show info</a> 

Answer 1

あなたはハンドラに文字列値を割り当てることはできませんので、下記のチェックがあなたのために動作しますonclick 。

function tp_visible(action){ 
    if(action==1){ 
     document.getElementById("tp").style.display='block'; 
     document.getElementById("tp_action").onclick= function() 
      { tp_visible(0); return false;}; 
    } 
    else { 
     document.getElementById("tp").style.display='none'; 
     document.getElementById("tp_action").onclick= function() 
      {tp_visible(1); return false; } 
    } 
    return false; 
} 

Answer 2

あなたはJavascriptで出始めている場合、私はあなたが多くの優れたのJavaScriptライブラリの一つでaquainted、決して再び自分のためにこのようなものを書いてみる取得をお勧め - それはあなたの心を失うことにつながります。

• YUI
• JQuery
• DoJo

また、あなたは、Javascriptの良いビットを購入し、Douglas Crockfordによってビデオのセットを見ることより悪い行うことができます。