3

给定以下简单的html：

<div class="someContainer">
  <h5>Some other information</h5>
</div>

以及以下主干视图：

var view = Backbone.View.extend({
  events: {
   'click .someContainer': performAction
  },
  performAction: function (evt) { 
    // Do things here
  } 
});

我发现自己做了很多下面的代码，这对我来说似乎是一种代码味道。我做错了什么还是有更好的方法来做到这一点？

...performAction: function (evt) { 
 // Check to see if the evt.target that was clicked is the container and not the h5 (child)

 if ($(evt.target).hasClass('someContainer')) { 
  // Everything is ok, the evt.target is the container
 } else { 
  // the evt.target is NOT the container but the child element so...
  var $container = $(evt.target).parent('.someContainer');

  // At this point I now have the correct element I am looking for
 }
}

这很有效，但我不确定这是可以在任何地方编写的好代码。我可以制作一个我可以调用的方法，但我不确定它是否能真正纠正代码异味，它只是将它外包给其他地方。

4

1 回答 1

9

您可以evt.currentTarget改用：

事件冒泡阶段中的当前 DOM 元素。

演示：http: //jsfiddle.net/ambiguous/UgA5M/

或者你可以使用$container = $(evt.target).closest('.someContainer')而不用担心嵌套。

演示：http: //jsfiddle.net/ambiguous/B49LG/

您使用哪种方法取决于您的具体情况。如果您在某种控件上有一个点击处理程序，那么closest可能更有意义；如果您真的想要将单击处理程序绑定到的元素（或认为您拥有，这毕竟都是基于delegate），然后使用currentTarget.

于 2012-04-09T19:37:25.883 回答