のJavaスクリプト：私はシンプルな電卓を作ってるんだし、私の機能が動作しない.innerHTML

Question

を働いていない：

HTML：

<div id="mathLine">0</div> 
<div id="score">none</div> 
<button id="clear" class = "num" ocnlick="clear(0)">C</button> 
<button id="exp" class = "num" >exp</button> 
<button id="sqrt" class = "num">sqrt</button> 
<button id="buttonide" class = "num">/</button> 
<button id="seven7" class = "num" onclick="numerical(7)">7</button> 
<button id="eight8" class = "num" onclick="numerical(8)">8</button> 
<button id="nine9" class = "num" onclick="numerical(9)">9</button> 
<button id="multiply" class = "num" onclick="numerical('X')">X</button> 

JS

var equation = []; 
function numerical(number){ 
    equation.push(number); 
    document.getElementById('mathLine').innerHTML = equation; 
    console.log(equation); 
}; 
function clear (j) { 
    for (j; j < (equation.length+2); j++){ 
    equation.pop(); 
    } 
    console.log(equation); 
    document.getElementById('mathLine').innerHTML = "0"; 
}; 

一部詳細情報：

• jsファイルはhtmlで正しく実装されています。
• （数値）第一の機能が正常に動作しています。
• リスト項目、私はそれがすべてだから私は見当がつかない

を働いたタグでhtmlとjsファイルを置く別のHTMLファイル、でそれをしようとしたとき、何が間違っている...たぶん私はあまりにもそれを見ています長いと私は何かを見ていない明白な？ここ

Answer 1

あなたがタイプミスを持っています。 "ocnlick"を "onclick"に変更してみてください。

Answer 2

私の信念は、あなたの配列-クリアが動作していないということです - それはあなたがそれから要素を削除している間、インデックスがめちゃくちゃ得ることができるよう、配列を反復処理するために、通常は安全ではありません。あなたはおそらく、配列の長さをチェックするwhileループとforループを置き換えることもできますが、簡単な提案は、アレイを再割り当てする代わりに、次のようになります。

equation = [];