次のコードは、index.htmlをで呼び出している

Question

jqueryの中でのonclickにパラメータを渡す方法

<script src="js/jquery/jquery-1.8.2.min.js"></script> 
    <script src="js/px.js" type="text/javascript" ></script> 
    <script type="text/javascript"> 
     var newGame = new px1("black"); 
     new px1Generator(newGame); 
    </script> 

以下のコードは、px.jsである

私は100または900を送りたい

var px1 = function (color) { 
    if (color == "white") { 
     this.x = "100"; 
    } else if (color == "black") { 
     this.x = "900"; 
    }}; 
var px1Generator= function (px1obj) { 
    for (i = 0; i < 10; i++) { 
     $("#mainGame").append("<div class='pawn white'></div>"); 
    } 
    //for 10 object upon add onClickEvent 
    $(".pawn.white").on("click", function (event, px1obj) { 
     alert(px1obj.x) 
    }); 
}; 

onclick関数、それを行う方法？ ユーザーがdivをクリックしたときのエラーの合意

Answer 1

上書きするコールバック関数の署名からpx1objパラメータを削除する必要があります。

px1objはすでにpx1Generator関数のスコープ内に存在し、そしてあなたはonclickのコールバックのスコープ内でそれをオーバーライドしています。

$(function() { 
 
    function px1(color) { 
 
    if (color == "white") { 
 
      this.x = "100"; 
 
    } else if (color == "black") { 
 
      this.x = "900"; 
 
    }}; 
 
    function px1Generator(px1obj) { 
 
     for (i = 0; i < 10; i++) { 
 
      $("#mainGame").append("<div class='pawn white'>a</div>"); 
 
     } 
 
     //for 10 object upon add onClickEvent 
 
     $(".pawn.white").on("click", function (event) { 
 
     console.log(px1obj.x) 
 
     }); 
 
    }; 
 

 
    var a = new px1("black"); 
 

 
    px1Generator(a); 
 
});

<script src="https://ajax.googleapis.com/ajax/libs/jquery/1.11.1/jquery.min.js"></script> 
 
<div id="mainGame"></div>

Answer 2

あなたはここに二つの問題があります。まず、あなたが今、あなたのコードを実行するときは、このエラーが表示されます：あなたはそれが存在する前に、機能をインスタンス化しようとしているので、

Uncaught TypeError: px1 is not a constructor

です。最初に関数を定義するか、明示的な関数定義を使用するように変更する必要があります。

第2に、決して定義されないclickイベントハンドラのパラメータとしてpx1objをパラメータとして再定義したため、問題自体です。これにより、outserスコープで定義されたpx1objが非表示になります。この問題を解決するには、単にイベントハンドラからそのパラメータを削除：

var newGame = new px1("black"); 
 
new px1Generator(newGame); 
 

 
function px1(color) { 
 
    if (color == "white") { 
 
    this.x = "100"; 
 
    } else if (color == "black") { 
 
    this.x = "900"; 
 
    } 
 
}; 
 

 
function px1Generator(px1obj) { 
 
    for (i = 0; i < 10; i++) { 
 
    $("#mainGame").append("<div class='pawn white'></div>"); 
 
    } 
 
    //for 10 object upon add onClickEvent 
 
    $(".pawn.white").on("click", function(e) { 
 
    console.log(px1obj.x) 
 
    }); 
 
};

<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.0/jquery.min.js"></script> 
 
<div class="pawn white">pawn white</div>