一些东西:
大多数人会提出类似var self = this
的建议,因为它既快速又简单。
但是var self = this
并没有将视图对象与视图逻辑完全分开,视图逻辑来自更正式的 C# 背景并查看您的代码,听起来像是您想做的事情。
为了让回调仅在事件触发时执行,请将处理程序包装在一个函数中,以便立即对其进行评估,但仅在keydown
事件触发时执行(参见下面的代码)。
理解 JS 中的作用域:无论执行上下文是什么,也是当前作用域。您的侦听器已添加到 on 的方法(称为listen
)中Keyboard.prototype
,但keydown
实际上触发了该事件window
-处理程序在与其定义位置不同的上下文中执行;它在调用它的上下文中执行,在这种情况下,window
它的范围是 ,除非您通过或在定义它时window
将它绑定到另一个对象。bind
apply
在您的代码中,window
是用户与之交互的视图,并且Keyboard
是该视图的控制器。在 MVC 模式中,就像您在 C#/.NET 中可能习惯的那样,视图不会告诉自己当事情发生时要做什么,控制器会告诉视图要做什么。因此,如果您要像许多人那样使用分配对控制器的引用var self = this
,则视图将自行管理——但仅适用于keydown
事件的特定处理程序。这是不一致的,并且在大型项目中将变得难以管理。
一个办法:
Keyboard.prototype.listen = function() {
window.addEventListener('keydown', function(e) {
this.handle_keydown(e);
}.bind(this), false);
}
更好的解决方案:
Keyboard.prototype.view = window;
Keyboard.prototype.listen = function() {
this.view.addEventListener('keydown', function(e) {
this.handle_keydown(e);
}.bind(this), false);
}
最好的解决方案(直到 ES6class
准备好):
// define
function addViewController(view) {
function ViewController() {
this.handle_keydown = function(args) {
// handle keydown events
};
this.listen = function() {
this.view.addEventListener('keydown', function(e) {
this.handle_keydown(e);
}.bind(this), false);
};
this.view = view;
return this;
}
return new ViewController(view);
}
// implement
var keyboard = addViewController(window);
keyboard.listen();
- 注意:
.bind()
与 ECMAScript 5+ 兼容;如果您需要旧版浏览器的解决方案,Mozilla 已经发布了一个很好的替代方法来.bind()
使用functions
and .call()
:
https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Global_Objects/Function/bind
编辑:这是keyboard
使用这个新的模块化解决方案的实例化对象的样子: