我有两组几乎相同的事件处理程序。有没有办法将它们组合起来使代码更干?
this.E.name
只是一个指向DOM元素的指针/引用。
请注意,我的ID不同于事件类型,因为可能的副本不适用。
/*set 1
*/
$(this.E.main).on("click", function () {
Page.flip('main');
});
$(this.E.feed).on("click", function () {
Page.flip('feed');
});
$(this.E.so_button).on("click", function () {
Backbone.trigger('user_sign_out');
Pane.flip();
Page.flip();
Storage.clear();
});
/* set 2
*/
$(this.E.main_nav).on("click", function () {
Page.flip('main');
});
$(this.E.feed_nav).on("click", function () {
Page.flip('feed');
});
$(this.E.so_button_nav).on("click", function () {
Backbone.trigger('user_sign_out');
Pane.flip();
Page.flip();
Storage.clear();
});
答案 0 :(得分:2)
你可能会更棘手,只是迭代对象
var valid = ['main', 'feed', 'so_button']; // in case there are other props in object
$.each(this.E, function(key, value) {
if ( valid.indexOf(key) !== -1 || valid.indexOf(key + '_nav') !== -1 ) {
$(this.E[key]).on('click', function() {
if ( key.indexOf('so_button') !== -1 ) {
Backbone.trigger('user_sign_out');
Pane.flip();
Page.flip();
Storage.clear();
} else {
Page.flip(key)
}
});
}
});
答案 1 :(得分:1)
您可以使用逗号,
分隔符和方括号[]
来组合多个选择器,并使用$(this)
来引用所点击的当前对象:
$([this.E.main, this.E.feed, this.E.main_nav, this.E.feed_nav]).on("click", function () {
Page.flip($(this).attr('name'));
});
$([this.E.so_button, this.E.so_button_nav]).on("click", function () {
Backbone.trigger('user_sign_out');
Pane.flip();
Page.flip();
Storage.clear();
});
希望这有帮助。
答案 2 :(得分:1)
你可以创建一个实用程序javascript函数/对象,它为事件中的每个例程都有一个方法。一个衬里仍然是一个衬里,而四个衬里将成为一个衬里。您不会大大减少代码量,但维护会更好,未来的可维护性会更好,尤其是当更多逻辑添加到一个衬里时。
答案 3 :(得分:0)
试试这个
var mainId = "#" + this.E.main.id;
var mainNavId = "#" + this.E.main_nav.id;
$(mainId + "," + mainNavId ).on("click", function () {
Page.flip('main');
});