Question

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

当我开始时，我加载了所有已定义驱动程序的列表。然后我遍历定义以执行以下操作：

创建一个驱动程序实例，
听取驱动程序上的事件，
告诉司机连接。

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

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

我创建了一个测试示例，通过使用几个简单对象和一个计时器来重现我的问题。这将需要在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时，变量都有意义。但是，当事件触发并且makeOnTimerFunction返回的函数被调用时，变量总是相同的。它们不会反映首次调用makeOnTimerFunction时的值。

Answer 1

这是你的问题：

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

您正在有效地创建一个全局变量self，每次运行startTimer方法时都会被覆盖。最后，它将被覆盖，因此它指向最后一个驱动程序实例，因此意外结果。

解决方案：使用var创建一个范围为方法的新变量：

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

关闭为发出的事件创建侦听器的问题

1 个答案: