JQuery点击事件多次触发

时间:2012-05-01 01:25:45

标签: javascript jquery event-handling click

我知道已经有很多关于点击事件被点燃的事情,我想我已经阅读了所有内容,但仍然看不出这里出了什么问题。

希望我完全错过了其他人可以轻易拿起的东西......

一些背景

我的代码在企业社交网络平台内部工作,并为内容分析创建了一个BI仪表板(大约1000行的内容,主要是特定于域的,所以太多了,不能全部发布)。

导致我悲痛的部分是构建仪表板可视化本身的功能。

这里......

function makePage(){
 $("#policyCount").text(policyCount);
  var docTypes=getGlobalDocTypes(polOwners); //returns a constrained vocab array
  var statusTypes=getGlobalStatusTypes(polOwners); //returns a constrained vocab array
  $.each(polOwners,function(){ // polOwners is a global array that contains the BI data to be visualised
   html=""
   var ownerName = this.name.split(":")[1]; // name is a str in format "Owner:HR"
   html += "<div id='" + ownerName + "' class='ownerData'>";
   html += "<div class='ownerHeading'>" + ownerName + "</div>";
   html += this.policies.length + " Policy documents maintained<br />"; // policies is an array of docs managed by owner

   divIDReview = "dboard_" + ownerName + "reviewchart";
   html += "<div id='" + divIDReview + "' class='dboardelement'></div>";

   divIDType = "dboard_" + ownerName + "typechart";
   html += "<div id='" + divIDType + "' class='dboardelement'></div>";

   divIDStatus = "dboard_" + ownerName + "statuschart";
   html += "<div id='" + divIDStatus + "' class='dboardelement'></div>";

   html += "<div id='" + ownerName + "ToggleTable' class='toggletable' owner='" + ownerName + "'>";
   html += "Click to display all " + ownerName + " documents<br /></div>";
   html += "<div id='" + ownerName + "polTable' class='poltable'>";
   html += getPolTable(this.policies); // Returns an HTML table of doc metadata
   html += "</div>";

   html += "</div>";
   $("#owners").append(html); // When this function is called #owners is an empty div

   $(".toggletable").mouseover(function(){
     $(this).css({'cursor':'pointer','text-decoration':'underline'});
    });

   $(".toggletable").mouseout(function(){
     $(this).css( {'cursor':'default','text-decoration':'none'});
    });


   $(".toggletable").each(function(i, elem){
    $(elem).click(function(){
      if ($(this).next(".poltable").css("display")=="none"){
       // Currently hidden - so show
       if (debug){console.log($(this).attr("id") + " was clicked")}
       $(this).html("Click to hide " + $(this).attr('owner') + " documents<br/>");
       $(this).next(".poltable").css("display","block");
       } else {
       if (debug){console.log($(this).attr("id") + " was clicked")}
       $(this).html("Click to display all " + $(this).attr('owner') + " documents<br    />");
       $(this).next(".poltable").css("display","none");
       }      
    });
   });

   // the next section calls functions that use the Google vis api to draw  pie charts

   drawPie(300,200, "Review Status", "Status", "Policies",    getReviewStatus(this.policies), ["green","orange","red"], divIDReview); 

   drawPie(300,200, "Document Types", "Type", "Docs", getDocTypes(this.policies, docTypes), [], divIDType); 


   drawPie(300,200, "Document Status", "Status", "Docs", getStatusTypes(this.policies, statusTypes), [], divIDStatus); 
 }); 
 }

希望这足以说明问题。

您将看到代码为每个polOwner构建一个仪表板显示,其中包含三个饼图和一个隐藏或显示基础数据表的选项。

我首先将点击事件应用于.toggletable课程。当多次触发时,我使用.each的另一个答案中描述的方法将唯一事件附加到类的每个实例。

那么,会发生什么?

目前有9个polOwners乍一看,click事件似乎只是切换每个其他表的显示状态。然而,控制台日志显示这是因为它针对第一个实例触发9次,针对第二个实例触发8次,针对第三个实例触发7次等等,其中奇数使表格处于备用状态(当此工作时,显示将更改为一个.toggle动画。

有关信息,虽然我是文本编辑人员,但我确实拥有MS Expression Web 4的副本,这是一个用于错误检查HTML的有用工具。我已经粘贴了整个生成的标记(近4000行)的副本,并且看不到任何错误的嵌套或结构错误。

任何想法的人?

1 个答案:

答案 0 :(得分:7)

你有一些嵌套循环:

// jQuery each on polOwners
$.each(polOwners,function(){
    // ... code that appends .toggletable class 

    // jQuery each on .toggletable class
    $(".toggletable").each(function(i, elem){
        // code that runs on the toggletable element
    });
});

对于每个polOwner,您要添加一个带有toggletable类的div。然后在那里你用toggletable类循环遍历每个div并添加一个click事件。

这为第一个polOwner增加了1次点击,为第二个polowner增加了2次,为第三个增加了3次,依此类推。

每个toggletable移出polOwner,你应该很好