使用trigger()进行重复事件处理是不好的做法吗？

Question

我经常使用jQuery .trigger()来处理重复事件.

例如,如果我有一个具有四个导航按钮的小部件(向上,向下,向左,向右),并且每个按钮上都绑定了一个事件处理程序,如:

$('#up').on('click', function() {
    // move up
    ...
});

到目前为止,小部件运行良好,现在我想添加键盘快捷键,up/down/left/right arrow key使每个按钮的工作方式与up/down/left/right button按下时完全相同.

在这种情况下,我经常使用.trigger()键盘快捷键作为这些按钮的包装.

$(document).on('keydown', function(e) {
    // up arrow key
    if (e.keyCode == 38) $('#up').trigger('click');
    ...
});

这样使用是不好的做法.trigger()吗？

我认为将图书馆作为黑匣子处理时非常方便.如果已经有一些按钮来帮助控制库,我可以通过触发这些按钮上的点击事件来使用库的功能,即使不知道库的API.

Answer 1

这不是很糟糕的练习,但还有其他方法可以做到,即使它们不一定更好,例如:

$('#up').on('click', clickHandler);

$(document).on('keydown', function(e) {
    if (e.which === 38) clickHandler();
});

function clickHandler() {
    //do stuff
}

要么

$(document).on('click keydown', function(e) {
    if (e.type == 'keydown' && e.which === 38) {
         // do stuff for key
    }else if (e.type == 'click' && e.target.id == 'up') {
         // do stuff when click on #up
    }
});

我认为这是基于意见的,但我更喜欢第一个,你调用的外部函数,而不是触发事件本身,即使他们做同样的事情.

我不会使用最后一个,因为它被委托并触发任何点击等.