creación de objetos JavaScript

Question

JavaScript novato aquí, yo estaba pasando por algo de código js en el trabajo cuando me encontré con una función de ayuda para la creación de objetos, que fue como esto

createElement = function(name, data){ 
    if(name == TYPES.TEXT){ 
    return new Text(data); 
    } 
    else if(name == TYPES.WORD){ 
    return new Word(data); 
    } 
    else if(name == TYPES.PARAGRAPH){ 
    return new Paragraph(data); 
    } 
    else if(name == TYPES.TABLE){ 
    return new Table(data); 
    } 
    <list goes on and on and on... > 
} 

mientras que esto no hacer el trabajo lo haría desea saber si hay una forma mejor y más limpia de escribir esto.

Answer 1

Tienes razón, la if..then o switch lógica es code smell y casi siempre se puede refactorizar en algo más elegante. En este caso, una fábrica en base a un nombre puede ser reprogramado para utilizar un diccionario con la clave como el nombre y el valor que la función para devolver

var dictionary = {}; 
dictionary[TYPES.TEXT] = Text; 
dictionary[TYPES.WORD] = Word; 
dictionary[TYPES.PARAGRAPH] = Paragraph; 
dictionary[TYPES.TABLE] = Table; 

createElement = function(name, data){ 
    return new dictionary[name](data); 
} 

ejemplo vivo: http://jsfiddle.net/KkMnd/

EDIT: esa línea en el método createElement podría/debería verificar primero que algo esté configurado para el TYPES.* pasado. Una buena forma es verificar que haya un elemento en el diccionario antes de intentar llamar a ese método.

return (typeof dictionary[name] == 'function') ? new dictionary[name](data) : some_default_value; 

Answer 2

Sería un poco más limpio, pero semánticamente lo mismo para usar una declaración de interruptor.

function createElement(name,data){ 
switch(name) 
{ 
case TYPES.TEXT: 
    return new Text(data) 
    break; 
case TYPES.WORD: 
    return new WORD(data) 
    break; 
default: 
    // etc. code to be executed if no values match 
} 
}