creazione di oggetti JavaScript

Question

JavaScript newbie qui, stavo passando un certo codice js al lavoro quando mi sono imbattuto in una funzione di supporto per la creazione di oggetti, che è andato come questo

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... > 
} 

mentre questo non ottenere il lavoro fatto lo farei piacerebbe sapere se c'è un modo migliore e più pulito di scrivere questo.

Answer 1

Hai ragione, la if..then eccessiva o la switch logica è una code smell e può quasi sempre essere refactored in qualcosa di più elegante. In questo caso, una fabbrica basata su un nome può essere riscritta in un dizionario con chiave come quel nome e valore come la funzione per restituire

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

esempio dal vivo: http://jsfiddle.net/KkMnd/

EDIT: Quella linea in il metodo createElement potrebbe/dovrebbe prima verificare che qualcosa sia stato configurato per il TYPES.* passato. Un buon modo è controllare che ci sia un elemento nel dizionario prima di provare a chiamare quel metodo.

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

Answer 2

Sarebbe un po 'più pulito ma semanticamente lo stesso per utilizzare una dichiarazione di commutazione.

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