キャッチされていないReferenceError：calcModが定義されていません

Question

このD & Dキャラクタージェネレーターアプリを作成しています。私はロールスロイスとダイスを2つ持っています。これは、ランダムにキャラクタを生成するか、自分の統計を入力する方法です。これらの関数がスクリプトの最後にHTMLに書き込むように、私は自分のコードをリファクタリングしました。しかし、私は情報が配置されるための前提条件があります。例えば、ケツ変数（敏捷性、憲法など）は、あなたがサイコロを振るかどうか、あるいはサイコロを保つかどうかによって異なります。私は事を投げ捨てていると思うif/else文を入れました。ここにコードがあります。

//Variables 
var dex; 
var con; 
var str; 
var intel; 
var wis; 
var cha; 
var hitPoints; 

if($("#RollTheDice").click()) { 
    dex = statRoll(); 
    con = statRoll(); 
    str = statRoll(); 
    intel = statRoll(); 
    wis = statRoll(); 
    cha = statRoll(); 
}else if ($("KeepTheDice").click()) { 
    dex = +document.getElementById("Dex").value; 
    con = +document.getElementById("Con").value 
    str = +document.getElementById("Strength").value; 
    intel = +document.getElementById("Intel").value; 
    wis = +document.getElementById("Wis").value; 
    cha = +document.getElementById("Cha").value; 
    hitPoints = +document.getElementById("HP").value; 
} 

//Modifiers 
var dexmod = calcMod(dex); 
var conmod = calcMod(con); 
var strmod = calcMod(str); 
var intelmod = calcMod(intel); 
var wismod = calcMod(wis); 
var chamod = calcMod(cha); 

今、私がやっていることはかなり正しいとは知っていますが、何がわかりません。私はそれがif/elseステートメントであると想像します。

私が手にエラーは誰が義務付けるかどう私はここで少しこだわっている

Uncaught ReferenceError: calcMod is not defined 
Anonymous Function 

です。

Answer 1

簡単な答えは、calcMod()がコード内に定義されていないように見えます（少なくとも呼び出された場所のコンテキストではない）。また、*.click()は、その要素がクリックされていないかチェックしません。あなたは（私は信じている）if-statementを維持し、呼び出しボタンの名前のために評価することができます

<button onclick="someFunction()">Click Here!</button> 

：あなたは、関数を呼び出しますあなたのボタンで呼び出されるのOnClick（）イベント、のような何かをしたいです。

最後に、上記のメモを統合するだけで、同じ名前の変数を保持してください。それがあなたが使いたいものなら、どこにでも3文字の略語をつけてください。それはちょうど良い形です。

Answer 2

ifを使用して、Javascript/jQueryのクリックに反応することはありません。関数を.click()に渡し、誰かが要素をクリックするとその関数が呼び出されます。

$("#RollTheDice").click(function() { 
    dex = statRoll(); 
    con = statRoll(); 
    str = statRoll(); 
    intel = statRoll(); 
    wis = statRoll(); 
    cha = statRoll(); 
}); 
$("KeepTheDice").click(function() { 
    dex = +document.getElementById("Dex").value; 
    con = +document.getElementById("Con").value 
    str = +document.getElementById("Strength").value; 
    intel = +document.getElementById("Intel").value; 
    wis = +document.getElementById("Wis").value; 
    cha = +document.getElementById("Cha").value; 
    hitPoints = +document.getElementById("HP").value; 
});