Efficacité du code dans Node.js pour plusieurs écouteurs d'événements

Question

J'ai un code qui émet et gère les événements dans le même fichier .js. Les événements d'émission peuvent être de type "bonjour", "monde", "utilisateur", etc. à titre d'exemple. Toutes les fonctions du gestionnaire d'événements (.on) ont une approche similaire pour gérer les événements à l'intérieur des fonctions de base des erreurs et des rappels (exactement la même chose). Y a-t-il une suggestion pour refactoriser et améliorer le code redondant.

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

Vous pouvez créer une fonction d'aide, comme:

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 
    }); 
    }); 
} 

Ensuite, il suffit l'appeler comme:

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

Ou si vous voulez vraiment réduire encore la répétition dans la arguments que vous pourriez créer un assistant comme:

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 
    }); 
    }); 
} 

vous permettant de puis simplement faire:

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

Answer 2

Une façon simple de le faire - De cette façon, sera plus souple à tout changement que vous pourriez avoir besoin de faire dans les événements de rappel fonctions

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)