为发出的事件创build侦听器的闭包问题

我已经实现了封装与外部设备交互的驱动程序的对象。 当驱动程序收到数据时,它会发出一个事件。

当我开始时,我加载一个所有定义的驱动程序的列表。 然后我遍历定义来执行以下操作:

  • 创build一个驱动实例,
  • 听取司机的事件
  • 告诉驱动程序连接。

作为发出事件的侦听器的一部分,我定义了一个函数来传递设备代表的一些细节。

但是,我的代码似乎有问题。 当一个驱动程序发出一个事件时,被触发的函数只显示一组细节。 这些细节是在for循环中定义的最后一个,我创build驱动程序并监听事件。

我创build了一个testing示例,通过使用几个简单的对象和计时器来重新创build我的问题。 这将需要在node.js中运行以查看问题。

var EventEmitter = require('events').EventEmitter; var util = require('util'); // define vehicle object function Vehicle(options) { options = options || {}; this.id = ((options.id != null) ? options.id : -1); this.name = ((options.name != null) ? options.name : 'unknown'); } util.inherits(Driver, EventEmitter); // define driver object function Driver(options) { options = options || {}; this.id = ((options.id != null) ? options.id : -1); this.name = ((options.name != null) ? options.name : 'unknown'); } Driver.prototype.startTimer = function(delay) { self = this; console.log('starting timer for ' + self.name + ' with a delay of ' + delay); setTimeout(function() { console.log('firing timer for ' + self.name); self.emit('timer', 'some data'); }, delay); } // function to create listener function makeOnTimerFunction(driver, vehicle) { console.log("creating function for d name : " + driver.name + ' d id: ' + driver.id + ' v id: ' + vehicle.id + ' v name: ' + vehicle.name); return function(d) { console.log(" d name: " + driver.name + ' d id: ' + driver.id + ' v id: ' + vehicle.id + ' v name: ' + vehicle.name + ' d: ' + d); } } var vehicles = new Array(); vehicles.push(new Vehicle({id: 1001, name: 'Max'})); vehicles.push(new Vehicle({id: 1002, name: 'Cheif'})); vehicles.push(new Vehicle({id: 1003, name: 'Seigfreid'})); var driver = null; for (var i = 0; i < 3; i++) { driver = new Driver({ id: vehicles[i].id, name: vehicles[i].name, }); driver.on('timer', makeOnTimerFunction(driver, vehicles[i])); driver.startTimer(1000 * i); } 

“`

我得到的输出如下:

 creating function for d name : Max d id: 1001 v id: 1001 v name: Max starting timer for Max with a delay of 0 creating function for d name : Cheif d id: 1002 v id: 1002 v name: Cheif starting timer for Cheif with a delay of 1000 creating function for d name : Seigfreid d id: 1003 v id: 1003 v name: Seigfreid starting timer for Seigfreid with a delay of 2000 firing timer for Seigfreid d name: Seigfreid d id: 1003 v id: 1003 v name: Seigfreid d: some data firing timer for Seigfreid d name: Seigfreid d id: 1003 v id: 1003 v name: Seigfreid d: some data firing timer for Seigfreid d name: Seigfreid d id: 1003 v id: 1003 v name: Seigfreid d: some data 

我期望看到如下:

 creating function for d name : Max d id: 1001 v id: 1001 v name: Max starting timer for Max with a delay of 0 creating function for d name : Cheif d id: 1002 v id: 1002 v name: Cheif starting timer for Cheif with a delay of 1000 creating function for d name : Seigfreid d id: 1003 v id: 1003 v name: Seigfreid starting timer for Seigfreid with a delay of 2000 firing timer for Max d name: Max d id: 1001 v id: 1001 v name: Max d: some data firing timer for Chief d name: Chief d id: 1002 v id: 1002 v name: Chief d: some data firing timer for Seigfried d name: Seigfreid d id: 1003 v id: 1003 v name: Seigfreid d: some data 

问题是计时器总是引用最后一个名为Seigfreid的对象。

—-编辑—-

只是为了清除问题,问题似乎与makeOnTimerFunction函数。 它返回的函数是事件发生时触发的函数。 当makeOnTimerFunction ,variables都是有意义的。 但是,当事件触发并调用makeOnTimerFunction返回的函数时,variables总是相同的。 当第一次调用makeOnTimerFunction时,它们并不反映这些值。

这是你的问题:

 Driver.prototype.startTimer = function(delay) { self = this; <---- 

您正在有效地创build一个全局variablesself ,每次运行startTimer方法时都会被覆盖。 最后,它会被覆盖,所以它指向最后一个驱动程序实例,因此你意想不到的结果。

解决scheme:使用var创build一个新的variables作用域的方法:

 Driver.prototype.startTimer = function(delay) { var self = this;