Альтернатива для использования eval в Javascript

Question

В моем Javascript у меня есть своего рода фабрика. В зависимости от переданной строки Я создаю новый экземпляр этой строки в качестве объекта. Например

function getEmployee (employeeType, department){ 
     var fn = eval(employeeType); 
     return new fn(department); 
    } 

Я не хочу использовать Eval здесь. Я мог бы использовать оператор switch в свойстве employeeType, но мне не нужны зависимости от необходимости знать тип employeeType.

Есть ли другой способ, как я мог это сделать?

Answer 1

Чтобы расширить мой оригинальный комментарий, всякий раз, когда я столкнулся с проблемой, как это, общее решение заключается в создании какой-то глобальный реестр действительных имен:

function Manager() { /* manager employee type */ } 
function Intern() { /* intern employee type */ } 

var registeredTypes = { 
    "manager": Manager, 
    "intern": Intern 
}; 

Затем написать функцию:

function getEmployee (employeeType, department) { 
    if (!(employeeType in registeredTypes)) 
     throw "Invalid employee type: " + employeeType; 
    var fn = registeredTypes[employeeType]; 
    return new fn(department); 
} 

Однако, если вы действительно обеспокоены зависимостями, вы можете даже разделить на это немного дальше, обеспечивая глобальный метод для добавления новых типов сотрудников в реестр:

function registerEmployeeType(employeeType, fn) { 
    if (employeeType in registeredTypes) 
     throw "Employee type already registered: " + employeeType; 
    if (typeof(fn) != "function") 
     throw "May only register employee types as functions: " + employeeType; 
    registeredTypes[employeeType] = fn; 
} 

Теперь ваши основные функции, getEmployee и registerEmployeeType, могут быть объявлены сами по себе без какой-либо информации о зарегистрированных типах сотрудников. Они могут даже быть зарегистрированы из разных файлов сценариев при условии, что они выполняются после определяется registerEmployeeType. Например:

// core.js 
var registeredTypes = { }; 
function getEmployee (employeeType, department) ... 
function registerEmployeeType (employeeType, fn) ... 

// manager.js 
registerEmployeeType("manager", function() { /* manager employee type */ }); 

// intern.js 
registerEmployeeType("intern", function() { /* intern employee type */ }); 

// index.html 
<script src="core.js"></script> 
<script src="manager.js"></script> 
<script src="intern.js"></script> 

Answer 2

Вы должны передать функции для employeeType вместо строки (или более классический - объект с фиксированным способом), а затем вызвать функцию вместо eval его.

Answer 3

Спасибо p.s.w.g Я нашел ответ. Я регистрирую типы перед моей фабрикой. Я все еще передаю тип как строку на свой завод, но посмотрю на зарегистрированные типы.

Так что все ваши комментарии.

function getEmployee (employeeType, department){ 
    var fn = registeredTypes[employeeType]; 
    return new fn(department); 
}