2017-01-31 4 views

答えて

3

あなたは関数を呼び出し、その結果をハンドラに渡します。関数を呼び出すことなく関数を渡す必要があります。

また、実際に金額を増やす必要があります。それ以外の場合は、常にinnerHTML = 0 + 1です。

function goldclick() { 
    gold += goldperclick; 
    goldCounter.innerHTML = gold; 
} 

goldGenerator.addEventListener("click", goldclick); 

動作するはずです。この

goldGenerator.addEventListener("click", goldclick); 

2

金の変数を増やす必要があります。要素に加算の結果を割り当てるため、常に1になります。

goldGenerator.addEventListener("click", goldclick()); 

それはあなたがaddEventListenerに機能goldclickを渡すべきである

goldGenerator.addEventListener("click", goldclick); 

次のようになります。

function goldclick() { 
    //Add this 
    gold += goldperclick; 
    goldCounter.innerHTML = gold 


} 
+1

より正確には、ゴールド変数はクリックあたりの金額によって増加するはずです: gold = gold + goldperclick; goldCounter.innerHTML = gold; –

1

この行に問題があります。

あなたのコードが現在行っていることは、goldclickを呼び出して、この関数の結果をaddEventListenerに渡すことです。

1

変更この

goldGenerator.addEventListener("click", goldclick()); 

あなたはコールバックではなく、関数呼び出しの結果としてgoldClickを渡したいです。

関連する問題