2016-12-05 14 views
-3

私はこのコードでstart()のみを定義すればすべて動作しますが、time()を宣言するとエラーが発生します: " TypeError:startは関数ではありません "。問題はどこだ?? `あなたが使用した、:ここで コード、Javascript、TypeError: "function"は関数ではありません

//start 
 
function start(){ 
 
\t 
 
\t //removes title and start boxes 
 
\t var body=document.getElementsByTagName("body")[0]; 
 
\t var start_box=document.getElementById("start_box"); 
 
\t var title_box=document.getElementById("title_box"); 
 
\t body.removeChild(start_box); 
 
\t body.removeChild(title_box); 
 
\t 
 
\t //creates stats box 
 
\t var stats_box=document.createElement("div"); 
 
\t stats_box.id="stats_box"; 
 
\t var time=document.createElement("p"); 
 
\t time.id="time"; 
 
\t var points=document.createElement("p"); 
 
\t points.id="points"; 
 
\t stats_box.appendChild(points); 
 
\t stats_box.appendChild(time); 
 
\t body.appendChild(stats_box); 
 
\t 
 
\t //creates play box 
 
\t var play_box=document.createElement("div"); 
 
\t play_box.id="play_box"; 
 
\t body.appendChild(play_box); 
 
\t 
 
} 
 

 
//time 
 
function time(){ 
 
\t var time=document.getElementById("time"); 
 
\t for(x=30,x>=0,x--){ 
 
\t \t time.innerHTML("Time:"+x); 
 
\t } 
 
}
<!DOCTYPE HTML> \t 
 
<html> 
 
\t <head> 
 
\t \t <title>PICK 'EM ALL</title> 
 
\t \t <link rel="stylesheet" href="pta.css" type="text/css"> 
 
\t \t <script src="pta.js" type="text/javascript"></script> 
 
\t </head> 
 
\t <body> 
 
\t \t <div id="title_box"> 
 
\t \t \t <p id="title">PICK 'EM ALL</p> 
 
\t \t </div> 
 
\t \t <div id="start_box" onclick="start()"> 
 
\t \t \t <p id="start">START</p> 
 
\t \t </div> 
 
\t </body> 
 
</html>

+9

それはちょうど誤植だのセミコロンである必要があり、forループを使用することです'の代わりに'; 'の代わりに'; 'を使うので、スクリプト全体は解析されず、定義されたものは何も生成されません。 for(x = 30; x> 0; x - ) 'ここには2つの貴重な教訓があります:1.あなたのウェブコンソールを開きます。 2.構文エラーがスクリプト全体を爆破する可能性があります。 –

+3

あなたのコードはまた、[*暗黙のグローバルの恐怖*](http://blog.niftysnippets.org/2008/03/horror-of-implicit-globals.html)に落ちています:あなたの変数を宣言してください( 'x'例えば、「時間」に)。 –

+1

OPについては、Lintingを組み込んだ良いエディタを使用することをお勧めします。これらのエラーは即座に取り上げられます。 – Keith

答えて

-1

問題は、それが代わりにコンマ

for(x=30;x>=0;x--){ 
     time.innerHTML("Time:"+x); 
    } 
+5

Typosは答えが必要ではなく、コメントだけ(私はすでにそれを行っています)と投票を終えています。 –

関連する問題