JavaScript: refactoring, évitant Array.push()

Question

J'ai une fonction dans un objet comme celui-ci:

arrayofElements: function(item) { 

    var result = []; 

    if (item.isA) { 
     result.push(new Div('aClass', labels['A'])); 
    } 

    if (item.isC) { 
     result.push(new Div('cClass', labels['C'])); 
    } 

    if (item.isD) { 
     result.push(new Div('dClass', labels['D'])); 
    } 

    return result; 
}, 

Comment cela peut-il être refactorisé? Je n'aime pas avoir à pousser() chaque élément conditionnellement.

Answer 1

Vous pouvez déplacer les is* propriétés à un sous-objet de façon à les isoler, et boucle sur la sous les propriétés de -object

item.is={A:'aClass', C:'cClass'}; 
... 
arrayofElements: function(item) { 
    var result = []; 
    for (p in item.is) { 
     result.push(new Div(item.is[p], labels[p])); 
    } 
    return result; 
}, 

les valeurs item.is pourraient être les classes (comme indiqué), ou les valeurs item.is pourraient être des objets avec une propriété class, ou vous pouvez utiliser le nom de la propriété p comme un index dans anoth er objet pour obtenir les classes. Ce qui dépend en grande partie de ce que les item représentent et à quoi les classes d'éléments sont le plus étroitement associées.

Answer 2

arrayofElements: function(item) { 

var result = [], 
    items = ['A', 'C', 'D']; 
while(var i=0; i < items.length; i++) { 
    if(item["is" + items[i]]) { 
    result.push(new Div(items[i] + 'Class', labels[items[i]])) 
    } 
} 
return result; 
}, 

Rappelez-vous que obj.foo est le même que obj [ « foo »] en Javascript

Answer 3

Une variante de la solution de Larry:

arrayofElements: function(item) { 
    var result = [], letters = ['A','C','D'], ltr, i, len; 
    len = letters.length; 
    for (i = 0; i < len; i++) { 
    ltr = letters[i]; 
    if (item["is"+ltr]) { 
     result.push(new Div(ltr.toLowerCase()+"Class", labels[ltr])); 
    } 
    } 
    return result; 
}, 

Answer 4

Cette fonction évite d'utiliser push() et est fonctionnellement identique à la vôtre:

arrayofElements: function(item) { 

    if (item.isA) { 
     return [new Div('aClass', labels['A'])]; 
    } 

    if (item.isC) { 
     return [new Div('cClass', labels['C'])]; 
    } 

    if (item.isD) { 
     return [new Div('dClass', labels['D'])]; 
    } 

    return []; 
},