Node.js 中多个事件侦听器的代码效率

Question

我有一个代码可以在同一个 .js 文件中发出和处理事件。例如，发射事件的类型可以是“hello”、“world”、“user”等。所有事件处理函数（.on）都有类似的方法来处理错误和回调的核心函数内部的事件（完全相同）。有没有重构和改进冗余代码的建议。

localEvents.on("hello", function(request , reply){
    console.log("inside event for hello");
    hello.intialize() {
       // handle initialise error and callback //
    };
});

localEvents.on("world", function(request , reply){
    console.log("inside event for world");
    world.initialise() {
       // handle initialise error and callback //
    };
});

localEvents.on("user", function(request , reply){
    console.log("inside event for user");
    user.initialise() {
       // handle initialise error and callback //
    };
});

localEvents.emit(task, request, reply);

Answer 1

您可以创建一个辅助函数，例如：

function addHandler(evName, obj) {
  localEvents.on(evName, function(request, reply) {
    console.log("inside event for %s", evName);
    obj.initialise(function(err) {
       // handle initialise error and callback
    });
  });
}

然后就这样称呼它：

addHandler('hello', hello);
addHandler('world', world);
addHandler('user', user);

或者如果你真的想减少参数中的重复，你可以创建一个帮助器，比如：

function addHandler(evName) {
  var obj = eval(evName);
  localEvents.on(evName, function(request, reply) {
    console.log("inside event for %s", evName);
    obj.initialise(function(err) {
       // handle initialise error and callback
    });
  });
}

然后让你简单地做：

addHandler('hello');
addHandler('world');
addHandler('user');

Answer 2

这样做的简单方法 - 这种方式对于您可能需要在事件回调函数中进行的任何更改都会更加灵活

var eventsList = {
    "hello": function(request, reply) {
        console.log("inside event for hello");
        hello.initialise() {
            // handle initialise error and callback //
        };
    },
    "world": function(request, reply) {
        console.log("inside event for world");
        world.initialise() {
            // handle initialise error and callback //
        };
    },
    "user": function(request, reply) {
        console.log("inside event for user");
        user.initialise() {
            // handle initialise error and callback //
        };
    }
}

function generateEvents(events){
    for(var e in events){
        localEvents.on(e, events.e)
    }
}

generateEvents(eventsList)