cette utilisation d'eval dans un serveur de noeud est-elle une mauvaise idée?

Question

Je souhaite créer un objet dynamiquement dans un noeud. Pour ce faire, j'ai utilisé un code comme celui-ci. Cette utilisation d'eval dans un serveur de noeud est-elle une mauvaise idée?

var a1 = require(./a1.js), 
    a2 = require(./a2.js), 
    ... 
    aN = require(./aN.js); 

    function createObj(pObjName, pObjValue){ 
     var tmp = new eval(pObjName)(pObjValue); 
     //where pObjName is a1 or a1 or .... or aN 
    } 

Answer 1

D'après ce que vous montrez, il n'y a pas besoin d'utiliser eval:

const Classes = { 
    a1 : require('./a1'), 
    a2 : require('./a2'), 
    ... 
}; 

function createObj(pObjName, pObjValue){ 
    var tmp = new Classes[pObjName](pObjValue); 
    ... 
} 

Answer 2

Vous semblez vouloir créer des objets avec un ensemble de propriétés?

Vous pouvez regarder le Object.create()-method

Answer 3

Si vous voulez vous épargner le temps de déclarer tous les dépendances, vous pouvez même écrire une fonction sans utiliser eval qui gère le cas encore plus efficace:

function createObj(pObjName, pObjValue) { 
    var tmp = new (require('./' + pObjName))(pObjValue); 
    // ... 
} 

S'il vous plaît noter que ce n'est sûr que si createObj() est garanti d'être invoqué par un pObjName qui est ce que vous attendez, sinon vous devrez valider d'abord, peut-être quelque chose comme ceci:

function createObj(pObjName, pObjValue) { 
    if (!/^a\d$/.test(pObjName)) { 
    throw new TypeError('invalid name') 
    } 

    var tmp = new (require('./' + pObjName))(pObjValue); 
    // ... 
}